nextcloud / gallery

lib/Config/ConfigParser.php

Spacing +12 added lines, -12 removed lines

@@ -43,11 +43,11 @@ 
 	 * @return null|array
 	 */
 	public function getFeaturesList($folder, $configName) {
-		$featuresList = [];
+		$featuresList = [ ];
 		$parsedConfig = $this->parseConfig($folder, $configName);
 		$key = 'features';
 		if (array_key_exists($key, $parsedConfig)) {
-			$featuresList = $this->parseFeatures($parsedConfig[$key]);
+			$featuresList = $this->parseFeatures($parsedConfig[ $key ]);
 		}
 
 		return $featuresList;
@@ -72,7 +72,7 @@ 
 		list($config, $completionStatus) =
 			$this->buildAlbumConfig($currentConfig, $parsedConfig, $completionStatus, $level);
 
-		return [$config, $completionStatus];
+		return [ $config, $completionStatus ];
 	}
 
 	/**
@@ -152,17 +152,17 @@ 
 	private function buildAlbumConfig($currentConfig, $parsedConfig, $completionStatus, $level) {
 		foreach ($completionStatus as $key => $complete) {
 			if (!$this->isConfigItemComplete($key, $parsedConfig, $complete)) {
-				$parsedConfigItem = $parsedConfig[$key];
+				$parsedConfigItem = $parsedConfig[ $key ];
 				if ($this->isConfigUsable($key, $parsedConfigItem, $level)) {
 					list($configItem, $itemComplete) =
 						$this->addConfigItem($key, $parsedConfigItem, $level);
 					$currentConfig = array_merge($currentConfig, $configItem);
-					$completionStatus[$key] = $itemComplete;
+					$completionStatus[ $key ] = $itemComplete;
 				}
 			}
 		}
 
-		return [$currentConfig, $completionStatus];
+		return [ $currentConfig, $completionStatus ];
 	}
 
 	/**
@@ -177,7 +177,7 @@ 
 	private function isConfigItemComplete($key, $parsedConfig, $complete) {
 		return !(!$complete
 				 && array_key_exists($key, $parsedConfig)
-				 && !empty($parsedConfig[$key]));
+				 && !empty($parsedConfig[ $key ]));
 	}
 
 	/**
@@ -217,13 +217,13 @@ 
 	private function addConfigItem($key, $parsedConfigItem, $level) {
 		if ($key === 'sorting' && !array_key_exists('type', $parsedConfigItem)) {
 
-			return [[], false];
+			return [ [ ], false ];
 		} else {
-			$parsedConfigItem['level'] = $level;
-			$configItem = [$key => $parsedConfigItem];
+			$parsedConfigItem[ 'level' ] = $level;
+			$configItem = [ $key => $parsedConfigItem ];
 			$itemComplete = true;
 
-			return [$configItem, $itemComplete];
+			return [ $configItem, $itemComplete ];
 		}
 	}
 
@@ -237,7 +237,7 @@ 
 	private function isConfigInheritable($parsedConfigItem) {
 		$inherit = false;
 		if (array_key_exists('inherit', $parsedConfigItem)) {
-			$inherit = $parsedConfigItem['inherit'];
+			$inherit = $parsedConfigItem[ 'inherit' ];
 		}
 
 		if ($inherit === 'yes') {

lib/Middleware/CheckMiddleware.php

Spacing +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
 			$message = $exception->getMessage();
 			$code = $exception->getCode();
 
-			$this->logger->debug("[TokenCheckException] {exception}", ['exception' => $message]);
+			$this->logger->debug("[TokenCheckException] {exception}", [ 'exception' => $message ]);
 
 			return $this->computeResponse($message, $code);
 		}
@@ -133,7 +133,7 @@ 
 
 		$this->logger->debug(
 			'[CheckException] Unauthorised Request params: {params}',
-			['params' => $params]
+			[ 'params' => $params ]
 		);
 
 		return new TemplateResponse($this->appName, 'authenticate', $params, 'guest');
@@ -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/Controller/HttpError.php

Spacing +4 added lines, -4 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)) {
-			$logger->logException($exception, ['app' => 'gallery']);
+		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/Preview.php

Spacing +6 added lines, -6 removed lines

@@ -89,7 +89,7 @@ 
 			$preview = $this->prepareEmptyThumbnail($file, $status);
 		}
 
-		return [$preview, $status];
+		return [ $preview, $status ];
 	}
 
 	/**
@@ -145,7 +145,7 @@ 
 			$status = $this->getHttpStatusCode($exception);
 		}
 
-		return [$file, $status];
+		return [ $file, $status ];
 	}
 
 	/**
@@ -173,7 +173,7 @@ 
 			list($preview, $status) = $this->getErrorData();
 		}
 
-		return [$preview, $status];
+		return [ $preview, $status ];
 	}
 
 	/**
@@ -184,7 +184,7 @@ 
 	 * @return array<null|int>
 	 */
 	private function getErrorData($status = Http::STATUS_INTERNAL_SERVER_ERROR) {
-		return [null, $status];
+		return [ null, $status ];
 	}
 
 	/**
@@ -211,13 +211,13 @@ 
 	 * @return array<string,null|string>
 	 */
 	private function prepareEmptyThumbnail($file, $status) {
-		$thumbnail = [];
+		$thumbnail = [ ];
 		if ($status !== Http::STATUS_NOT_FOUND) {
 			$mimeType = '';
 			if ($file) {
 				$mimeType = $file->getMimeType();
 			}
-			$thumbnail = ['preview' => null, 'mimetype' => $mimeType];
+			$thumbnail = [ 'preview' => null, 'mimetype' => $mimeType ];
 		}
 
 		return $thumbnail;

lib/Controller/PreviewController.php

Spacing +5 added lines, -5 removed lines

@@ -102,8 +102,8 @@ 
 		foreach ($idsArray as $id) {
 			// Casting to integer here instead of using array_map to extract IDs from the URL
 			list($thumbnail, $status) = $this->getThumbnail((int)$id, $square, $scale);
-			$thumbnail['fileid'] = $id;
-			$thumbnail['status'] = $status;
+			$thumbnail[ 'fileid' ] = $id;
+			$thumbnail[ 'status' ] = $status;
 
 			$this->eventSource->send('preview', $thumbnail);
 		}
@@ -129,7 +129,7 @@ 
 		/** @type File $file */
 		list($file, $status) = $this->getFile($fileId);
 		if ($this->request->getHeader('If-None-Match') === $file->getEtag()) {
-			return new DataResponse([], Http::STATUS_NOT_MODIFIED);
+			return new DataResponse([ ], Http::STATUS_NOT_MODIFIED);
 		}
 		list($file, $preview, $status) = $this->getData($fileId, $width, $height);
 
@@ -141,14 +141,14 @@ 
 				], $status
 			);
 		}
