Completed
Pull Request — master (#991)
by Maxence
02:17
created
lib/Db/AssignedUsersMapper.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
 	 * Check if user exists before assigning it to a card
64 64
 	 *
65 65
 	 * @param Entity $entity
66
-	 * @return null|Entity
66
+	 * @return AssignedUsers|null
67 67
 	 */
68 68
 	public function insert(Entity $entity) {
69 69
 		$user = $this->userManager->get($entity->getParticipant());
Please login to merge, or discard this patch.
lib/Service/AttachmentService.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@
 block discarded – undo
163 163
 	 * @param $cardId
164 164
 	 * @param $attachmentId
165 165
 	 * @param $request
166
-	 * @return mixed
166
+	 * @return \OCP\AppFramework\Db\Entity
167 167
 	 * @throws \OCA\Deck\NoPermissionException
168 168
 	 * @throws \OCP\AppFramework\Db\DoesNotExistException
169 169
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
Please login to merge, or discard this patch.
lib/Db/CardMapper.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -136,6 +136,9 @@
 block discarded – undo
136 136
 		return parent::delete($entity);
137 137
 	}
138 138
 
139
+	/**
140
+	 * @param integer $stackId
141
+	 */
139 142
 	public function deleteByStack($stackId) {
140 143
 		$cards = $this->findAllByStack($stackId);
141 144
 		foreach ($cards as $card) {
Please login to merge, or discard this patch.
lib/Db/BoardMapper.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -173,6 +173,9 @@
 block discarded – undo
173 173
 		return ($board->getOwner() === $userId);
174 174
 	}
175 175
 
176
+	/**
177
+	 * @param string $id
178
+	 */
176 179
 	public function findBoardId($id) {
177 180
 		return $id;
178 181
 	}
Please login to merge, or discard this patch.
lib/Db/DeckMapper.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -45,6 +45,9 @@
 block discarded – undo
45 45
 		return $this->findEntity($sql, [$id]);
46 46
 	}
47 47
 
48
+	/**
49
+	 * @param string $sql
50
+	 */
48 51
 	protected function execute($sql, array $params = [], $limit = null, $offset = null) {
49 52
 		return parent::execute($sql, $params, $limit, $offset);
50 53
 	}
Please login to merge, or discard this patch.
lib/Db/LabelMapper.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -66,6 +66,9 @@  discard block
 block discarded – undo
66 66
 		return $result;
67 67
 	}
68 68
 
69
+	/**
70
+	 * @param integer $labelId
71
+	 */
69 72
 	public function deleteLabelAssignments($labelId) {
70 73
 		$sql = 'DELETE FROM `*PREFIX*deck_assigned_labels` WHERE label_id = ?';
71 74
 		$stmt = $this->db->prepare($sql);
@@ -73,6 +76,9 @@  discard block
 block discarded – undo
73 76
 		$stmt->execute();
74 77
 	}
75 78
 
79
+	/**
80
+	 * @param integer $cardId
81
+	 */
76 82
 	public function deleteLabelAssignmentsForCard($cardId) {
77 83
 		$sql = 'DELETE FROM `*PREFIX*deck_assigned_labels` WHERE card_id = ?';
78 84
 		$stmt = $this->db->prepare($sql);
Please login to merge, or discard this patch.
lib/Activity/ActivityManager.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@
 block discarded – undo
124 124
 	}
125 125
 
126 126
 	/**
127
-	 * @param $subjectIdentifier
127
+	 * @param string $subjectIdentifier
128 128
 	 * @param array $subjectParams
129 129
 	 * @param bool $ownActivity
130 130
 	 * @return string
Please login to merge, or discard this patch.
lib/Activity/DeckProvider.php 1 patch
Doc Comments   +14 added lines patch added patch discarded remove patch
@@ -189,6 +189,9 @@  discard block
 block discarded – undo
189 189
 		return $event;
190 190
 	}
191 191
 
192
+	/**
193
+	 * @param string $paramName
194
+	 */
192 195
 	private function parseParamForBoard($paramName, $subjectParams, $params) {
193 196
 		if (array_key_exists($paramName, $subjectParams)) {
194 197
 			$params[$paramName] = [
@@ -200,6 +203,10 @@  discard block
 block discarded – undo
200 203
 		}
201 204
 		return $params;
202 205
 	}
206
+
207
+	/**
208
+	 * @param string $paramName
209
+	 */
203 210
 	private function parseParamForStack($paramName, $subjectParams, $params) {
204 211
 		if (array_key_exists($paramName, $subjectParams)) {
205 212
 			$params[$paramName] = [
@@ -211,6 +218,9 @@  discard block
 block discarded – undo
211 218
 		return $params;
212 219
 	}
213 220
 
221
+	/**
222
+	 * @param string $paramName
223
+	 */
214 224
 	private function parseParamForAttachment($paramName, $subjectParams, $params) {
215 225
 		if (array_key_exists($paramName, $subjectParams)) {
216 226
 			$params[$paramName] = [
@@ -285,6 +295,7 @@  discard block
 block discarded – undo
285 295
 	 *
286 296
 	 * @param $subjectParams
287 297
 	 * @param $params
298
+	 * @param IEvent $event
288 299
 	 * @return mixed
289 300
 	 */
290 301
 	private function parseParamForChanges($subjectParams, $params, $event) {
@@ -312,6 +323,9 @@  discard block
 block discarded – undo
312 323
 		return $params;
313 324
 	}
314 325
 
326
+	/**
327
+	 * @param string $endpoint
328
+	 */
315 329
 	public function deckUrl($endpoint) {
316 330
 		return $this->urlGenerator->linkToRouteAbsolute('deck.page.index') . '#!' . $endpoint;
317 331
 	}
Please login to merge, or discard this patch.
lib/Notification/NotificationHelper.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -95,6 +95,9 @@  discard block
 block discarded – undo
95 95
 		$this->cardMapper->markNotified($card);
96 96
 	}
97 97
 
98
+	/**
99
+	 * @param \OCA\Deck\Db\Card $card
100
+	 */
98 101
 	public function markDuedateAsRead($card) {
99 102
 		$notification = $this->notificationManager->createNotification();
100 103
 		$notification
@@ -104,6 +107,9 @@  discard block
 block discarded – undo
104 107
 		$this->notificationManager->markProcessed($notification);
105 108
 	}
106 109
 
110
+	/**
111
+	 * @param \OCA\Deck\Db\Card $card
112
+	 */
107 113
 	public function sendCardAssigned($card, $userId) {
108 114
 		$boardId = $this->cardMapper->findBoardId($card->getId());
109 115
 		$board = $this->getBoard($boardId);
Please login to merge, or discard this patch.