nextcloud / deck

lib/Db/DeckMapper.php

Doc Comments +3 added lines

@@ -45,6 +45,9 @@
 		return $this->findEntity($sql, [$id]);
 	}
 
+	/**
+	 * @param string $sql
+	 */
 	protected function execute($sql, array $params = [], $limit = null, $offset = null) {
 		return parent::execute($sql, $params, $limit, $offset);
 	}

lib/Db/LabelMapper.php

Doc Comments +6 added lines

@@ -66,6 +66,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param integer $labelId
+	 */
 	public function deleteLabelAssignments($labelId) {
 		$sql = 'DELETE FROM `*PREFIX*deck_assigned_labels` WHERE label_id = ?';
 		$stmt = $this->db->prepare($sql);
@@ -73,6 +76,9 @@ 
 		$stmt->execute();
 	}
 
+	/**
+	 * @param integer $cardId
+	 */
 	public function deleteLabelAssignmentsForCard($cardId) {
 		$sql = 'DELETE FROM `*PREFIX*deck_assigned_labels` WHERE card_id = ?';
 		$stmt = $this->db->prepare($sql);

lib/Middleware/SharingMiddleware.php

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
 namespace OCA\Deck\Middleware;
 
 use OCA\Deck\StatusException;
-use OCA\Deck\BadRequestException;
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Middleware;
 use OCP\AppFramework\Http\JSONResponse;

lib/Db/AclMapper.php

Doc Comments +3 added lines

@@ -48,6 +48,9 @@
 		return $entity->getBoardId();
 	}
 
+	/**
+	 * @param string $participant
+	 */
 	public function findByParticipant($type, $participant) {
 		$sql = 'SELECT * from *PREFIX*deck_board_acl WHERE type = ? AND participant = ?';
 		return $this->findEntities($sql, [$type, $participant]);

lib/AppInfo/Application.php

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
 	}
 
 	protected function registerCommentsEventHandler() {
-		$this->getContainer()->getServer()->getCommentsManager()->registerEventHandler(function () {
+		$this->getContainer()->getServer()->getCommentsManager()->registerEventHandler(function() {
 			return $this->getContainer()->query(CommentEventHandler::class);
 		});
 	}

lib/Activity/ActivityManager.php

Spacing +6 added lines, -6 removed lines

@@ -133,7 +133,7 @@ 
 		$subject = '';
 		switch ($subjectIdentifier) {
 			case self::SUBJECT_BOARD_CREATE:
-				$subject = $ownActivity ? $this->l10n->t('You have created a new board {board}'): $this->l10n->t('{user} has created a new board {board}');
+				$subject = $ownActivity ? $this->l10n->t('You have created a new board {board}') : $this->l10n->t('{user} has created a new board {board}');
 				break;
 			case self::SUBJECT_BOARD_DELETE:
 				$subject = $ownActivity ? $this->l10n->t('You have deleted the board {board}') : $this->l10n->t('{user} has deleted the board {board}');
@@ -368,7 +368,7 @@ 
 				break;
 		}
 
-		if ($subject === self::SUBJECT_CARD_UPDATE_DESCRIPTION){
+		if ($subject === self::SUBJECT_CARD_UPDATE_DESCRIPTION) {
 			$subjectParams['diff'] = true;
 		}
 		if ($subject === self::SUBJECT_CARD_UPDATE_STACKID) {
@@ -379,7 +379,7 @@ 
 		$event->setApp('deck')
 			->setType('deck')
 			->setAuthor($this->userId)
-			->setObject($objectType, (int)$object->getId(), $object->getTitle())
+			->setObject($objectType, (int) $object->getId(), $object->getTitle())
 			->setSubject($subject, array_merge($subjectParams, $additionalParams))
 			->setTimestamp(time());
 
@@ -440,7 +440,7 @@ 
 					$objectId = $entity->getObjectId();
 					break;
 				default:
-					throw new InvalidArgumentException('No entity relation present for '. $className . ' to ' . $objectType);
+					throw new InvalidArgumentException('No entity relation present for ' . $className . ' to ' . $objectType);
 			}
 			return $this->cardMapper->find($objectId);
 		}
@@ -455,11 +455,11 @@ 
 					$objectId = $entity->getBoardId();
 					break;
 				default:
-					throw new InvalidArgumentException('No entity relation present for '. $className . ' to ' . $objectType);
+					throw new InvalidArgumentException('No entity relation present for ' . $className . ' to ' . $objectType);
 			}
 			return $this->boardMapper->find($objectId);
 		}
-		throw new InvalidArgumentException('No entity relation present for '. $className . ' to ' . $objectType);
+		throw new InvalidArgumentException('No entity relation present for ' . $className . ' to ' . $objectType);
 	}
 
 	private function findDetailsForStack($stackId) {

Doc Comments +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
 	}
 
 	/**
-	 * @param $subjectIdentifier
+	 * @param string $subjectIdentifier
 	 * @param array $subjectParams
 	 * @param bool $ownActivity
 	 * @return string

lib/Activity/CommentEventHandler.php

Spacing +3 added lines, -3 removed lines

@@ -43,12 +43,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);
@@ -60,7 +60,7 @@ 
 			CommentsEvent::EVENT_UPDATE,
 			CommentsEvent::EVENT_DELETE,
 		];
-		if(in_array($eventType, $applicableEvents)) {
+		if (in_array($eventType, $applicableEvents)) {
 			$this->notificationHandler($event);
 			return;
 		}

lib/Service/CardService.php

Spacing +2 added lines, -2 removed lines

@@ -91,8 +91,8 @@
 		$card->setLabels($this->labelMapper->findAssignedLabelsForCard($cardId));
 		$card->setAttachmentCount($this->attachmentService->count($cardId));
 		$user = $this->userManager->get($this->currentUser);
-		$lastRead = $this->commentsManager->getReadMark('deckCard', (string)$card->getId(), $user);
-		$count = $this->commentsManager->getNumberOfCommentsForObject('deckCard', (string)$card->getId(), $lastRead);
+		$lastRead = $this->commentsManager->getReadMark('deckCard', (string) $card->getId(), $user);
+		$count = $this->commentsManager->getNumberOfCommentsForObject('deckCard', (string) $card->getId(), $lastRead);
 		$card->setCommentsUnread($count);
 	}