@@ -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 |