@@ -158,7 +158,7 @@ discard block |
||
| 158 | 158 | } |
| 159 | 159 | |
| 160 | 160 | /** |
| 161 | - * @return array |
|
| 161 | + * @return integer |
|
| 162 | 162 | */ |
| 163 | 163 | private function getBoardPrerequisites() { |
| 164 | 164 | $groups = $this->groupManager->getUserGroupIds( |
@@ -532,6 +532,9 @@ discard block |
||
| 532 | 532 | return $this->aclMapper->delete($acl); |
| 533 | 533 | } |
| 534 | 534 | |
| 535 | + /** |
|
| 536 | + * @param Board $board |
|
| 537 | + */ |
|
| 535 | 538 | private function enrichWithStacks($board, $since = -1) { |
| 536 | 539 | $stacks = $this->stackMapper->findAll($board->getId(), null, null, $since); |
| 537 | 540 | |
@@ -96,7 +96,7 @@ discard block |
||
| 96 | 96 | public function findAll($since = 0) { |
| 97 | 97 | $userInfo = $this->getBoardPrerequisites(); |
| 98 | 98 | $userBoards = $this->boardMapper->findAllByUser($userInfo['user'], null, null, $since); |
| 99 | - $groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'],null, null, $since); |
|
| 99 | + $groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'], null, null, $since); |
|
| 100 | 100 | $complete = array_merge($userBoards, $groupBoards); |
| 101 | 101 | $result = []; |
| 102 | 102 | /** @var Board $item */ |
@@ -132,7 +132,7 @@ discard block |
||
| 132 | 132 | */ |
| 133 | 133 | public function find($boardId) { |
| 134 | 134 | |
| 135 | - if ( is_numeric($boardId) === false ) { |
|
| 135 | + if (is_numeric($boardId) === false) { |
|
| 136 | 136 | throw new BadRequestException('board id must be a number'); |
| 137 | 137 | } |
| 138 | 138 | |
@@ -181,7 +181,7 @@ discard block |
||
| 181 | 181 | */ |
| 182 | 182 | public function isArchived($mapper, $id) { |
| 183 | 183 | |
| 184 | - if (is_numeric($id) === false) { |
|
| 184 | + if (is_numeric($id) === false) { |
|
| 185 | 185 | throw new BadRequestException('id must be a number'); |
| 186 | 186 | } |
| 187 | 187 | |
@@ -215,7 +215,7 @@ discard block |
||
| 215 | 215 | throw new BadRequestException('mapper must be provided'); |
| 216 | 216 | } |
| 217 | 217 | |
| 218 | - if (is_numeric($id) === false) { |
|
| 218 | + if (is_numeric($id) === false) { |
|
| 219 | 219 | throw new BadRequestException('id must be a number'); |
| 220 | 220 | } |
| 221 | 221 | |
@@ -353,7 +353,7 @@ discard block |
||
| 353 | 353 | * @throws BadRequestException |
| 354 | 354 | */ |
| 355 | 355 | public function deleteForce($id) { |
| 356 | - if (is_numeric($id) === false) { |
|
| 356 | + if (is_numeric($id) === false) { |
|
| 357 | 357 | throw new BadRequestException('id must be a number'); |
| 358 | 358 | } |
| 359 | 359 | |
@@ -387,7 +387,7 @@ discard block |
||
| 387 | 387 | throw new BadRequestException('color must be provided'); |
| 388 | 388 | } |
| 389 | 389 | |
| 390 | - if ( is_bool($archived) === false ) { |
|
| 390 | + if (is_bool($archived) === false) { |
|
| 391 | 391 | throw new BadRequestException('archived must be a boolean'); |
| 392 | 392 | } |
| 393 | 393 | |
@@ -535,7 +535,7 @@ discard block |
||
| 535 | 535 | private function enrichWithStacks($board, $since = -1) { |
| 536 | 536 | $stacks = $this->stackMapper->findAll($board->getId(), null, null, $since); |
| 537 | 537 | |
| 538 | - if(\count($stacks) === 0) { |
|
| 538 | + if (\count($stacks) === 0) { |
|
| 539 | 539 | return; |
| 540 | 540 | } |
| 541 | 541 | |