nextcloud / gallery

lib/Controller/HttpError.php

Spacing +3 added lines, -3 removed lines

@@ -50,11 +50,11 @@ 
 
 		// If the exception is not of type ForbiddenServiceException only show a
 		// generic error message to avoid leaking information.
-		if(!($exception instanceof ForbiddenServiceException)) {
+		if (!($exception instanceof ForbiddenServiceException)) {
 			$logger->logException($exception, ['app' => 'gallery']);
 			$message = sprintf('An error occurred. Request ID: %s', $request->getId());
 		} else {
-			$message = $exception->getMessage() . ' (' . $code . ')';
+			$message = $exception->getMessage().' ('.$code.')';
 		}
 
 		return new JSONResponse(
@@ -77,7 +77,7 @@ 
 		$message = $exception->getMessage();
 		$code = $this->getHttpStatusCode($exception);
 		$url = $urlGenerator->linkToRoute(
-			$appName . '.page.error_page', ['code' => $code]
+			$appName.'.page.error_page', ['code' => $code]
 		);
 
 		$response = new RedirectResponse($url);

lib/Controller/PathManipulation.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 		$subFolders = explode('/', $relativePath);
 
 		if (count($subFolders) > 2) {
-			$reducedPath = $currFolderPath . $subFolders[0] . '/' . array_pop($subFolders);
+			$reducedPath = $currFolderPath.$subFolders[0].'/'.array_pop($subFolders);
 		} else {
 			$reducedPath = $path;
 		}

lib/Middleware/CheckMiddleware.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 	 */
 	private function redirectToErrorPage($message, $code) {
 		$url = $this->urlGenerator->linkToRoute(
-			$this->appName . '.page.error_page', ['code' => $code]
+			$this->appName.'.page.error_page', ['code' => $code]
 		);
 
 		$response = new RedirectResponse($url);

lib/Middleware/CheckException.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 	 * @param int $code the HTTP status code
 	 */
 	public function __construct($msg, $code = 0) {
-		Util::writeLog('gallery', 'Exception: ' . $msg . ' (' . $code . ')', Util::ERROR);
+		Util::writeLog('gallery', 'Exception: '.$msg.' ('.$code.')', Util::ERROR);
 		parent::__construct($msg, $code);
 	}
 

lib/Middleware/EnvCheckMiddleware.php

Spacing +3 added lines, -3 removed lines

@@ -180,7 +180,7 @@ 
 		) {
 			$message =
 				'Passed token seems to be valid, but it does not contain all necessary information . ("'
-				. $token . '")';
+				. $token.'")';
 			throw new CheckException($message, Http::STATUS_NOT_FOUND);
 		}
 	}
@@ -194,7 +194,7 @@ 
 	 */
 	private function checkItemType($share) {
 		if ($share->getNodeType() === null) {
-			$message = 'No item type set for share id: ' . $share->getId();
+			$message = 'No item type set for share id: '.$share->getId();
 			throw new CheckException($message, Http::STATUS_NOT_FOUND);
 		}
 	}
@@ -236,7 +236,7 @@ 
 			$this->checkPassword($share, $password);
 		} else {
 			throw new CheckException(
-				'Unknown share type ' . $share->getShareType() . ' for share id '
+				'Unknown share type '.$share->getShareType().' for share id '
 				. $share->getId(), Http::STATUS_NOT_FOUND
 			);
 		}

lib/Config/ConfigValidator.php

Spacing +4 added lines, -4 removed lines

