fix: Run cs:fix

Signed-off-by: Marcel Klehr <[email protected]>
nextcloud · Sep 7, 2024 · fa12da2 · fa12da2
1 parent 184a85f
commit fa12da2
Show file tree

Hide file tree

Showing 30 changed files with 79 additions and 81 deletions.
diff --git a/lib/Activity/ActivityPublisher.php b/lib/Activity/ActivityPublisher.php
@@ -106,7 +106,7 @@ public function publishShare(ChangeEvent $event): void {
 		$activity->setTimestamp(time());
 
 		/**
-		 * @var $sharedFolder SharedFolder
+		 * @var SharedFolder $sharedFolder
 		 */
 		try {
 			$sharedFolder = $this->sharedFolderMapper->find($event->getId());
@@ -141,7 +141,7 @@ public function publishFolder(ChangeEvent $event): void {
 		$activity->setTimestamp(time());
 
 		/**
-		 * @var $folder Folder
+		 * @var Folder $folder
 		 */
 		try {
 			$folder = $this->folderMapper->find($event->getId());
@@ -163,7 +163,7 @@ public function publishFolder(ChangeEvent $event): void {
 		}
 
 		/**
-		 * @var $shares SharedFolder[]
+		 * @var SharedFolder[] $shares
 		 */
 		$shares = $this->sharedFolderMapper->findByOwner($this->authorizer->getUserId());
 		$shares = array_merge($shares, $this->sharedFolderMapper->findByUser($this->authorizer->getUserId()));
@@ -193,7 +193,7 @@ public function publishBookmark(ChangeEvent $event): void {
 		$activity->setTimestamp(time());
 
 		/**
-		 * @var $bookmark Bookmark
+		 * @var Bookmark $bookmark
 		 */
 		try {
 			$bookmark = $this->bookmarkMapper->find($event->getId());
@@ -213,7 +213,7 @@ public function publishBookmark(ChangeEvent $event): void {
 		}
 
 		/**
-		 * @var $shares SharedFolder[]
+		 * @var SharedFolder[] $shares
 		 */
 		$shares = $this->sharedFolderMapper->findByOwner($this->authorizer->getUserId());
 		$shares = array_merge($shares, $this->sharedFolderMapper->findByUser($this->authorizer->getUserId()));

diff --git a/lib/BackgroundJobs/BackupJob.php b/lib/BackgroundJobs/BackupJob.php
@@ -45,7 +45,7 @@ public function __construct(
 	}
 
 	protected function run($argument) {
-		$userIds = $this->config->getUsersForUserValue('bookmarks', 'backup.enabled', (string) true); // if users have set this in older versions
+		$userIds = $this->config->getUsersForUserValue('bookmarks', 'backup.enabled', (string)true); // if users have set this in older versions
 		$userIds = array_merge($userIds, $this->config->getUsersForUserValue('bookmarks', 'backup.enabled', 'true'));
 		$userIds = array_unique($userIds);
 		if (empty($userIds)) {

diff --git a/lib/Command/ClearPreviews.php b/lib/Command/ClearPreviews.php
@@ -36,7 +36,7 @@ protected function configure() {
 	/**
 	 * Execute the command
 	 *
-	 * @param InputInterface  $input
+	 * @param InputInterface $input
 	 * @param OutputInterface $output
 	 *
 	 * @return int

diff --git a/lib/Controller/AdminController.php b/lib/Controller/AdminController.php
@@ -54,7 +54,7 @@ public function __construct(
 	 */
 	public function setSetting(string $setting, float|bool|int|string $value): JSONResponse {
 		try {
-			$this->settingsService->setSetting($setting, (string) $value);
+			$this->settingsService->setSetting($setting, (string)$value);
 			return new JSONResponse([], Http::STATUS_OK);
 		} catch (\Exception $e) {
 			return new JSONResponse([], Http::STATUS_BAD_REQUEST);

diff --git a/lib/Controller/BookmarkController.php b/lib/Controller/BookmarkController.php
@@ -193,11 +193,11 @@ private function _getRootFolderId(): ?int {
 		if ($this->authorizer->getToken() !== null) {
 			try {
 				/**
-				 * @var $publicFolder PublicFolder
+				 * @var PublicFolder $publicFolder
 				 */
 				$publicFolder = $this->publicFolderMapper->find($this->authorizer->getToken());
 				$this->rootFolderId = $publicFolder->getFolderId();
-			} catch (DoesNotExistException | MultipleObjectsReturnedException $e) {
+			} catch (DoesNotExistException|MultipleObjectsReturnedException $e) {
 				$this->logger->error($e->getMessage()."\n".$e->getMessage());
 			}
 		}
@@ -242,7 +242,7 @@ public function getSingleBookmark($id): JSONResponse {
 		}
 		try {
 			/**
-			 * @var $bm Bookmark
+			 * @var Bookmark $bm
 			 */
 			$bm = $this->bookmarkMapper->find($id);
 		} catch (DoesNotExistException $e) {
@@ -379,7 +379,7 @@ public function getBookmarks(
 				// If we have this user's permission to see the contents of their folder, simply set the userID
 				// to theirs
 				$userId = $folderEntity->getUserId();
-			} catch (DoesNotExistException | MultipleObjectsReturnedException $e) {
+			} catch (DoesNotExistException|MultipleObjectsReturnedException $e) {
 				return new DataResponse(['status' => 'error', 'data' => ['Not found']], Http::STATUS_BAD_REQUEST);
 			}
 			$params->setFolder($this->toInternalFolderId($folder));
@@ -395,7 +395,7 @@ public function getBookmarks(
 		} else {
 			try {
 				$result = $this->bookmarkMapper->findAllInPublicFolder($this->authorizer->getToken(), $params);
-			} catch (DoesNotExistException | MultipleObjectsReturnedException $e) {
+			} catch (DoesNotExistException|MultipleObjectsReturnedException $e) {
 				return new DataResponse(['status' => 'error', 'data' => ['Not found']], Http::STATUS_BAD_REQUEST);
 			}
 		}
@@ -527,7 +527,7 @@ public function deleteBookmark($id): JSONResponse {
 
 		try {
 			$this->bookmarkMapper->find($id);
-		} catch (DoesNotExistException | MultipleObjectsReturnedException) {
+		} catch (DoesNotExistException|MultipleObjectsReturnedException) {
 			return new JSONResponse(['status' => 'success']);
 		}
 
@@ -606,7 +606,7 @@ public function getBookmarkImage($id) {
 				return new NotFoundResponse();
 			}
 			return $this->doImageResponse($image);
-		} catch (DoesNotExistException | MultipleObjectsReturnedException | Exception $e) {
+		} catch (DoesNotExistException|MultipleObjectsReturnedException|Exception $e) {
 			return new NotFoundResponse();
 		}
 	}
@@ -633,7 +633,7 @@ public function getBookmarkFavicon($id) {
 				return new NotFoundResponse();
 			}
 			return $this->doImageResponse($image);
-		} catch (DoesNotExistException | MultipleObjectsReturnedException | Exception $e) {
+		} catch (DoesNotExistException|MultipleObjectsReturnedException|Exception $e) {
 			return new NotFoundResponse();
 		}
 	}

diff --git a/lib/Controller/FoldersController.php b/lib/Controller/FoldersController.php
@@ -85,11 +85,11 @@ private function _getRootFolderId(): ?int {
 		if ($this->authorizer->getToken() !== null) {
 			try {
 				/**
-				 * @var $publicFolder PublicFolder
+				 * @var PublicFolder $publicFolder
 				 */
 				$publicFolder = $this->publicFolderMapper->find($this->authorizer->getToken());
 				$this->rootFolderId = $publicFolder->getFolderId();
-			} catch (DoesNotExistException | MultipleObjectsReturnedException $e) {
+			} catch (DoesNotExistException|MultipleObjectsReturnedException $e) {
 				$this->logger->error($e->getMessage()."\n".$e->getMessage());
 			}
 		}
@@ -869,7 +869,7 @@ public function deleteShare(int $shareId): DataResponse {
 
 		try {
 			$this->folders->deleteShare($shareId);
-		} catch (UnsupportedOperation | DoesNotExistException | MultipleObjectsReturnedException | Exception $e) {
+		} catch (UnsupportedOperation|DoesNotExistException|MultipleObjectsReturnedException|Exception $e) {
 			return new Http\DataResponse(['status' => 'error', 'data' => ['Internal error']], Http::STATUS_INTERNAL_SERVER_ERROR);
 		}
 		return new Http\DataResponse(['status' => 'success']);

diff --git a/lib/Controller/InternalBookmarkController.php b/lib/Controller/InternalBookmarkController.php
@@ -66,8 +66,8 @@ public function __construct(
 	public function getBookmarks(
 		$page = 0,
 		$tags = [],
-		$conjunction = "or",
-		$sortby = "",
+		$conjunction = 'or',
+		$sortby = '',
 		$search = [],
 		$limit = 10,
 		$untagged = false,
@@ -103,7 +103,7 @@ public function getSingleBookmark($id): JSONResponse {
 	 *
 	 * @NoAdminRequired
 	 */
-	public function newBookmark($url = "", $title = null, $description = null, $tags = null, $folders = [], $target = null): JSONResponse {
+	public function newBookmark($url = '', $title = null, $description = null, $tags = null, $folders = [], $target = null): JSONResponse {
 		return $this->publicController->newBookmark($url, $title, $description, $tags, $folders, $target);
 	}
 
@@ -119,7 +119,7 @@ public function newBookmark($url = "", $title = null, $description = null, $tags
 	 *
 	 * @NoAdminRequired
 	 */
-	public function editBookmark($id = null, $url = null, $title = null, $description = "", $tags = [], $folders = null, $target = null): JSONResponse {
+	public function editBookmark($id = null, $url = null, $title = null, $description = '', $tags = [], $folders = null, $target = null): JSONResponse {
 		return $this->publicController->editBookmark($id, $url, $title, $description, $tags, $folders, $target);
 	}
 
@@ -141,7 +141,7 @@ public function deleteBookmark($id = -1): JSONResponse {
 	public function deleteAllBookmarks(): DataResponse {
 		try {
 			$this->bookmarks->deleteAll($this->userId);
-		} catch (UnsupportedOperation | DoesNotExistException | MultipleObjectsReturnedException $e) {
+		} catch (UnsupportedOperation|DoesNotExistException|MultipleObjectsReturnedException $e) {
 			return new DataResponse(['status' => 'error', 'data' => ['Internal server error']], Http::STATUS_INTERNAL_SERVER_ERROR);
 		}
 		return new DataResponse(['status' => 'success']);
@@ -153,7 +153,7 @@ public function deleteAllBookmarks(): DataResponse {
 	 * @return JSONResponse
 	 * @NoAdminRequired
 	 */
-	public function clickBookmark($url = ""): JSONResponse {
+	public function clickBookmark($url = ''): JSONResponse {
 		return $this->publicController->clickBookmark($url);
 	}
 

diff --git a/lib/Controller/InternalTagsController.php b/lib/Controller/InternalTagsController.php
@@ -24,7 +24,7 @@ public function __construct($appName, $request, TagsController $publicController
 	 *
 	 * @NoAdminRequired
 	 */
-	public function deleteTag($old_name = ""): JSONResponse {
+	public function deleteTag($old_name = ''): JSONResponse {
 		return $this->publicController->deleteTag($old_name);
 	}
 
@@ -36,7 +36,7 @@ public function deleteTag($old_name = ""): JSONResponse {
 	 *
 	 * @NoAdminRequired
 	 */
-	public function renameTag($old_name = "", $new_name = "", $name = ''): JSONResponse {
+	public function renameTag($old_name = '', $new_name = '', $name = ''): JSONResponse {
 		return $this->publicController->renameTag($old_name, $new_name, $name);
 	}
 

diff --git a/lib/Controller/TagsController.php b/lib/Controller/TagsController.php
@@ -34,8 +34,8 @@ public function __construct($appName, $request, $userId, DB\TagMapper $tagMapper
 	 * @NoCSRFRequired
 	 *
 	 */
-	public function deleteTag($old_name = ""): JSONResponse {
-		if ($old_name === "") {
+	public function deleteTag($old_name = ''): JSONResponse {
+		if ($old_name === '') {
 			return new JSONResponse([], Http::STATUS_BAD_REQUEST);
 		}
 
@@ -53,12 +53,12 @@ public function deleteTag($old_name = ""): JSONResponse {
 	 * @NoCSRFRequired
 	 *
 	 */
-	public function renameTag($old_name = "", $new_name = "", $name = ''): JSONResponse {
+	public function renameTag($old_name = '', $new_name = '', $name = ''): JSONResponse {
 		if ($new_name === '') {
 			$new_name = $name;
 		}
 
-		if ($old_name === "" || $new_name === "") {
+		if ($old_name === '' || $new_name === '') {
 			return new JSONResponse(['status' => 'error', 'data' => ['Must provide old_name and a new name']], Http::STATUS_BAD_REQUEST);
 		}
 
@@ -74,8 +74,8 @@ public function renameTag($old_name = "", $new_name = "", $name = ''): JSONRespo
 	 *
 	 */
 	public function fullTags($count = false): JSONResponse {
-		header("Cache-Control: no-cache, must-revalidate");
-		header("Expires: Sat, 26 Jul 1997 05:00:00 GMT");
+		header('Cache-Control: no-cache, must-revalidate');
+		header('Expires: Sat, 26 Jul 1997 05:00:00 GMT');
 
 		if ($count === true) {
 			$tags = $this->tagMapper->findAllWithCount($this->userId);

diff --git a/lib/Controller/WebViewController.php b/lib/Controller/WebViewController.php
@@ -114,11 +114,11 @@ public function link(string $token) {
 		$userName = 'Unknown';
 		try {
 			/**
-			 * @var $publicFolder PublicFolder
+			 * @var PublicFolder $publicFolder
 			 */
 			$publicFolder = $this->publicFolderMapper->find($token);
 			/**
-			 * @var $folder Folder
+			 * @var Folder $folder
 			 */
 			$folder = $this->folderMapper->find($publicFolder->getFolderId());
 			$title = $folder->getTitle();

diff --git a/lib/Db/Bookmark.php b/lib/Db/Bookmark.php
@@ -29,8 +29,6 @@
  * @method setAvailable(boolean $available)
  * @method int getArchivedFile()
  * @method setArchivedFile(int $fileId)
- * @method string getTextContent()
- * @method string getHtmlContent()
  * @method string getUserId()
  * @method setUserId(string $userId)
  */
@@ -133,7 +131,7 @@ public function setTextContent(?string $content): void {
 
 	public function getTextContent(): string {
 		// Remove non-utf-8 characters from string: https://stackoverflow.com/questions/1401317/remove-non-utf8-characters-from-string
-		return (string) mb_convert_encoding($this->textContent, 'UTF-8', 'UTF-8');
+		return (string)mb_convert_encoding($this->textContent, 'UTF-8', 'UTF-8');
 	}
 
 	public function setHtmlContent(?string $content): void {
@@ -146,10 +144,10 @@ public function setHtmlContent(?string $content): void {
 
 	public function getHtmlContent(): string {
 		// Remove non-utf-8 characters from string: https://stackoverflow.com/questions/1401317/remove-non-utf8-characters-from-string
-		return (string) mb_convert_encoding($this->htmlContent, 'UTF-8', 'UTF-8');
+		return (string)mb_convert_encoding($this->htmlContent, 'UTF-8', 'UTF-8');
 	}
 
 	public function isWebLink() {
-		return (bool) preg_match('/^https?:/i', $this->getUrl());
+		return (bool)preg_match('/^https?:/i', $this->getUrl());
 	}
 }
diff --git a/lib/Db/BookmarkMapper.php b/lib/Db/BookmarkMapper.php
@@ -453,7 +453,7 @@ private function _filterDuplicated(IQueryBuilder $qb, QueryParameters $params) {
 		if ($params->getDuplicated()) {
 			$subQuery = $this->db->getQueryBuilder();
 			$subQuery->select('trdup.parent_folder')
-			->from('*PREFIX*bookmarks_tree', 'trdup')
+				->from('*PREFIX*bookmarks_tree', 'trdup')
 				->where($subQuery->expr()->eq('b.id', 'trdup.id'))
 				->andWhere($subQuery->expr()->neq('trdup.parent_folder', 'tree.parent_folder'))
 				->andWhere($subQuery->expr()->eq('trdup.type', $qb->createPositionalParameter(TreeMapper::TYPE_BOOKMARK)));

diff --git a/lib/Db/BookmarkWithTagsAndParent.php b/lib/Db/BookmarkWithTagsAndParent.php
@@ -34,7 +34,7 @@ public function toArray(): array {
 					$array[$field] = [];
 				} else {
 					$array[$field] = array_values(array_unique(array_map(static function ($id) {
-						return (int) $id;
+						return (int)$id;
 					}, explode(',', $this->{$field}))));
 				}
 				continue;

diff --git a/lib/Db/TreeMapper.php b/lib/Db/TreeMapper.php
@@ -221,11 +221,11 @@ protected function getGetChildrenOrderQuery(): IQueryBuilder {
 	protected function getFindChildrenQuery(string $type): IQueryBuilder {
 		$qb = $this->selectFromType($type);
 		$qb
-				->join('i', 'bookmarks_tree', 't', $qb->expr()->eq('t.id', 'i.id'))
-				->where($qb->expr()->eq('t.parent_folder', $qb->createParameter('parent_folder')))
-				->andWhere($qb->expr()->eq('t.type', $qb->createNamedParameter($type)))
-				->andWhere($qb->expr()->isNull('t.soft_deleted_at'))
-				->orderBy('t.index', 'ASC');
+			->join('i', 'bookmarks_tree', 't', $qb->expr()->eq('t.id', 'i.id'))
+			->where($qb->expr()->eq('t.parent_folder', $qb->createParameter('parent_folder')))
+			->andWhere($qb->expr()->eq('t.type', $qb->createNamedParameter($type)))
+			->andWhere($qb->expr()->isNull('t.soft_deleted_at'))
+			->orderBy('t.index', 'ASC');
 		return $qb;
 	}
 
@@ -1137,7 +1137,7 @@ public function changeFolderOwner(Folder $folder, $userId): void {
 	 * @param int $folderId
 	 * @param Share $share
 	 * @return boolean
-	 *@throws MultipleObjectsReturnedException
+	 * @throws MultipleObjectsReturnedException
 	 */
 	public function isFolderSharedWithUser(int $folderId, string $userId): bool {
 		try {
@@ -1216,7 +1216,7 @@ public function deleteOldTrashbinItems(int $limit, float|int $maxAge): void {
 			$this->logger->error('Could not query for old trash bin items', ['exception' => $e]);
 			return;
 		}
-		while($row = $result->fetch()) {
+		while ($row = $result->fetch()) {
 			try {
 				$this->deleteEntry($row['type'], $row['id'], $row['parent_folder']);
 			} catch (DoesNotExistException $e) {

diff --git a/lib/ExportResponse.php b/lib/ExportResponse.php
@@ -33,10 +33,10 @@ public function __construct($returnstring) {
 		$dateTime = OC::$server->getDateTimeFormatter();
 
 		$export_name = '"' . $productName . ' Bookmarks (' . $userName . ') (' . $dateTime->formatDate(time()) . ').html"';
-		$this->addHeader("Cache-Control", "private");
-		$this->addHeader("Content-Type", " application/stream");
-		$this->addHeader("Content-Length", strlen($returnstring));
-		$this->addHeader("Content-Disposition", "attachment; filename=" . $export_name);
+		$this->addHeader('Cache-Control', 'private');
+		$this->addHeader('Content-Type', ' application/stream');
+		$this->addHeader('Content-Length', strlen($returnstring));
+		$this->addHeader('Content-Disposition', 'attachment; filename=' . $export_name);
 		$this->returnstring = $returnstring;
 	}
 

diff --git a/lib/Flow/Bookmark.php b/lib/Flow/Bookmark.php
@@ -104,7 +104,7 @@ public function prepareRuleMatcher(IRuleMatcher $ruleMatcher, string $eventName,
 		}
 		try {
 			$this->bookmark = $this->bookmarkMapper->find($event->getId());
-		} catch (DoesNotExistException | MultipleObjectsReturnedException $e) {
+		} catch (DoesNotExistException|MultipleObjectsReturnedException $e) {
 			return;
 		}
 		$ruleMatcher->setEntitySubject($this, $this->bookmark);

diff --git a/lib/Service/Authorizer.php b/lib/Service/Authorizer.php
@@ -197,7 +197,7 @@ protected function getMaskFromFlags($canWrite, $canShare): int {
 	 * @return boolean
 	 */
 	public static function hasPermission(int $perm, int $perms): bool {
-		return (boolean)($perms & $perm);
+		return (bool)($perms & $perm);
 	}
 
 	/**