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