-		$preview['name'] = $file->getName();
+		$preview[ 'name' ] = $file->getName();
 
 		$response = new ImageResponse($preview, $status);
 		$response->setETag($file->getEtag());
 		$lastModified = new \DateTime();
 		$lastModified->setTimestamp($file->getMTime());
 		$response->setLastModified($lastModified);
-		$response->cacheFor(3600*24);
+		$response->cacheFor(3600 * 24);
 		return $response;
 	}
 

lib/Controller/Files.php

Spacing +4 added lines, -4 removed lines

@@ -65,8 +65,8 @@ 
 	 * @return array <string,array<string,string|int>>|Http\JSONResponse
 	 */
 	private function getFilesAndAlbums($location, $features, $etag, $mediatypes) {
-		$files = [];
-		$albums = [];
+		$files = [ ];
+		$albums = [ ];
 		$updated = true;
 		/** @var Folder $folderNode */
 		list($folderPathFromRoot, $folderNode) =
@@ -100,7 +100,7 @@ 
 	private function fixPaths($files, $folderPathFromRoot) {
 		if (!empty($files)) {
 			foreach ($files as &$file) {
-				$file['path'] = $this->getReducedPath($file['path'], $folderPathFromRoot);
+				$file[ 'path' ] = $this->getReducedPath($file[ 'path' ], $folderPathFromRoot);
 			}
 		}
 
@@ -146,7 +146,7 @@ 
 		if (is_null($filename)) {
 			$filename = $file->getName();
 		}
-		$download['name'] = $filename;
+		$download[ 'name' ] = $filename;
 
 		return $download;
 	}

lib/Controller/FilesController.php

Spacing +3 added lines, -3 removed lines

@@ -117,7 +117,7 @@ 
 		} catch (ServiceException $exception) {
 			$code = $this->getHttpStatusCode($exception);
 			$url = $this->urlGenerator->linkToRoute(
-				$this->appName . '.page.error_page', ['code' => $code]
+				$this->appName.'.page.error_page', [ 'code' => $code ]
 			);
 
 			$response = new RedirectResponse($url);
@@ -127,8 +127,8 @@ 
 		}
 
 		// That's the only exception out of all the image media types we serve
