Completed
Pull Request — master (#653)
by rakekniven
23:11
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/Service/PermissionService.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@
 block discarded – undo
141 141
 
142 142
 	/**
143 143
 	 * @param $boardId
144
-	 * @return bool
144
+	 * @return boolean|null
145 145
 	 */
146 146
 	public function userIsBoardOwner($boardId) {
147 147
 		try {
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/Notification/NotificationHelper.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -94,6 +94,9 @@
 block discarded – undo
94 94
 		$this->cardMapper->markNotified($card);
95 95
 	}
96 96
 
97
+	/**
98
+	 * @param \OCA\Deck\Db\Card $card
99
+	 */
97 100
 	public function sendCardAssigned($card, $userId) {
98 101
 		$boardId = $this->cardMapper->findBoardId($card->getId());
99 102
 		$board = $this->getBoard($boardId);
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/Activity/DeckProvider.php 1 patch
Doc Comments   +14 added lines patch added patch discarded remove patch
@@ -166,6 +166,9 @@  discard block
 block discarded – undo
166 166
 		return $event;
167 167
 	}
168 168
 
169
+	/**
170
+	 * @param string $paramName
171
+	 */
169 172
 	private function parseParamForBoard($paramName, $subjectParams, $params) {
170 173
 		if (array_key_exists($paramName, $subjectParams)) {
171 174
 			$params[$paramName] = [
@@ -177,6 +180,10 @@  discard block
 block discarded – undo
177 180
 		}
178 181
 		return $params;
179 182
 	}
183
+
184
+	/**
185
+	 * @param string $paramName
186
+	 */
180 187
 	private function parseParamForStack($paramName, $subjectParams, $params) {
181 188
 		if (array_key_exists($paramName, $subjectParams)) {
182 189
 			$params[$paramName] = [
@@ -188,6 +195,9 @@  discard block
 block discarded – undo
188 195
 		return $params;
189 196
 	}
190 197
 
198
+	/**
199
+	 * @param string $paramName
200
+	 */
191 201
 	private function parseParamForAttachment($paramName, $subjectParams, $params) {
192 202
 		if (array_key_exists($paramName, $subjectParams)) {
193 203
 			$params[$paramName] = [
@@ -226,6 +236,7 @@  discard block
 block discarded – undo
226 236
 	 *
227 237
 	 * @param $subjectParams
228 238
 	 * @param $params
239
+	 * @param IEvent $event
229 240
 	 * @return mixed
230 241
 	 */
231 242
 	private function parseParamForChanges($subjectParams, $params, $event) {
@@ -252,6 +263,9 @@  discard block
 block discarded – undo
252 263
 		return $params;
253 264
 	}
254 265
 
266
+	/**
267
+	 * @param string $endpoint
268
+	 */
255 269
 	public function deckUrl($endpoint) {
256 270
 		return $this->urlGenerator->linkToRoute('deck.page.index') . '#!' . $endpoint;
257 271
 	}
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.