Passed
Pull Request — master (#1483)
by Julius
03:07
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/Controller/LabelApiController.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -32,10 +32,10 @@
 block discarded – undo
32 32
 use OCA\Deck\Controller\Helper\ApiHelper;
33 33
 
34 34
  /**
35
- * Class BoardApiController
36
- *
37
- * @package OCA\Deck\Controller
38
- */
35
+  * Class BoardApiController
36
+  *
37
+  * @package OCA\Deck\Controller
38
+  */
39 39
 class LabelApiController extends ApiController {
40 40
 
41 41
 	private $labelService;
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/Service/DefaultBoardService.php 1 patch
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	private $config;
41 41
 	private $l10n;
42 42
 
43
-    public function __construct(
43
+	public function __construct(
44 44
 			IL10N $l10n,
45 45
 			BoardMapper $boardMapper,
46 46
 			BoardService $boardService,
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 		$this->config = $config;
56 56
 		$this->boardMapper = $boardMapper;
57 57
 		$this->l10n = $l10n;
58
-    }
58
+	}
59 59
 
60 60
 	/**
61 61
 	 * @param $userId
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 		}
74 74
 
75 75
 		return false;
76
-    }
76
+	}
77 77
 
78 78
 	/**
79 79
 	 * @param $title
@@ -100,9 +100,9 @@  discard block
 block discarded – undo
100 100
 			throw new BadRequestException('color must be provided');
101 101
 		}
102 102
 
103
-        $defaultBoard = $this->boardService->create($title, $userId, $color);
104
-        $defaultStacks = [];
105
-        $defaultCards = [];
103
+		$defaultBoard = $this->boardService->create($title, $userId, $color);
104
+		$defaultStacks = [];
105
+		$defaultCards = [];
106 106
 
107 107
 		$boardId = $defaultBoard->getId();		
108 108
 
@@ -115,5 +115,5 @@  discard block
 block discarded – undo
115 115
 		$defaultCards[] = $this->cardService->create($this->l10n->t('Example Task 1'), $defaultStacks[2]->getId(), 'text', 0, $userId);
116 116
 
117 117
 		return $defaultBoard;
118
-    }
118
+	}
119 119
 }
120 120
\ No newline at end of file
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/Controller/CardApiController.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -31,10 +31,10 @@
 block discarded – undo
31 31
  use OCA\Deck\Service\CardService;
32 32
 
33 33
  /**
34
- * Class BoardApiController
35
- *
36
- * @package OCA\Deck\Controller
37
- */
34
+  * Class BoardApiController
35
+  *
36
+  * @package OCA\Deck\Controller
37
+  */
38 38
 class CardApiController extends ApiController {
39 39
 	private $cardService;
40 40
 	private $userId;
Please login to merge, or discard this patch.