Completed
Pull Request — master (#939)
by Julius
02:38
created
lib/Service/BoardService.php 2 patches
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.