Passed
Pull Request — master (#1545)
by Julius
03:01
created
lib/Db/BoardMapper.php 1 patch
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 	}
Please login to merge, or discard this patch.
lib/Collaboration/Resources/ResourceProviderCard.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@
 block discarded – undo
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
Please login to merge, or discard this patch.
lib/Service/BoardService.php 1 patch
Doc Comments   +10 added lines, -1 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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) {
Please login to merge, or discard this patch.
lib/Service/StackService.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -326,8 +326,8 @@
 block discarded – undo
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
Please login to merge, or discard this patch.
lib/Activity/ActivityManager.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
lib/DAV/CalendarObject.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -94,6 +94,7 @@
 block discarded – undo
94 94
 
95 95
 	/**
96 96
 	 * @inheritDoc
97
+	 * @return string
97 98
 	 */
98 99
 	function get() {
99 100
 		if ($this->card) {
Please login to merge, or discard this patch.
lib/Notification/NotificationHelper.php 1 patch
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
lib/Service/CardService.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -99,6 +99,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.