Passed
Pull Request — master (#885)
by Maxence
03:47 queued 01:01
created
lib/Command/UserExport.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -99,16 +99,16 @@
 block discarded – undo
99 99
 		$data = [];
100 100
 		foreach ($boards as $board) {
101 101
 			$fullBoard = $this->boardMapper->find($board->getId(), true, true);
102
-			$data[$board->getId()] = (array)$fullBoard->jsonSerialize();
102
+			$data[$board->getId()] = (array) $fullBoard->jsonSerialize();
103 103
 			$stacks = $this->stackMapper->findAll($board->getId());
104 104
 			foreach ($stacks as $stack) {
105
-				$data[$board->getId()]['stacks'][] = (array)$stack->jsonSerialize();
105
+				$data[$board->getId()]['stacks'][] = (array) $stack->jsonSerialize();
106 106
 				$cards = $this->cardMapper->findAllByStack($stack->getId());
107 107
 				foreach ($cards as $card) {
108 108
 					$fullCard = $this->cardMapper->find($card->getId());
109 109
 					$assignedUsers = $this->assignedUsersMapper->find($card->getId());
110 110
 					$fullCard->setAssignedUsers($assignedUsers);
111
-					$data[$board->getId()]['stacks'][$stack->getId()]['cards'][] = (array)$fullCard->jsonSerialize();
111
+					$data[$board->getId()]['stacks'][$stack->getId()]['cards'][] = (array) $fullCard->jsonSerialize();
112 112
 				}
113 113
 			}
114 114
 		}
Please login to merge, or discard this patch.
lib/Service/FileService.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 	 * @throws NotPermittedException
86 86
 	 */
87 87
 	private function getFolder(Attachment $attachment) {
88
-		$folderName = 'file-card-' . (int)$attachment->getCardId();
88
+		$folderName = 'file-card-' . (int) $attachment->getCardId();
89 89
 		try {
90 90
 			$folder = $this->appData->getFolder($folderName);
91 91
 		} catch (NotFoundException $e) {
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 	 * @return array
117 117
 	 * @throws StatusException
118 118
 	 */
119
-	private function getUploadedFile () {
119
+	private function getUploadedFile() {
120 120
 		$file = $this->request->getUploadedFile('file');
121 121
 		$error = null;
122 122
 		$phpFileUploadErrors = [
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
 	 * @throws \Exception
191 191
 	 */
192 192
 	private function getFileFromRootFolder(Attachment $attachment) {
193
-		$folderName = 'file-card-' . (int)$attachment->getCardId();
193
+		$folderName = 'file-card-' . (int) $attachment->getCardId();
194 194
 		$instanceId = $this->config->getSystemValue('instanceid', null);
195 195
 		if ($instanceId === null) {
196 196
 			throw new \Exception('no instance id!');
Please login to merge, or discard this patch.
lib/Db/AttachmentMapper.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -67,14 +67,14 @@  discard block
 block discarded – undo
67 67
 
68 68
 		$cursor = $qb->execute();
69 69
 		$row = $cursor->fetch(PDO::FETCH_ASSOC);
70
-		if($row === false) {
70
+		if ($row === false) {
71 71
 			$cursor->closeCursor();
72 72
 			throw new DoesNotExistException('Did expect one result but found none when executing' . $qb);
73 73
 		}
74 74
 
75 75
 		$row2 = $cursor->fetch();
76 76
 		$cursor->closeCursor();
77
-		if($row2 !== false ) {
77
+		if ($row2 !== false) {
78 78
 			throw new MultipleObjectsReturnedException('Did not expect more than one result when executing' . $query);
79 79
 		}
80 80
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 
98 98
 		$entities = [];
99 99
 		$cursor = $qb->execute();
100
-		while($row = $cursor->fetch()){
100
+		while ($row = $cursor->fetch()) {
101 101
 			$entities[] = $this->mapRowToEntity($row);
102 102
 		}
103 103
 		$cursor->closeCursor();
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 
128 128
 		$entities = [];
129 129
 		$cursor = $qb->execute();
130
-		while($row = $cursor->fetch()){
130
+		while ($row = $cursor->fetch()) {
131 131
 			$entities[] = $this->mapRowToEntity($row);
132 132
 		}
133 133
 		$cursor->closeCursor();
Please login to merge, or discard this patch.
lib/Controller/CardApiController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -143,7 +143,7 @@
 block discarded – undo
143 143
 	}
144 144
 
145 145
 	public function assignUser($userId) {
146
-		$card = $this->cardService->assignUser($this->request->getParam('cardId'), $userId);;
146
+		$card = $this->cardService->assignUser($this->request->getParam('cardId'), $userId); ;
147 147
 		return new DataResponse($card, HTTP::STATUS_OK);
148 148
 	}
149 149
 
Please login to merge, or discard this patch.
appinfo/routes.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -112,6 +112,6 @@
 block discarded – undo
112 112
 		['name' => 'attachment_api#delete', 'url' => '/api/v1.0/boards/{boardId}/stacks/{stackId}/cards/{cardId}/attachments/{attachmentId}', 'verb' => 'DELETE'],
113 113
 		['name' => 'attachment_api#restore', 'url' => '/api/v1.0/boards/{boardId}/stacks/{stackId}/cards/{cardId}/attachments/{attachmentId}/restore', 'verb' => 'PUT'],
114 114
 
115
-		['name' => 'board_api#preflighted_cors', 'url' => '/api/v1.0/{path}','verb' => 'OPTIONS', 'requirements' => ['path' => '.+']],
115
+		['name' => 'board_api#preflighted_cors', 'url' => '/api/v1.0/{path}', 'verb' => 'OPTIONS', 'requirements' => ['path' => '.+']],
116 116
 	]
117 117
 ];
Please login to merge, or discard this patch.
appinfo/app.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
  *
22 22
  */
23 23
 
24
-if ((@include_once __DIR__ . '/../vendor/autoload.php')===false) {
24
+if ((@include_once __DIR__ . '/../vendor/autoload.php') === false) {
25 25
 	throw new Exception('Cannot include autoload. Did you run install dependencies using composer?');
26 26
 }
27 27
 
Please login to merge, or discard this patch.
lib/AppInfo/Application.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -142,7 +142,7 @@
 block discarded – undo
142 142
 	}
143 143
 
144 144
 	protected function registerCommentsEventHandler() {
145
-		$this->getContainer()->getServer()->getCommentsManager()->registerEventHandler(function () {
145
+		$this->getContainer()->getServer()->getCommentsManager()->registerEventHandler(function() {
146 146
 			return $this->getContainer()->query(CommentEventHandler::class);
147 147
 		});
148 148
 	}
Please login to merge, or discard this patch.
lib/Service/CardService.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -91,8 +91,8 @@
 block discarded – undo
91 91
 		$card->setLabels($this->labelMapper->findAssignedLabelsForCard($cardId));
92 92
 		$card->setAttachmentCount($this->attachmentService->count($cardId));
93 93
 		$user = $this->userManager->get($this->currentUser);
94
-		$lastRead = $this->commentsManager->getReadMark('deckCard', (string)$card->getId(), $user);
95
-		$count = $this->commentsManager->getNumberOfCommentsForObject('deckCard', (string)$card->getId(), $lastRead);
94
+		$lastRead = $this->commentsManager->getReadMark('deckCard', (string) $card->getId(), $user);
95
+		$count = $this->commentsManager->getNumberOfCommentsForObject('deckCard', (string) $card->getId(), $lastRead);
96 96
 		$card->setCommentsUnread($count);
97 97
 	}
98 98
 
Please login to merge, or discard this patch.
lib/Service/BoardService.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 	public function findAll($since = -1) {
93 93
 		$userInfo = $this->getBoardPrerequisites();
94 94
 		$userBoards = $this->boardMapper->findAllByUser($userInfo['user'], null, null, $since);
95
-		$groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'],null, null,  $since);
95
+		$groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'], null, null, $since);
96 96
 		$complete = array_merge($userBoards, $groupBoards);
97 97
 		$result = [];
98 98
 		/** @var Board $item */
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 	 */
128 128
 	public function find($boardId) {
129 129
 
130
-		if ( is_numeric($boardId) === false ) {
130
+		if (is_numeric($boardId) === false) {
131 131
 			throw new BadRequestException('board id must be a number');
132 132
 		}
133 133
 
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
 	 */
177 177
 	public function isArchived($mapper, $id) {
178 178
 
179
-		if (is_numeric($id) === false)  {
179
+		if (is_numeric($id) === false) {
180 180
 			throw new BadRequestException('id must be a number');
181 181
 		}
182 182
 
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
 			throw new BadRequestException('mapper must be provided');
211 211
 		}
212 212
 
213
-		if (is_numeric($id) === false)  {
213
+		if (is_numeric($id) === false) {
214 214
 			throw new BadRequestException('id must be a number');
215 215
 		}
216 216
 
@@ -345,7 +345,7 @@  discard block
 block discarded – undo
345 345
 	 * @throws BadRequestException
346 346
 	 */
347 347
 	public function deleteForce($id) {
348
-		if (is_numeric($id) === false)  {
348
+		if (is_numeric($id) === false) {
349 349
 			throw new BadRequestException('id must be a number');
350 350
 		}
351 351
 
@@ -379,7 +379,7 @@  discard block
 block discarded – undo
379 379
 			throw new BadRequestException('color must be provided');
380 380
 		}
381 381
 
382
-		if ( is_bool($archived) === false ) {
382
+		if (is_bool($archived) === false) {
383 383
 			throw new BadRequestException('archived must be a boolean');
384 384
 		}
385 385
 
Please login to merge, or discard this patch.