Icybee / module-files

lib/Block/ManageBlock/DownloadColumn.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  */
 class DownloadColumn extends ManageBlock\Column
 {
-	public function __construct(ManageBlock $manager, $id, array $options=[])
+	public function __construct(ManageBlock $manager, $id, array $options = [])
 	{
 		parent::__construct($manager, $id, [
 

lib/Block/EditBlock.php

Braces +1 added lines, -2 removed lines

@@ -75,8 +75,7 @@
 			try
 			{
 				$path = $this->record->pathname->relative;
-			}
-			catch (\Exception $e)
+			} catch (\Exception $e)
 			{
 				#
 				# the associated file might have disappeared

Spacing +6 added lines, -6 removed lines

@@ -36,8 +36,8 @@ 
 	{
 		parent::add_assets($document);
 
-		$document->css->add(Root\DIR . 'public/edit.css');
-		$document->js->add(Root\DIR . 'public/edit.js');
+		$document->css->add(Root\DIR.'public/edit.css');
+		$document->js->add(Root\DIR.'public/edit.js');
 	}
 
 	protected function lazy_get_values()
@@ -52,7 +52,7 @@ 
 
 	protected function lazy_get_children()
 	{
-		$folder = \ICanBoogie\REPOSITORY . 'tmp';
+		$folder = \ICanBoogie\REPOSITORY.'tmp';
 
 		if (!is_writable($folder))
 		{
@@ -60,7 +60,7 @@ 
 
 				Element::CHILDREN => [
 
-					$this->t('The folder %folder is not writable !', [ '%folder' => $folder ])
+					$this->t('The folder %folder is not writable !', ['%folder' => $folder])
 
 				]
 			];
@@ -117,9 +117,9 @@ 
 
 				Form::LABEL => 'file',
 				Element::REQUIRED => empty($nid),
-				\Brickrouge\File::FILE_WITH_LIMIT => $this->app->site->metas[$this->module->flat_id . '.max_file_size'],
+				\Brickrouge\File::FILE_WITH_LIMIT => $this->app->site->metas[$this->module->flat_id.'.max_file_size'],
 				Element::WEIGHT => -100,
-				\Brickrouge\File::T_UPLOAD_URL => Operation::encode($this->module->id . '/upload'),
+				\Brickrouge\File::T_UPLOAD_URL => Operation::encode($this->module->id.'/upload'),
 
 				'value' => $path
 

lib/Facets/SizeCriterion.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 		{
 			$value = $modifiers['size'];
 
-			if (in_array($value, [ 'l', 'm', 's' ]))
+			if (in_array($value, ['l', 'm', 's']))
 			{
 				$filters['size'] = $value;
 			}

Braces +1 added lines, -2 removed lines

@@ -32,8 +32,7 @@
 			if (in_array($value, [ 'l', 'm', 's' ]))
 			{
 				$filters['size'] = $value;
-			}
-			else
+			} else
 			{
 				unset($filters['size']);
 			}

lib/Storage/Base64.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	static public function encode($data)
 	{
-		return strtr(base64_encode($data), [ '+' => '-', '/' => '_' ]);
+		return strtr(base64_encode($data), ['+' => '-', '/' => '_']);
 	}
 
 	/**
@@ -37,7 +37,7 @@ 
 	 */
 	static public function decode($data)
 	{
-		return base64_decode(strtr($data, [ '-' => '+', '_' => '/' ]));
+		return base64_decode(strtr($data, ['-' => '+', '_' => '/']));
 	}
 
 	/**

lib/Operation/SaveOperation.php

Braces +2 added lines, -4 removed lines

@@ -114,8 +114,7 @@ 
 			$pathname = \ICanBoogie\REPOSITORY . 'tmp' . DIRECTORY_SEPARATOR . $filename;
 
 			$file->move($pathname);
-		}
-		else if ($path && strpos($path, \ICanBoogie\strip_root(\ICanBoogie\REPOSITORY . "files")) !== 0)
+		} else if ($path && strpos($path, \ICanBoogie\strip_root(\ICanBoogie\REPOSITORY . "files")) !== 0)
 		{
 			$file = $this->resolve_request_file_from_pathname($path);
 
@@ -175,8 +174,7 @@ 
 
 				]);
 			}
-		}
-		else if (!$this->key)
+		} else if (!$this->key)
 		{
 			$errors->add(File::HTTP_FILE, "File is required.");
 		}

Spacing +6 added lines, -6 removed lines

@@ -111,17 +111,17 @@ 
 		if ($file && $file->is_valid)
 		{
 			$filename = \ICanBoogie\generate_v4_uuid();
-			$pathname = \ICanBoogie\REPOSITORY . 'tmp' . DIRECTORY_SEPARATOR . $filename;
+			$pathname = \ICanBoogie\REPOSITORY.'tmp'.DIRECTORY_SEPARATOR.$filename;
 
 			$file->move($pathname);
 		}
-		else if ($path && strpos($path, \ICanBoogie\strip_root(\ICanBoogie\REPOSITORY . "files")) !== 0)
+		else if ($path && strpos($path, \ICanBoogie\strip_root(\ICanBoogie\REPOSITORY."files")) !== 0)
 		{
 			$file = $this->resolve_request_file_from_pathname($path);
 
 			if (!$file)
 			{
-				$this->response->errors->add(File::HTTP_FILE, "Invalid or deleted file: %pathname", [ 'pathname' => $path ]);
+				$this->response->errors->add(File::HTTP_FILE, "Invalid or deleted file: %pathname", ['pathname' => $path]);
 			}
 		}
 
@@ -161,7 +161,7 @@ 
 				$error_message = $errors->add(
 					File::HTTP_FILE,
 					"Maximum file size is :size Mb",
-					[ ':size' => round($max_file_size / 1024) ]
+					[':size' => round($max_file_size / 1024)]
 				);
 			}
 
@@ -170,7 +170,7 @@ 
 				$error_message = $errors->add(
 					File::HTTP_FILE,
 					"Only the following file types are accepted: %accepted.",
-					[ '%accepted' => implode(', ', $this->accept)
+					['%accepted' => implode(', ', $this->accept)
 				]);
 			}
 
@@ -220,7 +220,7 @@ 
 	protected function resolve_request_file_from_pathname($pathname)
 	{
 		$filename = basename($pathname);
-		$info_pathname = \ICanBoogie\REPOSITORY . 'tmp' . DIRECTORY_SEPARATOR . $filename . '.info';
+		$info_pathname = \ICanBoogie\REPOSITORY.'tmp'.DIRECTORY_SEPARATOR.$filename.'.info';
 
 		if (!file_exists($info_pathname))
 		{

lib/Module.php

Braces +3 added lines, -6 removed lines

@@ -45,8 +45,7 @@ 
 				mkdir($path);
 
 				file_put_contents($path . DIRECTORY_SEPARATOR . '.htaccess', 'Deny from all');
-			}
-			else
+			} else
 			{
 				$errors->add($this->id, "Unable to create %directory directory, its parent is not writable.", [ '%directory' => $path ]);
 			}
@@ -67,8 +66,7 @@ 
 				mkdir($path);
 
 				file_put_contents($path . DIRECTORY_SEPARATOR . '.htaccess', 'Allow from all');
-			}
-			else
+			} else
 			{
 				$errors->add($this->id, "Unable to create %directory directory, its parent is not writable", [ '%directory' => $path ]);
 			}
@@ -89,8 +87,7 @@ 
 				mkdir($path);
 
 				file_put_contents($path . DIRECTORY_SEPARATOR . '.htaccess', 'Deny from all');
-			}
-			else
+			} else
 			{
 				$errors->add($this->id, 'Unable to create %directory directory, its parent is not writable', [ '%directory' => $path ]);
 			}

Spacing +16 added lines, -16 removed lines

@@ -34,7 +34,7 @@ 
 		# $repository/tmp
 		#
 
-		$path = $repository . 'tmp';
+		$path = $repository.'tmp';
 
 		if (!file_exists($path))
 		{
@@ -44,11 +44,11 @@ 
 			{
 				mkdir($path);
 
-				file_put_contents($path . DIRECTORY_SEPARATOR . '.htaccess', 'Deny from all');
+				file_put_contents($path.DIRECTORY_SEPARATOR.'.htaccess', 'Deny from all');
 			}
 			else
 			{
-				$errors->add($this->id, "Unable to create %directory directory, its parent is not writable.", [ '%directory' => $path ]);
+				$errors->add($this->id, "Unable to create %directory directory, its parent is not writable.", ['%directory' => $path]);
 			}
 		}
 
@@ -56,7 +56,7 @@ 
 		# $repository/files
 		#
 
-		$path = $repository . 'files';
+		$path = $repository.'files';
 
 		if (!file_exists($path))
 		{
@@ -66,11 +66,11 @@ 
 			{
 				mkdir($path);
 
-				file_put_contents($path . DIRECTORY_SEPARATOR . '.htaccess', 'Allow from all');
+				file_put_contents($path.DIRECTORY_SEPARATOR.'.htaccess', 'Allow from all');
 			}
 			else
 			{
-				$errors->add($this->id, "Unable to create %directory directory, its parent is not writable", [ '%directory' => $path ]);
+				$errors->add($this->id, "Unable to create %directory directory, its parent is not writable", ['%directory' => $path]);
 			}
 		}
 
@@ -78,7 +78,7 @@ 
 		# $repository/files-index
 		#
 
-		$path = $repository . 'files-index';
+		$path = $repository.'files-index';
 
 		if (!file_exists($path))
 		{
@@ -88,11 +88,11 @@ 
 			{
 				mkdir($path);
 
-				file_put_contents($path . DIRECTORY_SEPARATOR . '.htaccess', 'Deny from all');
+				file_put_contents($path.DIRECTORY_SEPARATOR.'.htaccess', 'Deny from all');
 			}
 			else
 			{
-				$errors->add($this->id, 'Unable to create %directory directory, its parent is not writable', [ '%directory' => $path ]);
+				$errors->add($this->id, 'Unable to create %directory directory, its parent is not writable', ['%directory' => $path]);
 			}
 		}
 
@@ -118,22 +118,22 @@ 
 		# $repository/tmp
 		#
 
-		$path = $repository . 'tmp';
+		$path = $repository.'tmp';
 
 		if (!is_dir($path))
 		{
-			$errors->add($this->id, "The %directory directory is missing.", [ '%directory' => $path ]);
+			$errors->add($this->id, "The %directory directory is missing.", ['%directory' => $path]);
 		}
 
 		#
 		# $repository/files
 		#
 
-		$path = $repository . 'files';
+		$path = $repository.'files';
 
 		if (!is_dir($path))
 		{
-			$errors->add($this->id, "The %directory directory is missing.", [ '%directory' => $path ]);
+			$errors->add($this->id, "The %directory directory is missing.", ['%directory' => $path]);
 		}
 
 		return parent::is_installed($errors);
@@ -141,18 +141,18 @@ 
 
 	public function clean_temporary_files($lifetime = 3600)
 	{
-		$path = \ICanBoogie\REPOSITORY . 'tmp';
+		$path = \ICanBoogie\REPOSITORY.'tmp';
 
 		if (!is_dir($path))
 		{
-			\ICanBoogie\log_error('The directory %directory does not exists', [ '%directory' => $path ]);
+			\ICanBoogie\log_error('The directory %directory does not exists', ['%directory' => $path]);
 
 			return;
 		}
 
 		if (!is_writable($path))
 		{
-			\ICanBoogie\log_error('The directory %directory is not writable', [ '%directory' => $path ]);
+			\ICanBoogie\log_error('The directory %directory is not writable', ['%directory' => $path]);
 
 			return;
 		}

updates.php

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
 
 		foreach ($model->select("nid, path")->mode(\PDO::FETCH_NUM) as list($nid, $path))
 		{
-			$update('.' . pathinfo($path, PATHINFO_EXTENSION), $nid);
+			$update('.'.pathinfo($path, PATHINFO_EXTENSION), $nid);
 		}
 	}
 
@@ -84,7 +84,7 @@ 
 
 		foreach ($model->select("nid, uuid, path")->mode(\PDO::FETCH_NUM) as list($nid, $uuid, $path))
 		{
-			$path = $document_root . $path;
+			$path = $document_root.$path;
 
 			if (!file_exists($path))
 			{
@@ -94,7 +94,7 @@ 
 			}
 
 			$hash = $storage->add($path)->hash;
-			$index->add(IndexKey::from([ $nid, $uuid, $hash ]));
+			$index->add(IndexKey::from([$nid, $uuid, $hash]));
 		}
 
 		$model->remove_column('path');

lib/Hooks.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
 	{
 		static $index;
 
-		return $index ?: $index = new FileStorageIndex(\ICanBoogie\REPOSITORY . 'files-index');
+		return $index ?: $index = new FileStorageIndex(\ICanBoogie\REPOSITORY.'files-index');
 	}
 
 	/**
@@ -39,6 +39,6 @@ 
 	{
 		static $manager;
 
-		return $manager ?: $manager = new FileStorage(\ICanBoogie\REPOSITORY . 'files', $app->file_storage_index);
+		return $manager ?: $manager = new FileStorage(\ICanBoogie\REPOSITORY.'files', $app->file_storage_index);
 	}
 }

lib/Routing/FilesController.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 		return new FileResponse($pathname, $this->request, [
 
 			FileResponse::OPTION_ETAG => $pathname->hash,
-			FileResponse::OPTION_FILENAME => $record->title . $record->extension,
+			FileResponse::OPTION_FILENAME => $record->title.$record->extension,
 			FileResponse::OPTION_MIME => $record->mime
 
 		]);