[ticket/13904] No longer use fileerror class for extending filespec class

PHPBB3-13904
author: Marc Alexander <admin@m-a-styles.de> 2015-06-02 11:48:55 +0200
committer: Marc Alexander <admin@m-a-styles.de> 2015-09-09 08:27:46 +0200
commit: a53825ad760cc8437d8c26eb1f947622c0fcf229 (patch)
tree: 62feed6b143c62736d96a50124440dd600534820
parent: eb11973ea8af41623a9e6e6c320cb1fb0df7a222 (diff)
download: forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar
forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar.gz
forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar.bz2
forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar.xz
forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.zip
3 files changed, 40 insertions, 28 deletions
diff --git a/phpBB/phpbb/files/filespec.php b/phpBB/phpbb/files/filespec.php
index 2fdba2d793..d14c9a226d 100644
--- a/phpBB/phpbb/files/filespec.php
+++ b/phpBB/phpbb/files/filespec.php
@@ -119,6 +119,20 @@ class filespec
 	}
 
 	/**
+	 * Set error in error array
+	 *
+	 * @param mixed $error Content for error array
+	 *
+	 * @return \phpbb\files\filespec This instance of the filespec class
+	 */
+	public function set_error($error)
+	{
+		$this->error[] = $error;
+
+		return $this;
+	}
+
+	/**
 	 * Cleans destination filename
 	 *
 	 * @param real|unique|unique_ext $mode real creates a realname, filtering some characters, lowering every character. Unique creates an unique filename
diff --git a/phpBB/phpbb/files/upload.php b/phpBB/phpbb/files/upload.php
index 7ca29efe1a..291cdb266c 100644
--- a/phpBB/phpbb/files/upload.php
+++ b/phpBB/phpbb/files/upload.php
@@ -306,14 +306,12 @@ class upload
 
 		if (!preg_match('#^(https?://).*?\.(' . implode('|', $this->allowed_extensions) . ')$#i', $upload_url, $match))
 		{
-			$file = new fileerror($user->lang[$this->error_prefix . 'URL_INVALID']);
-			return $file;
+			return $this->factory->get('filespec')->set_error($user->lang[$this->error_prefix . 'URL_INVALID']);
 		}
 
 		if (empty($match[2]))
 		{
-			$file = new fileerror($user->lang[$this->error_prefix . 'URL_INVALID']);
-			return $file;
+			return $this->factory->get('filespec')->set_error($user->lang[$this->error_prefix . 'URL_INVALID']);e;
 		}
 
 		$url = parse_url($upload_url);
@@ -362,8 +360,7 @@ class upload
 
 		if (!($fsock = @fsockopen($host, $port, $errno, $errstr)))
 		{
-			$file = new fileerror($user->lang[$this->error_prefix . 'NOT_UPLOADED']);
-			return $file;
+			return $this->factory->get('filespec')->set_error($user->lang[$this->error_prefix . 'NOT_UPLOADED']);
 		}
 
 		// Make sure $path not beginning with /
@@ -404,8 +401,7 @@ class upload
 				{
 					$max_filesize = get_formatted_filesize($remote_max_filesize, false);
 
-					$file = new fileerror(sprintf($user->lang[$this->error_prefix . 'WRONG_FILESIZE'], $max_filesize['value'], $max_filesize['unit']));
-					return $file;
+					return $this->factory->get('filespec')->set_error(sprintf($user->lang[$this->error_prefix . 'WRONG_FILESIZE'], $max_filesize['value'], $max_filesize['unit']));
 				}
 
 				$data .= $block;
@@ -432,14 +428,12 @@ class upload
 						{
 							$max_filesize = get_formatted_filesize($remote_max_filesize, false);
 
-							$file = new fileerror(sprintf($user->lang[$this->error_prefix . 'WRONG_FILESIZE'], $max_filesize['value'], $max_filesize['unit']));
-							return $file;
+							return $this->factory->get('filespec')->set_error(sprintf($user->lang[$this->error_prefix . 'WRONG_FILESIZE'], $max_filesize['value'], $max_filesize['unit']));
 						}
 					}
 					else if (stripos($line, '404 not found') !== false)
 					{
-						$file = new fileerror($user->lang[$this->error_prefix . 'URL_NOT_FOUND']);
-						return $file;
+						return $this->factory->get('filespec')->set_error($this->error_prefix . 'URL_NOT_FOUND');
 					}
 				}
 			}
@@ -449,16 +443,14 @@ class upload
 			// Cancel upload if we exceed timeout
 			if (!empty($stream_meta_data['timed_out']) || time() >= $timer_stop)
 			{
-				$file = new fileerror($user->lang[$this->error_prefix . 'REMOTE_UPLOAD_TIMEOUT']);
-				return $file;
+				return $this->factory->get('filespec')->set_error($this->error_prefix . 'REMOTE_UPLOAD_TIMEOUT');
 			}
 		}
 		@fclose($fsock);
 
 		if (empty($data))
 		{
-			$file = new fileerror($user->lang[$this->error_prefix . 'EMPTY_REMOTE_DATA']);
-			return $file;
+			return $this->factory->get('filespec')->set_error($this->error_prefix . 'EMPTY_REMOTE_DATA');
 		}
 
 		$tmp_path = (!@ini_get('safe_mode') || strtolower(@ini_get('safe_mode')) == 'off') ? false : $phpbb_root_path . 'cache';
@@ -466,8 +458,7 @@ class upload
 
 		if (!($fp = @fopen($filename, 'wb')))
 		{
-			$file = new fileerror($user->lang[$this->error_prefix . 'NOT_UPLOADED']);
-			return $file;
+			return $this->factory->get('filespec')->set_error($this->error_prefix . 'NOT_UPLOADED');
 		}
 
 		$upload_ary['size'] = fwrite($fp, $data);
diff --git a/tests/upload/fileupload_test.php b/tests/upload/fileupload_test.php
index fe6096aebf..d4785c782c 100644
--- a/tests/upload/fileupload_test.php
+++ b/tests/upload/fileupload_test.php
@@ -22,6 +22,12 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	private $filesystem;
 
+	/** @var \Symfony\Component\DependencyInjection\ContainerInterface */
+	protected $container;
+
+	/** @var \phpbb\files\factory */
+	protected $factory;
+
 	protected function setUp()
 	{
 		// Global $config required by unique_id
@@ -44,12 +50,13 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 		$this->filesystem = $phpbb_filesystem = new \phpbb\filesystem\filesystem();
 
-		$phpbb_container = new phpbb_mock_container_builder($this->phpbb_root_path, $this->phpEx);
-		$phpbb_container->set('files.filespec', new \phpbb\files\filespec(
+		$this->container = new phpbb_mock_container_builder($this->phpbb_root_path, $this->phpEx);
+		$this->container->set('files.filespec', new \phpbb\files\filespec(
 			$this->filesystem,
 			new \phpbb\mimetype\guesser(array(
 				'mimetype.extension_guesser' => new \phpbb\mimetype\extension_guesser(),
 			))));
+		$this->factory = new \phpbb\files\factory($this->container);
 
 		$this->path = __DIR__ . '/fixture/';
 	}
@@ -76,7 +83,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_common_checks_invalid_extension()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('png'))
 			->set_max_filesize(100);
 		$file = $this->gen_valid_filespec();
@@ -86,7 +93,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_common_checks_invalid_filename()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('jpg'))
 			->set_max_filesize(100);
 		$file = $this->gen_valid_filespec();
