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/Service/FileService.php

Unused Use Statements -4 removed lines

@@ -23,15 +23,11 @@
 
 namespace OCA\Deck\Service;
 
-use OC\Security\CSP\ContentSecurityPolicyManager;
 use OCA\Deck\Db\Attachment;
 use OCA\Deck\StatusException;
 use OCP\AppFramework\Http\ContentSecurityPolicy;
-use OCP\AppFramework\Http\EmptyContentSecurityPolicy;
 use OCP\AppFramework\Http\FileDisplayResponse;
 use OCP\AppFramework\Http\StreamResponse;
-use OCP\Files\Cache\IScanner;
-use OCP\Files\Folder;
 use OCP\Files\IAppData;
 use OCP\Files\IRootFolder;
 use OCP\Files\NotFoundException;

lib/Service/PermissionService.php

Doc Comments +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
 
 	/**
 	 * @param $boardId
-	 * @return bool
+	 * @return boolean|null
 	 */
 	public function userIsBoardOwner($boardId) {
 		try {

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/Notification/NotificationHelper.php

Doc Comments +3 added lines

@@ -94,6 +94,9 @@
 		$this->cardMapper->markNotified($card);
 	}
 
+	/**
+	 * @param \OCA\Deck\Db\Card $card
+	 */
 	public function sendCardAssigned($card, $userId) {
 		$boardId = $this->cardMapper->findBoardId($card->getId());
 		$board = $this->getBoard($boardId);

lib/Controller/BoardApiController.php

Unused Use Statements -2 removed lines

@@ -25,10 +25,8 @@
 namespace OCA\Deck\Controller;
 
 use OCP\AppFramework\ApiController;
-use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\IRequest;
-
 use OCA\Deck\Service\BoardService;
 
 /**

lib/Controller/CardApiController.php

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
  namespace OCA\Deck\Controller;
 
  use OCP\AppFramework\ApiController;
- use OCP\AppFramework\Http;
  use OCP\AppFramework\Http\DataResponse;
  use OCP\IRequest;
  use OCA\Deck\Service\CardService;

lib/Db/BoardMapper.php

Doc Comments +3 added lines

@@ -173,6 +173,9 @@
 		return ($board->getOwner() === $userId);
 	}
 
+	/**
+	 * @param string $id
+	 */
 	public function findBoardId($id) {
 		return $id;
 	}