nextcloud / deck

appinfo/routes.php

Spacing +1 added lines, -1 removed lines

@@ -112,6 +112,6 @@
 		['name' => 'attachment_api#delete', 'url' => '/api/v1.0/boards/{boardId}/stacks/{stackId}/cards/{cardId}/attachments/{attachmentId}', 'verb' => 'DELETE'],
 		['name' => 'attachment_api#restore', 'url' => '/api/v1.0/boards/{boardId}/stacks/{stackId}/cards/{cardId}/attachments/{attachmentId}/restore', 'verb' => 'PUT'],
 
-		['name' => 'board_api#preflighted_cors', 'url' => '/api/v1.0/{path}','verb' => 'OPTIONS', 'requirements' => ['path' => '.+']],
+		['name' => 'board_api#preflighted_cors', 'url' => '/api/v1.0/{path}', 'verb' => 'OPTIONS', 'requirements' => ['path' => '.+']],
 	]
 ];

lib/Service/DefaultBoardService.php

Indentation +7 added lines, -7 removed lines

@@ -40,7 +40,7 @@ 
 	private $config;
 	private $l10n;
 
-    public function __construct(
+	public function __construct(
 			IL10N $l10n,
 			BoardMapper $boardMapper,
 			BoardService $boardService,
@@ -55,7 +55,7 @@ 
 		$this->config = $config;
 		$this->boardMapper = $boardMapper;
 		$this->l10n = $l10n;
-    }
+	}
 
 	/**
 	 * @param $userId
@@ -73,7 +73,7 @@ 
 		}
 
 		return false;
-    }
+	}
 
 	/**
 	 * @param $title
@@ -100,9 +100,9 @@ 
 			throw new BadRequestException('color must be provided');
 		}
 
-        $defaultBoard = $this->boardService->create($title, $userId, $color);
-        $defaultStacks = [];
-        $defaultCards = [];
+		$defaultBoard = $this->boardService->create($title, $userId, $color);
+		$defaultStacks = [];
+		$defaultCards = [];
 
 		$boardId = $defaultBoard->getId();		
 
@@ -115,5 +115,5 @@ 
 		$defaultCards[] = $this->cardService->create($this->l10n->t('Example Task 1'), $defaultStacks[2]->getId(), 'text', 0, $userId);
 
 		return $defaultBoard;
-    }
+	}
 }
\ No newline at end of file

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

appinfo/app.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
  *
  */
 
-if ((@include_once __DIR__ . '/../vendor/autoload.php')===false) {
+if ((@include_once __DIR__ . '/../vendor/autoload.php') === false) {
 	throw new Exception('Cannot include autoload. Did you run install dependencies using composer?');
 }
 

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

Unused Use Statements -1 removed lines

@@ -34,7 +34,6 @@
 use OCA\Deck\Db\StackMapper;
 use OCA\Deck\StatusException;
 use OCA\Deck\BadRequestException;
-use OCP\Comments\ICommentsManager;
 
 
 class StackService {

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
 	private function enrichStackWithCards($stack, $since = -1) {
 		$cards = $this->cardMapper->findAll($stack->getId(), null, null, $since);
 
-		if(\count($cards) === 0) {
+		if (\count($cards) === 0) {
 			return;
 		}
 
@@ -216,7 +216,7 @@ 
 	 */
 	public function delete($id) {
 
-		if ( is_numeric($id) === false ) {
+		if (is_numeric($id) === false) {
 			throw new BadRequestException('stack id must be a number');
 		}