@@ -111,7 +111,7 @@ discard block |
||
| 111 | 111 | /** |
| 112 | 112 | * Find all boards for a given user |
| 113 | 113 | * |
| 114 | - * @param $userId |
|
| 114 | + * @param string $userId |
|
| 115 | 115 | * @param $groups |
| 116 | 116 | * @param null $limit |
| 117 | 117 | * @param null $offset |
@@ -139,6 +139,9 @@ discard block |
||
| 139 | 139 | return $entries; |
| 140 | 140 | } |
| 141 | 141 | |
| 142 | + /** |
|
| 143 | + * @param string $userId |
|
| 144 | + */ |
|
| 142 | 145 | public function findAllByCircles($userId, $limit = null, $offset = null) { |
| 143 | 146 | if (!$this->circlesEnabled) { |
| 144 | 147 | return []; |
@@ -208,6 +211,9 @@ discard block |
||
| 208 | 211 | return ($board->getOwner() === $userId); |
| 209 | 212 | } |
| 210 | 213 | |
| 214 | + /** |
|
| 215 | + * @param integer $id |
|
| 216 | + */ |
|
| 211 | 217 | public function findBoardId($id) { |
| 212 | 218 | return $id; |
| 213 | 219 | } |
@@ -134,7 +134,7 @@ |
||
| 134 | 134 | } |
| 135 | 135 | |
| 136 | 136 | /** |
| 137 | - * @param $cardId |
|
| 137 | + * @param string $cardId |
|
| 138 | 138 | * @return Board |
| 139 | 139 | * @throws DoesNotExistException |
| 140 | 140 | * @throws MultipleObjectsReturnedException |
@@ -180,7 +180,7 @@ discard block |
||
| 180 | 180 | } |
| 181 | 181 | |
| 182 | 182 | /** |
| 183 | - * @return array |
|
| 183 | + * @return integer |
|
| 184 | 184 | */ |
| 185 | 185 | private function getBoardPrerequisites() { |
| 186 | 186 | $groups = $this->groupManager->getUserGroupIds( |
@@ -658,6 +658,9 @@ discard block |
||
| 658 | 658 | return $newBoard; |
| 659 | 659 | } |
| 660 | 660 | |
| 661 | + /** |
|
| 662 | + * @param Board $board |
|
| 663 | + */ |
|
| 661 | 664 | private function enrichWithStacks($board, $since = -1) { |
| 662 | 665 | $stacks = $this->stackMapper->findAll($board->getId(), null, null, $since); |
| 663 | 666 | |
@@ -668,6 +671,9 @@ discard block |
||
| 668 | 671 | $board->setStacks($stacks); |
| 669 | 672 | } |
| 670 | 673 | |
| 674 | + /** |
|
| 675 | + * @param Board $board |
|
| 676 | + */ |
|
| 671 | 677 | private function enrichWithLabels($board, $since = -1) { |
| 672 | 678 | $labels = $this->labelMapper->findAll($board->getId(), null, null, $since); |
| 673 | 679 | |
@@ -678,6 +684,9 @@ discard block |
||
| 678 | 684 | $board->setLabels($labels); |
| 679 | 685 | } |
| 680 | 686 | |
| 687 | + /** |
|
| 688 | + * @param Board $board |
|
| 689 | + */ |
|
| 681 | 690 | private function enrichWithUsers($board, $since = -1) { |
| 682 | 691 | $boardUsers = $this->permissionService->findUsers($board->getId()); |
| 683 | 692 | if(\count($boardUsers) === 0) { |
@@ -326,8 +326,8 @@ |
||
| 326 | 326 | } |
| 327 | 327 | |
| 328 | 328 | /** |
| 329 | - * @param $id |
|
| 330 | - * @param $order |
|
| 329 | + * @param integer $id |
|
| 330 | + * @param integer $order |
|
| 331 | 331 | * |
| 332 | 332 | * @return array |
| 333 | 333 | * @throws \OCA\Deck\NoPermissionException |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | } |
| 128 | 128 | |
| 129 | 129 | /** |
| 130 | - * @param $subjectIdentifier |
|
| 130 | + * @param string $subjectIdentifier |
|
| 131 | 131 | * @param array $subjectParams |
| 132 | 132 | * @param bool $ownActivity |
| 133 | 133 | * @return string |
@@ -240,6 +240,9 @@ discard block |
||
| 240 | 240 | return $subject; |
| 241 | 241 | } |
| 242 | 242 | |
| 243 | + /** |
|
| 244 | + * @param \OCP\AppFramework\Db\Entity $entity |
|
| 245 | + */ |
|
| 243 | 246 | public function triggerEvent($objectType, $entity, $subject, $additionalParams = [], $author = null) { |
| 244 | 247 | try { |
| 245 | 248 | $event = $this->createEvent($objectType, $entity, $subject, $additionalParams, $author); |
@@ -94,6 +94,7 @@ |
||
| 94 | 94 | |
| 95 | 95 | /** |
| 96 | 96 | * @inheritDoc |
| 97 | + * @return string |
|
| 97 | 98 | */ |
| 98 | 99 | function get() { |
| 99 | 100 | if ($this->card) { |
@@ -68,7 +68,7 @@ discard block |
||
| 68 | 68 | } |
| 69 | 69 | |
| 70 | 70 | /** |
| 71 | - * @param $card |
|
| 71 | + * @param \OCP\AppFramework\Db\Entity $card |
|
| 72 | 72 | * @throws \OCP\AppFramework\Db\DoesNotExistException |
| 73 | 73 | */ |
| 74 | 74 | public function sendCardDuedate($card) { |
@@ -99,6 +99,9 @@ discard block |
||
| 99 | 99 | $this->cardMapper->markNotified($card); |
| 100 | 100 | } |
| 101 | 101 | |
| 102 | + /** |
|
| 103 | + * @param \OCA\Deck\Db\Card $card |
|
| 104 | + */ |
|
| 102 | 105 | public function markDuedateAsRead($card) { |
| 103 | 106 | $notification = $this->notificationManager->createNotification(); |
| 104 | 107 | $notification |
@@ -108,6 +111,9 @@ discard block |
||
| 108 | 111 | $this->notificationManager->markProcessed($notification); |
| 109 | 112 | } |
| 110 | 113 | |
| 114 | + /** |
|
| 115 | + * @param \OCA\Deck\Db\Card $card |
|
| 116 | + */ |
|
| 111 | 117 | public function sendCardAssigned($card, $userId) { |
| 112 | 118 | $boardId = $this->cardMapper->findBoardId($card->getId()); |
| 113 | 119 | $board = $this->getBoard($boardId); |
@@ -99,6 +99,9 @@ discard block |
||
| 99 | 99 | $this->currentUser = $userId; |
| 100 | 100 | } |
| 101 | 101 | |
| 102 | + /** |
|
| 103 | + * @param \OCP\AppFramework\Db\Entity $card |
|
| 104 | + */ |
|
| 102 | 105 | public function enrich($card) { |
| 103 | 106 | $cardId = $card->getId(); |
| 104 | 107 | $this->cardMapper->mapOwner($card); |
@@ -153,7 +156,7 @@ discard block |
||
| 153 | 156 | /** |
| 154 | 157 | * @param $title |
| 155 | 158 | * @param $stackId |
| 156 | - * @param $type |
|
| 159 | + * @param string $type |
|
| 157 | 160 | * @param integer $order |
| 158 | 161 | * @param $description |
| 159 | 162 | * @param $owner |
@@ -582,7 +585,7 @@ discard block |
||
| 582 | 585 | /** |
| 583 | 586 | * @param $cardId |
| 584 | 587 | * @param $userId |
| 585 | - * @return bool|null|\OCP\AppFramework\Db\Entity |
|
| 588 | + * @return AssignedUsers|null |
|
| 586 | 589 | * @throws BadRequestException |
| 587 | 590 | * @throws \OCA\Deck\NoPermissionException |
| 588 | 591 | * @throws \OCP\AppFramework\Db\DoesNotExistException |