@@ -97,7 +104,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_common_checks_too_large()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('jpg'))
 			->set_max_filesize(100);
 		$file = $this->gen_valid_filespec();
@@ -108,7 +115,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_common_checks_valid_file()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('jpg'))
 			->set_max_filesize(1000);
 		$file = $this->gen_valid_filespec();
@@ -118,7 +125,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_local_upload()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('jpg'))
 			->set_max_filesize(1000);
 
@@ -130,7 +137,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_move_existent_file()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('jpg'))
 			->set_max_filesize(1000);
 
@@ -144,7 +151,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_move_existent_file_overwrite()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(array('jpg'))
 			->set_max_filesize(1000);
 
@@ -159,7 +166,7 @@ class phpbb_fileupload_test extends phpbb_test_case
 
 	public function test_valid_dimensions()
 	{
-		$upload = new \phpbb\files\upload($this->filesystem);
+		$upload = new \phpbb\files\upload($this->filesystem, $this->factory);
 		$upload->set_allowed_extensions(false)
 			->set_max_filesize(false)
 			->set_allowed_dimensions(1, 1, 100, 100);
author	Marc Alexander <admin@m-a-styles.de>	2015-06-02 11:48:55 +0200
committer	Marc Alexander <admin@m-a-styles.de>	2015-09-09 08:27:46 +0200
commit	a53825ad760cc8437d8c26eb1f947622c0fcf229 (patch)
tree	62feed6b143c62736d96a50124440dd600534820
parent	eb11973ea8af41623a9e6e6c320cb1fb0df7a222 (diff)
download	forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar.gz forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar.bz2 forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.tar.xz forums-a53825ad760cc8437d8c26eb1f947622c0fcf229.zip