@@ -99,16 +99,16 @@ |
||
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 | } |
@@ -85,7 +85,7 @@ discard block |
||
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 |
||
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 |
||
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!'); |
@@ -59,7 +59,7 @@ |
||
59 | 59 | public function index() { |
60 | 60 | $params = [ |
61 | 61 | 'user' => $this->userId, |
62 | - 'maxUploadSize' => (int)\OCP\Util::uploadLimit(), |
|
62 | + 'maxUploadSize' => (int) \OCP\Util::uploadLimit(), |
|
63 | 63 | ]; |
64 | 64 | |
65 | 65 | if ($this->defaultBoardService->checkFirstRun($this->userId, $this->appName)) { |
@@ -67,14 +67,14 @@ discard block |
||
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 |
||
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 |
||
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(); |
@@ -143,7 +143,7 @@ |
||
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 |
@@ -112,6 +112,6 @@ |
||
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 | ]; |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | */ |
122 | 122 | public function find($boardId) { |
123 | 123 | |
124 | - if ( is_numeric($boardId) === false ) { |
|
124 | + if (is_numeric($boardId) === false) { |
|
125 | 125 | throw new BadRequestException('board id must be a number'); |
126 | 126 | } |
127 | 127 | |
@@ -170,7 +170,7 @@ discard block |
||
170 | 170 | */ |
171 | 171 | public function isArchived($mapper, $id) { |
172 | 172 | |
173 | - if (is_numeric($id) === false) { |
|
173 | + if (is_numeric($id) === false) { |
|
174 | 174 | throw new BadRequestException('id must be a number'); |
175 | 175 | } |
176 | 176 | |
@@ -204,7 +204,7 @@ discard block |
||
204 | 204 | throw new BadRequestException('mapper must be provided'); |
205 | 205 | } |
206 | 206 | |
207 | - if (is_numeric($id) === false) { |
|
207 | + if (is_numeric($id) === false) { |
|
208 | 208 | throw new BadRequestException('id must be a number'); |
209 | 209 | } |
210 | 210 | |
@@ -335,7 +335,7 @@ discard block |
||
335 | 335 | * @throws BadRequestException |
336 | 336 | */ |
337 | 337 | public function deleteForce($id) { |
338 | - if (is_numeric($id) === false) { |
|
338 | + if (is_numeric($id) === false) { |
|
339 | 339 | throw new BadRequestException('id must be a number'); |
340 | 340 | } |
341 | 341 | |
@@ -369,7 +369,7 @@ discard block |
||
369 | 369 | throw new BadRequestException('color must be provided'); |
370 | 370 | } |
371 | 371 | |
372 | - if ( is_bool($archived) === false ) { |
|
372 | + if (is_bool($archived) === false) { |
|
373 | 373 | throw new BadRequestException('archived must be a boolean'); |
374 | 374 | } |
375 | 375 |
@@ -21,7 +21,7 @@ |
||
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 |
@@ -76,7 +76,7 @@ discard block |
||
76 | 76 | private function enrichStackWithCards($stack) { |
77 | 77 | $cards = $this->cardMapper->findAll($stack->getId()); |
78 | 78 | |
79 | - if(is_null($cards)) { |
|
79 | + if (is_null($cards)) { |
|
80 | 80 | return; |
81 | 81 | } |
82 | 82 | |
@@ -215,7 +215,7 @@ discard block |
||
215 | 215 | */ |
216 | 216 | public function delete($id) { |
217 | 217 | |
218 | - if ( is_numeric($id) === false ) { |
|
218 | + if (is_numeric($id) === false) { |
|
219 | 219 | throw new BadRequestException('stack id must be a number'); |
220 | 220 | } |
221 | 221 |