nextcloud / deck

lib/Service/PermissionService.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 	/**
 	 * Get current user permissions for a board
 	 *
-	 * @param Board|Entity $board
+	 * @param Board $board
 	 * @return array|bool
 	 * @internal param $boardId
 	 */

lib/Db/CardMapper.php

Doc Comments +4 added lines, -1 removed lines

@@ -92,7 +92,7 @@ 
 
 	/**
 	 * @param $id
-	 * @return RelationalEntity if not found
+	 * @return Entity if not found
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
 	 * @throws \OCP\AppFramework\Db\DoesNotExistException
 	 */
@@ -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/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/Service/StackService.php

Unused Use Statements -2 removed lines

@@ -30,8 +30,6 @@
 use OCA\Deck\Db\Stack;
 use OCA\Deck\Db\StackMapper;
 use OCA\Deck\StatusException;
-use OCP\ICache;
-use OCP\ICacheFactory;
 
 
 class StackService {

lib/Db/AttachmentMapper.php

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
 		$qb = $this->db->getQueryBuilder();
 		$qb->select('*')
 			->from('deck_attachment')
-			->where($qb->expr()->eq('id', (string)$id));
+			->where($qb->expr()->eq('id', (string) $id));
 
 		$cursor = $qb->execute();
 		$row = $cursor->fetch(PDO::FETCH_ASSOC);
@@ -71,13 +71,13 @@ 
 		$qb = $this->db->getQueryBuilder();
 		$qb->select('*')
 			->from('deck_attachment')
-			->where($qb->expr()->eq('card_id', (string)$cardId, IQueryBuilder::PARAM_INT))
-			->andWhere($qb->expr()->eq('deleted_at', (string)0, IQueryBuilder::PARAM_INT));
+			->where($qb->expr()->eq('card_id', (string) $cardId, IQueryBuilder::PARAM_INT))
+			->andWhere($qb->expr()->eq('deleted_at', (string) 0, IQueryBuilder::PARAM_INT));
 
 
 		$entities = [];
 		$cursor = $qb->execute();
-		while($row = $cursor->fetch()){
+		while ($row = $cursor->fetch()) {
 			$entities[] = $this->mapRowToEntity($row);
 		}
 		$cursor->closeCursor();
@@ -93,16 +93,16 @@ 
 			->where($qb->expr()->gt('deleted_at', '0', IQueryBuilder::PARAM_INT));
 		if ($withOffset) {
 			$qb
-				->andWhere($qb->expr()->lt('deleted_at', (string)$timeLimit, IQueryBuilder::PARAM_INT));
+				->andWhere($qb->expr()->lt('deleted_at', (string) $timeLimit, IQueryBuilder::PARAM_INT));
 		}
 		if ($cardId !== null) {
 			$qb
-				->andWhere($qb->expr()->eq('card_id', (string)$cardId, IQueryBuilder::PARAM_INT));
+				->andWhere($qb->expr()->eq('card_id', (string) $cardId, IQueryBuilder::PARAM_INT));
 		}
 
 		$entities = [];
 		$cursor = $qb->execute();
-		while($row = $cursor->fetch()){
+		while ($row = $cursor->fetch()) {
 			$entities[] = $this->mapRowToEntity($row);
 		}
 		$cursor->closeCursor();

lib/Service/FileService.php

Spacing +2 added lines, -2 removed lines

@@ -68,7 +68,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) {
@@ -94,7 +94,7 @@ 
 		return $attachment;
 	}
 
-	private function getUploadedFile () {
+	private function getUploadedFile() {
 		$file = $this->request->getUploadedFile('file');
 		$error = null;
 		$phpFileUploadErrors = [

Unused Use Statements -2 removed lines

@@ -24,10 +24,8 @@
 namespace OCA\Deck\Service;
 
 
-use OC\Security\CSP\ContentSecurityPolicyManager;
 use OCA\Deck\Db\Attachment;
 use OCP\AppFramework\Http\ContentSecurityPolicy;
-use OCP\AppFramework\Http\EmptyContentSecurityPolicy;
 use OCP\AppFramework\Http\FileDisplayResponse;
 use OCP\Files\IAppData;
 use OCP\Files\NotFoundException;