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/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/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/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

lib/Db/ChangeHelper.php

Unused Use Statements -4 removed lines

@@ -23,13 +23,9 @@
 
 namespace OCA\Deck\Db;
 
-use OCP\AppFramework\Db\DoesNotExistException;
-use OCP\ICache;
 use OCP\ICacheFactory;
 use OCP\IDBConnection;
 use OCP\IRequest;
-use OCP\IUserManager;
-use OCP\IGroupManager;
 
 class ChangeHelper {