@@ -32,8 +32,8 @@ 
 
 		switch ($key) {
 			case 'sorting':
-				$safe = $this->isSortingSafe('type',$parsedConfigItem, $safe);
-				$safe = $this->isSortingSafe('order',$parsedConfigItem, $safe);
+				$safe = $this->isSortingSafe('type', $parsedConfigItem, $safe);
+				$safe = $this->isSortingSafe('order', $parsedConfigItem, $safe);
 				break;
 			case 'design':
 				$safe = $this->isDesignColourSafe($parsedConfigItem, $safe);
@@ -50,9 +50,9 @@ 
 	 * @param bool $safe whether the current config has been deemed safe to use so far
 	 * @return bool
 	 */
-	private function isSortingSafe($key,$parsedConfigItem, $safe) {
+	private function isSortingSafe($key, $parsedConfigItem, $safe) {
 		if ($safe && array_key_exists($key, $parsedConfigItem)) {
-			$safe = $safe && $this->sortingValidator($key, $parsedConfigItem[ $key ]);
+			$safe = $safe && $this->sortingValidator($key, $parsedConfigItem[$key]);
 		}
 
 		return $safe;

lib/Config/ConfigException.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 	 * @param string $msg the message contained in the exception
 	 */
 	public function __construct($msg) {
-		Util::writeLog('gallery', 'Exception: ' . $msg, Util::ERROR);
+		Util::writeLog('gallery', 'Exception: '.$msg, Util::ERROR);
 		parent::__construct($msg);
 	}
 }

lib/Http/ImageResponse.php

Spacing +3 added lines, -3 removed lines

@@ -38,10 +38,10 @@
 		$this->preview = $image['preview'];
 
 		$this->setStatus($statusCode);
-		$this->addHeader('Content-type', $image['mimetype'] . '; charset=utf-8');
+		$this->addHeader('Content-type', $image['mimetype'].'; charset=utf-8');
 		$this->addHeader('Content-Disposition',
-						 'attachment; filename*=UTF-8\'\'' . rawurlencode($name) . '; filename="'
-						 . rawurlencode($name) . '"'
+						 'attachment; filename*=UTF-8\'\''.rawurlencode($name).'; filename="'
+						 . rawurlencode($name).'"'
 		);
 	}
 

lib/AppInfo/Application.php

Spacing +30 added lines, -30 removed lines

@@ -14,11 +14,11 @@ 
 
 ini_set("gd.jpeg_ignore_warning", true);
 
-require_once __DIR__ . '/../../vendor/autoload.php';
+require_once __DIR__.'/../../vendor/autoload.php';
 
 // A production environment will not have xdebug enabled and
 // a development environment should have the dev packages installed
-$c3 = __DIR__ . '/../../c3.php';
+$c3 = __DIR__.'/../../c3.php';
 if (extension_loaded('xdebug') && file_exists($c3)) {
 	include_once $c3;
 }
@@ -73,7 +73,7 @@ 
 		 * Controllers
 		 */
 		$container->registerService(
-			'PageController', function (IContainer $c) {
+			'PageController', function(IContainer $c) {
 			return new PageController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -85,7 +85,7 @@ 
 		}
 		);
 		$container->registerService(
-			'ConfigController', function (IContainer $c) {
+			'ConfigController', function(IContainer $c) {
 			return new ConfigController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -95,7 +95,7 @@ 
 		}
 		);
 		$container->registerService(
-			'ConfigPublicController', function (IContainer $c) {
+			'ConfigPublicController', function(IContainer $c) {
 			return new ConfigPublicController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -105,7 +105,7 @@ 
 		}
 		);
 		$container->registerService(
-			'ConfigApiController', function (IContainer $c) {
+			'ConfigApiController', function(IContainer $c) {
 			return new ConfigApiController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -115,7 +115,7 @@ 
 		}
 		);
 		$container->registerService(
-			'FilesController', function (IContainer $c) {
+			'FilesController', function(IContainer $c) {
 			return new FilesController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -129,7 +129,7 @@ 
 		}
 		);
 		$container->registerService(
-			'FilesPublicController', function (IContainer $c) {
+			'FilesPublicController', function(IContainer $c) {
 			return new FilesPublicController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -143,7 +143,7 @@ 
 		}
 		);
 		$container->registerService(
-			'FilesApiController', function (IContainer $c) {
+			'FilesApiController', function(IContainer $c) {
 			return new FilesApiController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -157,7 +157,7 @@ 
 		}
 		);
 		$container->registerService(
-			'PreviewController', function (IContainer $c) {
+			'PreviewController', function(IContainer $c) {
 			return new PreviewController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -172,7 +172,7 @@ 
 		}
 		);
 		$container->registerService(
-			'PreviewPublicController', function (IContainer $c) {
+			'PreviewPublicController', function(IContainer $c) {
 			return new PreviewPublicController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -187,7 +187,7 @@ 
 		}
 		);
 		$container->registerService(
-			'PreviewApiController', function (IContainer $c) {
+			'PreviewApiController', function(IContainer $c) {
 			return new PreviewApiController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -206,30 +206,30 @@ 
 		 * Core
 		 */
 		$container->registerService(
-			'EventSource', function (IAppContainer $c) {
+			'EventSource', function(IAppContainer $c) {
 			return new EventSource();
 		}
 		);
 		$container->registerService(
-			'Token', function (IContainer $c) {
+			'Token', function(IContainer $c) {
 			return $c->query('Request')
 					 ->getParam('token');
 		}
 		);
 		$container->registerService(
-			'Session', function (IAppContainer $c) {
+			'Session', function(IAppContainer $c) {
 			return $c->getServer()
 					 ->getSession();
 		}
 		);
 		$container->registerService(
-			'L10N', function (IAppContainer $c) {
+			'L10N', function(IAppContainer $c) {
 			return $c->getServer()
 					 ->getL10N('gallery'); // Keep the same translations
 		}
 		);
 		$container->registerService(
-			'UserFolder', function (IAppContainer $c) {
+			'UserFolder', function(IAppContainer $c) {
 			return $c->getServer()
 					 ->getUserFolder($c->query('UserId'));
 		}
@@ -239,12 +239,12 @@ 
 		 * OCA
 		 */
 		$container->registerService(
-			'ConfigParser', function () {
+			'ConfigParser', function() {
 			return new ConfigParser();
 		}
 		);
 		$container->registerService(
-			'Environment', function (IContainer $c) {
+			'Environment', function(IContainer $c) {
 			return new Environment(
 				$c->query('AppName'),
 				$c->query('UserId'),
@@ -271,14 +271,14 @@ 
 		if (\OCP\App::isEnabled('ocutility')) {
 			// @codeCoverageIgnoreStart
 			$container->registerService(
-				'UtilityContainer', function () {
+				'UtilityContainer', function() {
 				$app = new OcUtility();
 
 				return $app->getContainer();
 			}
 			);
 			$container->registerService(
-				'Helper', function (IContainer $c) {
+				'Helper', function(IContainer $c) {
 				return $c->query('UtilityContainer')
 						 ->query('OCA\OcUtility\Service\Helper');
 			}
@@ -286,7 +286,7 @@ 
 		} else {
 			// @codeCoverageIgnoreEnd
 			$container->registerService(
-				'Logger', function (IContainer $c) {
+				'Logger', function(IContainer $c) {
 				return $c->query('OCP\ILogger');
 			}
 			);
@@ -295,7 +295,7 @@ 
 		 * Services
 		 */
 		$container->registerService(
-			'SearchFolderService', function (IContainer $c) {
+			'SearchFolderService', function(IContainer $c) {
 			return new SearchFolderService(
 				$c->query('AppName'),
 				$c->query('Environment'),
@@ -304,7 +304,7 @@ 
 		}
 		);
 		$container->registerService(
-			'ConfigService', function (IContainer $c) {
+			'ConfigService', function(IContainer $c) {
 			return new ConfigService(
 				$c->query('AppName'),
 				$c->query('Environment'),
@@ -315,7 +315,7 @@ 
 		}
 		);
 		$container->registerService(
-			'SearchMediaService', function (IContainer $c) {
+			'SearchMediaService', function(IContainer $c) {
 			return new SearchMediaService(
 				$c->query('AppName'),
 				$c->query('Environment'),
@@ -324,12 +324,12 @@ 
 		}
 		);
 		$container->registerService(
-			'ThumbnailService', function () {
+			'ThumbnailService', function() {
 			return new ThumbnailService();
 		}
 		);
 		$container->registerService(
-			'PreviewService', function (IContainer $c) {
+			'PreviewService', function(IContainer $c) {
 			return new PreviewService(
 				$c->query('AppName'),
 				$c->query('Environment'),
@@ -339,7 +339,7 @@ 
 		}
 		);
 		$container->registerService(
-			'DownloadService', function (IContainer $c) {
+			'DownloadService', function(IContainer $c) {
 			return new DownloadService(
 				$c->query('AppName'),
 				$c->query('Environment'),
@@ -353,7 +353,7 @@ 
 		 */
 		$container->registerService(
 			'SharingCheckMiddleware',
-			function (IAppContainer $c) {
+			function(IAppContainer $c) {
 				return new SharingCheckMiddleware(
 					$c->query('AppName'),
 					$c->query('Request'),
@@ -366,7 +366,7 @@ 
 		);
 		$container->registerService(
 			'EnvCheckMiddleware',
-			function (IContainer $c) {
+			function(IContainer $c) {
 				return new EnvCheckMiddleware(
 					$c->query('AppName'),
 					$c->query('Request'),