@@ -77,7 +77,7 @@ |
||
| 77 | 77 | * DeckProvider constructor. |
| 78 | 78 | * |
| 79 | 79 | * @param IL10N $l10n |
| 80 | - * @param IUrlGenerator $urlGenerator |
|
| 80 | + * @param IURLGenerator $urlGenerator |
|
| 81 | 81 | * @param FullTextSearchService $fullTextSearchService |
| 82 | 82 | */ |
| 83 | 83 | public function __construct( |
@@ -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) { |
@@ -44,7 +44,6 @@ |
||
| 44 | 44 | use OCP\FullTextSearch\Model\ISearchResult; |
| 45 | 45 | use OCP\FullTextSearch\Model\ISearchTemplate; |
| 46 | 46 | use OCP\IL10N; |
| 47 | -use OCP\IURLGenerator; |
|
| 48 | 47 | |
| 49 | 48 | |
| 50 | 49 | /** |
@@ -96,7 +96,7 @@ discard block |
||
| 96 | 96 | $userId = $e->getArgument('userId'); |
| 97 | 97 | |
| 98 | 98 | $this->fullTextSearchManager->createIndex( |
| 99 | - DeckProvider::DECK_PROVIDER_ID, (string)$cardId, $userId, IIndex::INDEX_FULL |
|
| 99 | + DeckProvider::DECK_PROVIDER_ID, (string) $cardId, $userId, IIndex::INDEX_FULL |
|
| 100 | 100 | ); |
| 101 | 101 | } |
| 102 | 102 | |
@@ -108,7 +108,7 @@ discard block |
||
| 108 | 108 | $cardId = $e->getArgument('id'); |
| 109 | 109 | |
| 110 | 110 | $this->fullTextSearchManager->updateIndexStatus( |
| 111 | - DeckProvider::DECK_PROVIDER_ID, (string)$cardId, IIndex::INDEX_CONTENT |
|
| 111 | + DeckProvider::DECK_PROVIDER_ID, (string) $cardId, IIndex::INDEX_CONTENT |
|
| 112 | 112 | ); |
| 113 | 113 | } |
| 114 | 114 | |
@@ -120,7 +120,7 @@ discard block |
||
| 120 | 120 | $cardId = $e->getArgument('id'); |
| 121 | 121 | |
| 122 | 122 | $this->fullTextSearchManager->updateIndexStatus( |
| 123 | - DeckProvider::DECK_PROVIDER_ID, (string)$cardId, IIndex::INDEX_REMOVE |
|
| 123 | + DeckProvider::DECK_PROVIDER_ID, (string) $cardId, IIndex::INDEX_REMOVE |
|
| 124 | 124 | ); |
| 125 | 125 | } |
| 126 | 126 | |
@@ -149,7 +149,7 @@ discard block |
||
| 149 | 149 | * @return IIndexDocument |
| 150 | 150 | */ |
| 151 | 151 | public function generateIndexDocumentFromCard(Card $card): IIndexDocument { |
| 152 | - $document = new IndexDocument(DeckProvider::DECK_PROVIDER_ID, (string)$card->getId()); |
|
| 152 | + $document = new IndexDocument(DeckProvider::DECK_PROVIDER_ID, (string) $card->getId()); |
|
| 153 | 153 | |
| 154 | 154 | return $document; |
| 155 | 155 | } |
@@ -163,7 +163,7 @@ discard block |
||
| 163 | 163 | */ |
| 164 | 164 | public function fillIndexDocument(IIndexDocument $document) { |
| 165 | 165 | /** @var Card $card */ |
| 166 | - $card = $this->cardMapper->find((int)$document->getId()); |
|
| 166 | + $card = $this->cardMapper->find((int) $document->getId()); |
|
| 167 | 167 | |
| 168 | 168 | $document->setTitle($card->getTitle()); |
| 169 | 169 | $document->setContent($card->getDescription()); |
@@ -230,7 +230,7 @@ discard block |
||
| 230 | 230 | * @throws MultipleObjectsReturnedException |
| 231 | 231 | */ |
| 232 | 232 | public function getBoardFromCardId(int $cardId): Board { |
| 233 | - $boardId = (int)$this->cardMapper->findBoardId($cardId); |
|
| 233 | + $boardId = (int) $this->cardMapper->findBoardId($cardId); |
|
| 234 | 234 | /** @var Board $board */ |
| 235 | 235 | $board = $this->boardMapper->find($boardId); |
| 236 | 236 | |
@@ -33,7 +33,6 @@ |
||
| 33 | 33 | use OCA\Deck\Notification\Notifier; |
| 34 | 34 | use OCA\Deck\Service\FullTextSearchService; |
| 35 | 35 | use OCP\AppFramework\App; |
| 36 | -use OCA\Deck\Middleware\SharingMiddleware; |
|
| 37 | 36 | use OCP\Collaboration\Resources\IManager; |
| 38 | 37 | use OCP\Comments\CommentsEntityEvent; |
| 39 | 38 | use OCP\IGroup; |
@@ -170,7 +170,7 @@ discard block |
||
| 170 | 170 | } |
| 171 | 171 | |
| 172 | 172 | /** |
| 173 | - * @return array |
|
| 173 | + * @return integer |
|
| 174 | 174 | */ |
| 175 | 175 | private function getBoardPrerequisites() { |
| 176 | 176 | $groups = $this->groupManager->getUserGroupIds( |
@@ -605,6 +605,9 @@ discard block |
||
| 605 | 605 | return $delete; |
| 606 | 606 | } |
| 607 | 607 | |
| 608 | + /** |
|
| 609 | + * @param Board $board |
|
| 610 | + */ |
|
| 608 | 611 | private function enrichWithStacks($board, $since = -1) { |
| 609 | 612 | $stacks = $this->stackMapper->findAll($board->getId(), null, null, $since); |
| 610 | 613 | |
@@ -615,6 +618,9 @@ discard block |
||
| 615 | 618 | $board->setStacks($stacks); |
| 616 | 619 | } |
| 617 | 620 | |
| 621 | + /** |
|
| 622 | + * @param Board $board |
|
| 623 | + */ |
|
| 618 | 624 | private function enrichWithLabels($board, $since = -1) { |
| 619 | 625 | $labels = $this->labelMapper->findAll($board->getId(), null, null, $since); |
| 620 | 626 | |
@@ -625,6 +631,9 @@ discard block |
||
| 625 | 631 | $board->setLabels($labels); |
| 626 | 632 | } |
| 627 | 633 | |
| 634 | + /** |
|
| 635 | + * @param Board $board |
|
| 636 | + */ |
|
| 628 | 637 | private function enrichWithUsers($board, $since = -1) { |
| 629 | 638 | $boardUsers = $this->permissionService->findUsers($board->getId()); |
| 630 | 639 | if(\count($boardUsers) === 0) { |