@@ -50,12 +50,12 @@ discard block |
||
50 | 50 | * @param CommentsEvent $event |
51 | 51 | */ |
52 | 52 | public function handle(CommentsEvent $event) { |
53 | - if($event->getComment()->getObjectType() !== 'deckCard') { |
|
53 | + if ($event->getComment()->getObjectType() !== 'deckCard') { |
|
54 | 54 | return; |
55 | 55 | } |
56 | 56 | |
57 | 57 | $eventType = $event->getEvent(); |
58 | - if( $eventType === CommentsEvent::EVENT_ADD |
|
58 | + if ($eventType === CommentsEvent::EVENT_ADD |
|
59 | 59 | ) { |
60 | 60 | $this->notificationHandler($event); |
61 | 61 | $this->activityHandler($event); |
@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | $applicableEvents = [ |
66 | 66 | CommentsEvent::EVENT_UPDATE |
67 | 67 | ]; |
68 | - if(in_array($eventType, $applicableEvents)) { |
|
68 | + if (in_array($eventType, $applicableEvents)) { |
|
69 | 69 | $this->notificationHandler($event); |
70 | 70 | return; |
71 | 71 | } |
@@ -240,7 +240,7 @@ |
||
240 | 240 | foreach ($searchResult->getDocuments() as $document) { |
241 | 241 | try { |
242 | 242 | $board = |
243 | - $this->fullTextSearchService->getBoardFromCardId((int)$document->getId()); |
|
243 | + $this->fullTextSearchService->getBoardFromCardId((int) $document->getId()); |
|
244 | 244 | $path = '#!/board/' . $board->getId() . '//card/' . $document->getId(); |
245 | 245 | $document->setLink($this->urlGenerator->linkToRoute('deck.page.index') . $path); |
246 | 246 | } catch (DoesNotExistException $e) { |
@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | |
95 | 95 | $boardLabels = $this->labelMapper->findAll($boardId); |
96 | 96 | if (\is_array($boardLabels)) { |
97 | - foreach($boardLabels as $boardLabel) { |
|
97 | + foreach ($boardLabels as $boardLabel) { |
|
98 | 98 | if ($boardLabel->getTitle() === $title) { |
99 | 99 | throw new BadRequestException('title must be unique'); |
100 | 100 | break; |
@@ -168,7 +168,7 @@ discard block |
||
168 | 168 | |
169 | 169 | $boardLabels = $this->labelMapper->findAll($label->getBoardId()); |
170 | 170 | if (\is_array($boardLabels)) { |
171 | - foreach($boardLabels as $boardLabel) { |
|
171 | + foreach ($boardLabels as $boardLabel) { |
|
172 | 172 | if ($boardLabel->getId() === $label->getId()) { |
173 | 173 | continue; |
174 | 174 | } |
@@ -147,7 +147,7 @@ |
||
147 | 147 | * Assign a user to a card |
148 | 148 | */ |
149 | 149 | public function assignUser($userId) { |
150 | - $card = $this->cardService->assignUser($this->request->getParam('cardId'), $userId);; |
|
150 | + $card = $this->cardService->assignUser($this->request->getParam('cardId'), $userId); ; |
|
151 | 151 | return new DataResponse($card, HTTP::STATUS_OK); |
152 | 152 | } |
153 | 153 |
@@ -124,7 +124,7 @@ |
||
124 | 124 | } catch (QueryException $e) { |
125 | 125 | } |
126 | 126 | if ($boardId !== null) { |
127 | - $resource = $resourceManager->getResourceForUser(self::RESOURCE_TYPE, (string)$boardId, null); |
|
127 | + $resource = $resourceManager->getResourceForUser(self::RESOURCE_TYPE, (string) $boardId, null); |
|
128 | 128 | $resourceManager->invalidateAccessCacheForResource($resource); |
129 | 129 | } else { |
130 | 130 | $resourceManager->invalidateAccessCacheForProvider($this); |
@@ -155,7 +155,7 @@ discard block |
||
155 | 155 | * @throws \OCP\AppFramework\QueryException |
156 | 156 | */ |
157 | 157 | protected function registerCommentsEventHandler() { |
158 | - $this->getContainer()->getServer()->getCommentsManager()->registerEventHandler(function () { |
|
158 | + $this->getContainer()->getServer()->getCommentsManager()->registerEventHandler(function() { |
|
159 | 159 | return $this->getContainer()->query(CommentEventHandler::class); |
160 | 160 | }); |
161 | 161 | } |
@@ -175,7 +175,7 @@ discard block |
||
175 | 175 | /** @var IManager $resourceManager */ |
176 | 176 | $resourceManager = $this->getContainer()->query(IManager::class); |
177 | 177 | $resourceManager->registerResourceProvider(\OCA\Deck\Collaboration\Resources\ResourceProvider::class); |
178 | - \OC::$server->getEventDispatcher()->addListener('\OCP\Collaboration\Resources::loadAdditionalScripts', function () { |
|
178 | + \OC::$server->getEventDispatcher()->addListener('\OCP\Collaboration\Resources::loadAdditionalScripts', function() { |
|
179 | 179 | \OCP\Util::addScript('deck', 'build/collections'); |
180 | 180 | }); |
181 | 181 | } |
@@ -97,8 +97,8 @@ discard block |
||
97 | 97 | public function findAll($since = 0, $details = null) { |
98 | 98 | $userInfo = $this->getBoardPrerequisites(); |
99 | 99 | $userBoards = $this->boardMapper->findAllByUser($userInfo['user'], null, null, $since); |
100 | - $groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'],null, null, $since); |
|
101 | - $circleBoards = $this->boardMapper->findAllByCircles($userInfo['user'], null, null, $since); |
|
100 | + $groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups'], null, null, $since); |
|
101 | + $circleBoards = $this->boardMapper->findAllByCircles($userInfo['user'], null, null, $since); |
|
102 | 102 | $complete = array_merge($userBoards, $groupBoards, $circleBoards); |
103 | 103 | $result = []; |
104 | 104 | /** @var Board $item */ |
@@ -138,7 +138,7 @@ discard block |
||
138 | 138 | */ |
139 | 139 | public function find($boardId) { |
140 | 140 | |
141 | - if ( is_numeric($boardId) === false ) { |
|
141 | + if (is_numeric($boardId) === false) { |
|
142 | 142 | throw new BadRequestException('board id must be a number'); |
143 | 143 | } |
144 | 144 | |
@@ -186,7 +186,7 @@ discard block |
||
186 | 186 | */ |
187 | 187 | public function isArchived($mapper, $id) { |
188 | 188 | |
189 | - if (is_numeric($id) === false) { |
|
189 | + if (is_numeric($id) === false) { |
|
190 | 190 | throw new BadRequestException('id must be a number'); |
191 | 191 | } |
192 | 192 | |
@@ -220,7 +220,7 @@ discard block |
||
220 | 220 | throw new BadRequestException('mapper must be provided'); |
221 | 221 | } |
222 | 222 | |
223 | - if (is_numeric($id) === false) { |
|
223 | + if (is_numeric($id) === false) { |
|
224 | 224 | throw new BadRequestException('id must be a number'); |
225 | 225 | } |
226 | 226 | |
@@ -358,7 +358,7 @@ discard block |
||
358 | 358 | * @throws BadRequestException |
359 | 359 | */ |
360 | 360 | public function deleteForce($id) { |
361 | - if (is_numeric($id) === false) { |
|
361 | + if (is_numeric($id) === false) { |
|
362 | 362 | throw new BadRequestException('id must be a number'); |
363 | 363 | } |
364 | 364 | |
@@ -392,7 +392,7 @@ discard block |
||
392 | 392 | throw new BadRequestException('color must be provided'); |
393 | 393 | } |
394 | 394 | |
395 | - if ( is_bool($archived) === false ) { |
|
395 | + if (is_bool($archived) === false) { |
|
396 | 396 | throw new BadRequestException('archived must be a boolean'); |
397 | 397 | } |
398 | 398 | |
@@ -554,7 +554,7 @@ discard block |
||
554 | 554 | private function enrichWithStacks($board, $since = -1) { |
555 | 555 | $stacks = $this->stackMapper->findAll($board->getId(), null, null, $since); |
556 | 556 | |
557 | - if(\count($stacks) === 0) { |
|
557 | + if (\count($stacks) === 0) { |
|
558 | 558 | return; |
559 | 559 | } |
560 | 560 | |
@@ -564,7 +564,7 @@ discard block |
||
564 | 564 | private function enrichWithLabels($board, $since = -1) { |
565 | 565 | $labels = $this->labelMapper->findAll($board->getId(), null, null, $since); |
566 | 566 | |
567 | - if(\count($labels) === 0) { |
|
567 | + if (\count($labels) === 0) { |
|
568 | 568 | return; |
569 | 569 | } |
570 | 570 | |
@@ -573,7 +573,7 @@ discard block |
||
573 | 573 | |
574 | 574 | private function enrichWithUsers($board, $since = -1) { |
575 | 575 | $boardUsers = $this->permissionService->findUsers($board->getId()); |
576 | - if(\count($boardUsers) === 0) { |
|
576 | + if (\count($boardUsers) === 0) { |
|
577 | 577 | return; |
578 | 578 | } |
579 | 579 | $board->setUsers(array_values($boardUsers)); |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | |
99 | 99 | try { |
100 | 100 | $this->fullTextSearchManager->createIndex( |
101 | - DeckProvider::DECK_PROVIDER_ID, (string)$cardId, $userId, IIndex::INDEX_FULL |
|
101 | + DeckProvider::DECK_PROVIDER_ID, (string) $cardId, $userId, IIndex::INDEX_FULL |
|
102 | 102 | ); |
103 | 103 | } catch (FullTextSearchAppNotAvailableException $e) { |
104 | 104 | } |
@@ -113,7 +113,7 @@ discard block |
||
113 | 113 | |
114 | 114 | try { |
115 | 115 | $this->fullTextSearchManager->updateIndexStatus( |
116 | - DeckProvider::DECK_PROVIDER_ID, (string)$cardId, IIndex::INDEX_CONTENT |
|
116 | + DeckProvider::DECK_PROVIDER_ID, (string) $cardId, IIndex::INDEX_CONTENT |
|
117 | 117 | ); |
118 | 118 | } catch (FullTextSearchAppNotAvailableException $e) { |
119 | 119 | } |
@@ -128,7 +128,7 @@ discard block |
||
128 | 128 | |
129 | 129 | try { |
130 | 130 | $this->fullTextSearchManager->updateIndexStatus( |
131 | - DeckProvider::DECK_PROVIDER_ID, (string)$cardId, IIndex::INDEX_REMOVE |
|
131 | + DeckProvider::DECK_PROVIDER_ID, (string) $cardId, IIndex::INDEX_REMOVE |
|
132 | 132 | ); |
133 | 133 | } catch (FullTextSearchAppNotAvailableException $e) { |
134 | 134 | } |
@@ -139,7 +139,7 @@ discard block |
||
139 | 139 | * @param GenericEvent $e |
140 | 140 | */ |
141 | 141 | public function onBoardShares(GenericEvent $e) { |
142 | - $boardId = (int)$e->getArgument('boardId'); |
|
142 | + $boardId = (int) $e->getArgument('boardId'); |
|
143 | 143 | |
144 | 144 | $cards = array_map( |
145 | 145 | function(Card $item) { |
@@ -162,7 +162,7 @@ discard block |
||
162 | 162 | * @return IIndexDocument |
163 | 163 | */ |
164 | 164 | public function generateIndexDocumentFromCard(Card $card): IIndexDocument { |
165 | - $document = new IndexDocument(DeckProvider::DECK_PROVIDER_ID, (string)$card->getId()); |
|
165 | + $document = new IndexDocument(DeckProvider::DECK_PROVIDER_ID, (string) $card->getId()); |
|
166 | 166 | |
167 | 167 | return $document; |
168 | 168 | } |
@@ -176,11 +176,11 @@ discard block |
||
176 | 176 | */ |
177 | 177 | public function fillIndexDocument(IIndexDocument $document) { |
178 | 178 | /** @var Card $card */ |
179 | - $card = $this->cardMapper->find((int)$document->getId()); |
|
179 | + $card = $this->cardMapper->find((int) $document->getId()); |
|
180 | 180 | |
181 | 181 | $document->setTitle($card->getTitle()); |
182 | 182 | $document->setContent($card->getDescription()); |
183 | - $document->setAccess($this->generateDocumentAccessFromCardId((int)$card->getId())); |
|
183 | + $document->setAccess($this->generateDocumentAccessFromCardId((int) $card->getId())); |
|
184 | 184 | } |
185 | 185 | |
186 | 186 | |
@@ -241,7 +241,7 @@ discard block |
||
241 | 241 | * @throws MultipleObjectsReturnedException |
242 | 242 | */ |
243 | 243 | public function getBoardFromCardId(int $cardId): Board { |
244 | - $boardId = (int)$this->cardMapper->findBoardId($cardId); |
|
244 | + $boardId = (int) $this->cardMapper->findBoardId($cardId); |
|
245 | 245 | /** @var Board $board */ |
246 | 246 | $board = $this->boardMapper->find($boardId); |
247 | 247 |
@@ -281,9 +281,9 @@ |
||
281 | 281 | if (array_key_exists('comment', $subjectParams)) { |
282 | 282 | /** @var IComment $comment */ |
283 | 283 | try { |
284 | - $comment = $this->commentsManager->get((int)$subjectParams['comment']); |
|
284 | + $comment = $this->commentsManager->get((int) $subjectParams['comment']); |
|
285 | 285 | $event->setParsedMessage($comment->getMessage()); |
286 | - $params['comment'] =[ |
|
286 | + $params['comment'] = [ |
|
287 | 287 | 'type' => 'highlight', |
288 | 288 | 'id' => $subjectParams['comment'], |
289 | 289 | 'name' => $comment->getMessage() |