nextcloud / deck

lib/Activity/DeckProvider.php

Doc Comments +14 added lines

@@ -189,6 +189,9 @@ 
 		return $event;
 	}
 
+	/**
+	 * @param string $paramName
+	 */
 	private function parseParamForBoard($paramName, $subjectParams, $params) {
 		if (array_key_exists($paramName, $subjectParams)) {
 			$params[$paramName] = [
@@ -200,6 +203,10 @@ 
 		}
 		return $params;
 	}
+
+	/**
+	 * @param string $paramName
+	 */
 	private function parseParamForStack($paramName, $subjectParams, $params) {
 		if (array_key_exists($paramName, $subjectParams)) {
 			$params[$paramName] = [
@@ -211,6 +218,9 @@ 
 		return $params;
 	}
 
+	/**
+	 * @param string $paramName
+	 */
 	private function parseParamForAttachment($paramName, $subjectParams, $params) {
 		if (array_key_exists($paramName, $subjectParams)) {
 			$params[$paramName] = [
@@ -285,6 +295,7 @@ 
 	 *
 	 * @param $subjectParams
 	 * @param $params
+	 * @param IEvent $event
 	 * @return mixed
 	 */
 	private function parseParamForChanges($subjectParams, $params, $event) {
@@ -312,6 +323,9 @@ 
 		return $params;
 	}
 
+	/**
+	 * @param string $endpoint
+	 */
 	public function deckUrl($endpoint) {
 		return $this->urlGenerator->linkToRouteAbsolute('deck.page.index') . '#!' . $endpoint;
 	}

Spacing +2 added lines, -2 removed lines

@@ -281,9 +281,9 @@
 		if (array_key_exists('comment', $subjectParams)) {
 			/** @var IComment $comment */
 			try {
-				$comment = $this->commentsManager->get((int)$subjectParams['comment']);
+				$comment = $this->commentsManager->get((int) $subjectParams['comment']);
 				$event->setParsedMessage($comment->getMessage());
-				$params['comment'] =[
+				$params['comment'] = [
 					'type' => 'highlight',
 					'id' => $subjectParams['comment'],
 					'name' => $comment->getMessage()

lib/Cron/CardDescriptionActivity.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	private $cardMapper;
 
 	public function __construct(ActivityManager $activityManager, CardMapper $cardMapper) {
-		$this->activityManager  = $activityManager;
+		$this->activityManager = $activityManager;
 		$this->cardMapper = $cardMapper;
 	}
 

lib/Activity/CommentEventHandler.php

Spacing +3 added lines, -3 removed lines

@@ -50,12 +50,12 @@ 
 	 * @param CommentsEvent $event
 	 */
 	public function handle(CommentsEvent $event) {
-		if($event->getComment()->getObjectType() !== 'deckCard') {
+		if ($event->getComment()->getObjectType() !== 'deckCard') {
 			return;
 		}
 
 		$eventType = $event->getEvent();
-		if( $eventType === CommentsEvent::EVENT_ADD
+		if ($eventType === CommentsEvent::EVENT_ADD
 		) {
 			$this->notificationHandler($event);
 			$this->activityHandler($event);
@@ -65,7 +65,7 @@ 
 		$applicableEvents = [
 			CommentsEvent::EVENT_UPDATE
 		];
-		if(in_array($eventType, $applicableEvents)) {
+		if (in_array($eventType, $applicableEvents)) {
 			$this->notificationHandler($event);
 			return;
 		}

lib/Controller/BoardApiController.php

Unused Use Statements -2 removed lines

@@ -26,10 +26,8 @@
 
 use OCA\Deck\StatusException;
 use OCP\AppFramework\ApiController;
-use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\IRequest;
-
 use OCA\Deck\Service\BoardService;
 use Sabre\HTTP\Util;
 

lib/Notification/NotificationHelper.php

Doc Comments +6 added lines

@@ -95,6 +95,9 @@ 
 		$this->cardMapper->markNotified($card);
 	}
 
+	/**
+	 * @param \OCA\Deck\Db\Card $card
+	 */
 	public function markDuedateAsRead($card) {
 		$notification = $this->notificationManager->createNotification();
 		$notification
@@ -104,6 +107,9 @@ 
 		$this->notificationManager->markProcessed($notification);
 	}
 
+	/**
+	 * @param \OCA\Deck\Db\Card $card
+	 */
 	public function sendCardAssigned($card, $userId) {
 		$boardId = $this->cardMapper->findBoardId($card->getId());
 		$board = $this->getBoard($boardId);

lib/Provider/DeckProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	 * DeckProvider constructor.
 	 *
 	 * @param IL10N $l10n
-	 * @param IUrlGenerator $urlGenerator
+	 * @param IURLGenerator $urlGenerator
 	 * @param FullTextSearchService $fullTextSearchService
 	 */
 	public function __construct(

Spacing +1 added lines, -1 removed lines

@@ -240,7 +240,7 @@
 		foreach ($searchResult->getDocuments() as $document) {
 			try {
 				$board =
-					$this->fullTextSearchService->getBoardFromCardId((int)$document->getId());
+					$this->fullTextSearchService->getBoardFromCardId((int) $document->getId());
 				$path = '#!/board/' . $board->getId() . '//card/' . $document->getId();
 				$document->setLink($this->urlGenerator->linkToRoute('deck.page.index') . $path);
 			} catch (DoesNotExistException $e) {

Unused Use Statements -1 removed lines

@@ -44,7 +44,6 @@
 use OCP\FullTextSearch\Model\ISearchResult;
 use OCP\FullTextSearch\Model\ISearchTemplate;
 use OCP\IL10N;
-use OCP\IURLGenerator;
 
 
 /**

lib/Service/FullTextSearchService.php

Spacing +6 added lines, -6 removed lines

@@ -96,7 +96,7 @@ 
 		$userId = $e->getArgument('userId');
 
 		$this->fullTextSearchManager->createIndex(
-			DeckProvider::DECK_PROVIDER_ID, (string)$cardId, $userId, IIndex::INDEX_FULL
+			DeckProvider::DECK_PROVIDER_ID, (string) $cardId, $userId, IIndex::INDEX_FULL
 		);
 	}
 
@@ -108,7 +108,7 @@ 
 		$cardId = $e->getArgument('id');
 
 		$this->fullTextSearchManager->updateIndexStatus(
-			DeckProvider::DECK_PROVIDER_ID, (string)$cardId, IIndex::INDEX_CONTENT
+			DeckProvider::DECK_PROVIDER_ID, (string) $cardId, IIndex::INDEX_CONTENT
 		);
 	}
 
@@ -120,7 +120,7 @@ 
 		$cardId = $e->getArgument('id');
 
 		$this->fullTextSearchManager->updateIndexStatus(
-			DeckProvider::DECK_PROVIDER_ID, (string)$cardId, IIndex::INDEX_REMOVE
+			DeckProvider::DECK_PROVIDER_ID, (string) $cardId, IIndex::INDEX_REMOVE
 		);
 	}
 
@@ -149,7 +149,7 @@ 
 	 * @return IIndexDocument
 	 */
 	public function generateIndexDocumentFromCard(Card $card): IIndexDocument {
-		$document = new IndexDocument(DeckProvider::DECK_PROVIDER_ID, (string)$card->getId());
+		$document = new IndexDocument(DeckProvider::DECK_PROVIDER_ID, (string) $card->getId());
 
 		return $document;
 	}
@@ -163,7 +163,7 @@ 
 	 */
 	public function fillIndexDocument(IIndexDocument $document) {
 		/** @var Card $card */
-		$card = $this->cardMapper->find((int)$document->getId());
+		$card = $this->cardMapper->find((int) $document->getId());
 
 		$document->setTitle($card->getTitle());
 		$document->setContent($card->getDescription());
@@ -230,7 +230,7 @@ 
 	 * @throws MultipleObjectsReturnedException
 	 */
 	public function getBoardFromCardId(int $cardId): Board {
-		$boardId = (int)$this->cardMapper->findBoardId($cardId);
+		$boardId = (int) $this->cardMapper->findBoardId($cardId);
 		/** @var Board $board */
 		$board = $this->boardMapper->find($boardId);
 

lib/Service/LabelService.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
 
 		$boardLabels = $this->labelMapper->findAll($boardId);
 		if (\is_array($boardLabels)) {
-			foreach($boardLabels as $boardLabel) {
+			foreach ($boardLabels as $boardLabel) {
 				if ($boardLabel->getTitle() === $title) {
 					throw new BadRequestException('title must be unique');
 					break;
@@ -168,7 +168,7 @@ 
 
 		$boardLabels = $this->labelMapper->findAll($label->getBoardId());
 		if (\is_array($boardLabels)) {
-			foreach($boardLabels as $boardLabel) {
+			foreach ($boardLabels as $boardLabel) {
 				if ($boardLabel->getId() === $label->getId()) {
 					continue;
 				}

lib/Collaboration/Resources/ResourceProvider.php

Doc Comments -1 removed lines

@@ -55,7 +55,6 @@
 	/**
 	 * Get the type of a resource
 	 *
-	 * @param IResource $resource
 	 * @return string
 	 * @since 15.0.0
 	 */

Unused Use Statements -2 removed lines

@@ -25,7 +25,6 @@ 
 
 
 use OCA\Deck\Db\Acl;
-use OCA\Deck\Db\Board;
 use OCA\Deck\Db\BoardMapper;
 use OCA\Deck\Service\PermissionService;
 use OCP\AppFramework\Db\DoesNotExistException;
@@ -34,7 +33,6 @@ 
 use OCP\Collaboration\Resources\IManager;
 use OCP\Collaboration\Resources\IProvider;
 use OCP\Collaboration\Resources\IResource;
-use OCP\Collaboration\Resources\ResourceException;
 use OCP\IUser;
 
 class ResourceProvider implements IProvider {

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
 		} catch (QueryException $e) {
 		}
 		if ($boardId !== null) {
-			$resource = $resourceManager->getResourceForUser(self::RESOURCE_TYPE, (string)$boardId, null);
+			$resource = $resourceManager->getResourceForUser(self::RESOURCE_TYPE, (string) $boardId, null);
 			$resourceManager->invalidateAccessCacheForResource($resource);
 		} else {
 			$resourceManager->invalidateAccessCacheForProvider($this);