-		if ($download['mimetype'] === 'image/svg+xml') {
-			$download['mimetype'] = 'text/plain';
+		if ($download[ 'mimetype' ] === 'image/svg+xml') {
+			$download[ 'mimetype' ] = 'text/plain';
 		}
 
 		return new ImageResponse($download);

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/Controller/PageController.php

Spacing +7 added lines, -7 removed lines

@@ -126,7 +126,7 @@ 
 			return $this->showPublicPage($token);
 		} else {
 			$url = $this->urlGenerator->linkToRoute(
-				$this->appName . '.files_public.download',
+				$this->appName.'.files_public.download',
 				[
 					'token'    => $token,
 					'fileId'   => $node->getId(),
@@ -187,7 +187,7 @@ 
 	 * @return TemplateResponse
 	 */
 	public function slideshow() {
-		return new TemplateResponse($this->appName, 'slideshow', [], 'blank');
+		return new TemplateResponse($this->appName, 'slideshow', [ ], 'blank');
 	}
 
 	/**
@@ -229,7 +229,7 @@ 
 	private function showPublicPage($token) {
 		$albumName = $this->environment->getSharedFolderName();
 		list($server2ServerSharing, $protected) = $this->getServer2ServerProperties();
-		$downloadUrl = $this->urlGenerator->linkToRouteAbsolute('files_sharing.sharecontroller.downloadShare', ['token' => $token]);
+		$downloadUrl = $this->urlGenerator->linkToRouteAbsolute('files_sharing.sharecontroller.downloadShare', [ 'token' => $token ]);
 
 		// Parameters sent to the template
 		$params = [
@@ -244,13 +244,13 @@ 
 
 		// Will render the page using the template found in templates/public.php
 		$response = new PublicTemplateResponse($this->appName, 'public', $params);
-		$response->setHeaderTitle($params['albumName']);
-		$response->setHeaderDetails($this->l10n->t('shared by %s', [$params['displayName']]));
+		$response->setHeaderTitle($params[ 'albumName' ]);
+		$response->setHeaderDetails($this->l10n->t('shared by %s', [ $params[ 'displayName' ] ]));
 		$response->setHeaderActions([
 			new SimpleMenuAction('download', $this->l10n->t('Download'), 'icon-download-white', $downloadUrl, 0),
 			new SimpleMenuAction('download', $this->l10n->t('Download'), 'icon-download', $downloadUrl, 10),
 			new LinkMenuAction($this->l10n->t('Direct link'), 'icon-public', $downloadUrl),
-			new ExternalShareMenuAction($this->l10n->t('Add to your Nextcloud'), 'icon-external', $this->environment->getUserId(), $params['displayName'], $params['albumName'])
+			new ExternalShareMenuAction($this->l10n->t('Add to your Nextcloud'), 'icon-external', $this->environment->getUserId(), $params[ 'displayName' ], $params[ 'albumName' ])
 		]);
 		$this->addContentSecurityToResponse($response);
 
@@ -270,6 +270,6 @@ 
 		$password = $this->environment->getSharePassword();
 		$passwordProtected = ($password) ? 'true' : 'false';
 
-		return [$server2ServerSharing, $passwordProtected];
+		return [ $server2ServerSharing, $passwordProtected ];
 	}
 }