nextcloud / deck

lib/Db/AssignedUsersMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 	 * Check if user exists before assigning it to a card
 	 *
 	 * @param Entity $entity
-	 * @return null|Entity
+	 * @return AssignedUsers|null
 	 */
 	public function insert(Entity $entity) {
 		$user = $this->userManager->get($entity->getParticipant());

lib/Service/AttachmentService.php

Doc Comments +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
 	 * @param $cardId
 	 * @param $attachmentId
 	 * @param $request
-	 * @return mixed
+	 * @return \OCP\AppFramework\Db\Entity
 	 * @throws \OCA\Deck\NoPermissionException
 	 * @throws \OCP\AppFramework\Db\DoesNotExistException
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException

lib/Command/UserExport.php

Spacing +3 added lines, -3 removed lines

@@ -99,16 +99,16 @@
 		$data = [];
 		foreach ($boards as $board) {
 			$fullBoard = $this->boardMapper->find($board->getId(), true, true);
-			$data[$board->getId()] = (array)$fullBoard->jsonSerialize();
+			$data[$board->getId()] = (array) $fullBoard->jsonSerialize();
 			$stacks = $this->stackMapper->findAll($board->getId());
 			foreach ($stacks as $stack) {
-				$data[$board->getId()]['stacks'][] = (array)$stack->jsonSerialize();
+				$data[$board->getId()]['stacks'][] = (array) $stack->jsonSerialize();
 				$cards = $this->cardMapper->findAllByStack($stack->getId());
 				foreach ($cards as $card) {
 					$fullCard = $this->cardMapper->find($card->getId());
 					$assignedUsers = $this->assignedUsersMapper->find($card->getId());
 					$fullCard->setAssignedUsers($assignedUsers);
-					$data[$board->getId()]['stacks'][$stack->getId()]['cards'][] = (array)$fullCard->jsonSerialize();
+					$data[$board->getId()]['stacks'][$stack->getId()]['cards'][] = (array) $fullCard->jsonSerialize();
 				}
 			}
 		}

lib/Service/FileService.php

Spacing +3 added lines, -3 removed lines

@@ -85,7 +85,7 @@ 
 	 * @throws NotPermittedException
 	 */
 	private function getFolder(Attachment $attachment) {
-		$folderName = 'file-card-' . (int)$attachment->getCardId();
+		$folderName = 'file-card-' . (int) $attachment->getCardId();
 		try {
 			$folder = $this->appData->getFolder($folderName);
 		} catch (NotFoundException $e) {
@@ -116,7 +116,7 @@ 
 	 * @return array
 	 * @throws StatusException
 	 */
-	private function getUploadedFile () {
+	private function getUploadedFile() {
 		$file = $this->request->getUploadedFile('file');
 		$error = null;
 		$phpFileUploadErrors = [
@@ -190,7 +190,7 @@ 
 	 * @throws \Exception
 	 */
 	private function getFileFromRootFolder(Attachment $attachment) {
-		$folderName = 'file-card-' . (int)$attachment->getCardId();
+		$folderName = 'file-card-' . (int) $attachment->getCardId();
 		$instanceId = $this->config->getSystemValue('instanceid', null);
 		if ($instanceId === null) {
 			throw new \Exception('no instance id!');

lib/Db/AttachmentMapper.php

Spacing +4 added lines, -4 removed lines

@@ -67,14 +67,14 @@ 
 
 		$cursor = $qb->execute();
 		$row = $cursor->fetch(PDO::FETCH_ASSOC);
-		if($row === false) {
+		if ($row === false) {
 			$cursor->closeCursor();
 			throw new DoesNotExistException('Did expect one result but found none when executing' . $qb);
 		}
 
 		$row2 = $cursor->fetch();
 		$cursor->closeCursor();
-		if($row2 !== false ) {
+		if ($row2 !== false) {
 			throw new MultipleObjectsReturnedException('Did not expect more than one result when executing' . $query);
 		}
 
@@ -97,7 +97,7 @@ 
 
 		$entities = [];
 		$cursor = $qb->execute();
-		while($row = $cursor->fetch()){
+		while ($row = $cursor->fetch()) {
 			$entities[] = $this->mapRowToEntity($row);
 		}
 		$cursor->closeCursor();
@@ -127,7 +127,7 @@ 
 
 		$entities = [];
 		$cursor = $qb->execute();
-		while($row = $cursor->fetch()){
+		while ($row = $cursor->fetch()) {
 			$entities[] = $this->mapRowToEntity($row);
 		}
 		$cursor->closeCursor();

lib/Db/CardMapper.php

Doc Comments +3 added lines

@@ -136,6 +136,9 @@
 		return parent::delete($entity);
 	}
 
+	/**
+	 * @param integer $stackId
+	 */
 	public function deleteByStack($stackId) {
 		$cards = $this->findAllByStack($stackId);
 		foreach ($cards as $card) {

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/Activity/ActivityManager.php

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

Spacing +8 added lines, -8 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}');
@@ -302,10 +302,10 @@ 
 		try {
 			$object = $this->findObjectForEntity($objectType, $entity);
 		} catch (DoesNotExistException $e) {
-			\OC::$server->getLogger()->error('Could not create activity entry for ' . $subject . '. Entity not found.', (array)$entity);
+			\OC::$server->getLogger()->error('Could not create activity entry for ' . $subject . '. Entity not found.', (array) $entity);
 			return null;
 		} catch (MultipleObjectsReturnedException $e) {
-			\OC::$server->getLogger()->error('Could not create activity entry for ' . $subject . '. Entity not found.', (array)$entity);
+			\OC::$server->getLogger()->error('Could not create activity entry for ' . $subject . '. Entity not found.', (array) $entity);
 			return null;
 		}
 
@@ -375,7 +375,7 @@ 
 				break;
 		}
 
-		if ($subject === self::SUBJECT_CARD_UPDATE_DESCRIPTION){
+		if ($subject === self::SUBJECT_CARD_UPDATE_DESCRIPTION) {
 			$card = $subjectParams['card'];
 			if ($card->getLastEditor() === $this->userId) {
 				return null;
@@ -394,7 +394,7 @@ 
 		$event->setApp('deck')
 			->setType($eventType)
 			->setAuthor($author === null ? $this->userId : $author)
-			->setObject($objectType, (int)$object->getId(), $object->getTitle())
+			->setObject($objectType, (int) $object->getId(), $object->getTitle())
 			->setSubject($subject, array_merge($subjectParams, $additionalParams))
 			->setTimestamp(time());
 
@@ -455,7 +455,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);
 		}
@@ -470,11 +470,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) {