nextcloud / deck

lib/Controller/BoardApiController.php

Unused Use Statements -3 removed lines

@@ -24,12 +24,9 @@
 
 namespace OCA\Deck\Controller;
 
-use OCA\Deck\Db\ChangeHelper;
 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 -2 removed lines

@@ -23,9 +23,7 @@
 
  namespace OCA\Deck\Controller;
 
- use OCA\Deck\Db\ChangeHelper;
  use OCP\AppFramework\ApiController;
- use OCP\AppFramework\Http;
  use OCP\AppFramework\Http\DataResponse;
  use OCP\IRequest;
  use OCA\Deck\Service\CardService;

Indentation +4 added lines, -4 removed lines

@@ -31,10 +31,10 @@
  use OCA\Deck\Service\CardService;
 
  /**
- * Class BoardApiController
- *
- * @package OCA\Deck\Controller
- */
+  * Class BoardApiController
+  *
+  * @package OCA\Deck\Controller
+  */
 class CardApiController extends ApiController {
 	private $cardService;
 	private $userId;

lib/Service/BoardService.php

Doc Comments +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
 	}
 
 	/**
-	 * @return array
+	 * @return integer
 	 */
 	private function getBoardPrerequisites() {
 		$groups = $this->groupManager->getUserGroupIds(

Spacing +6 added lines, -6 removed lines

@@ -87,7 +87,7 @@ 
 	public function findAll($since = 0) {
 		$userInfo = $this->getBoardPrerequisites();
 		$userBoards = $this->boardMapper->findAllByUser($userInfo['user'], null, null, $since);
-		$groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'],null, null,  $since);
+		$groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'], null, null, $since);
 		$complete = array_merge($userBoards, $groupBoards);
 		$result = [];
 		foreach ($complete as &$item) {
@@ -121,7 +121,7 @@ 
 	 */
 	public function find($boardId) {
 
-		if ( is_numeric($boardId) === false ) {
+		if (is_numeric($boardId) === false) {
 			throw new BadRequestException('board id must be a number');
 		}
 
@@ -170,7 +170,7 @@ 
 	 */
 	public function isArchived($mapper, $id) {
 
-		if (is_numeric($id) === false)  {
+		if (is_numeric($id) === false) {
 			throw new BadRequestException('id must be a number');
 		}
 
@@ -204,7 +204,7 @@ 
 			throw new BadRequestException('mapper must be provided');
 		}
 
-		if (is_numeric($id) === false)  {
+		if (is_numeric($id) === false) {
 			throw new BadRequestException('id must be a number');
 		}
 
@@ -332,7 +332,7 @@ 
 	 * @throws BadRequestException
 	 */
 	public function deleteForce($id) {
-		if (is_numeric($id) === false)  {
+		if (is_numeric($id) === false) {
 			throw new BadRequestException('id must be a number');
 		}
 
@@ -366,7 +366,7 @@ 
 			throw new BadRequestException('color must be provided');
 		}
 
-		if ( is_bool($archived) === false ) {
+		if (is_bool($archived) === false) {
 			throw new BadRequestException('archived must be a boolean');
 		}
 

lib/Service/StackService.php

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');
 		}