@@ -251,7 +251,7 @@ discard block |
||
251 | 251 | * |
252 | 252 | * @param int $id Id of the milestone |
253 | 253 | * |
254 | - * @return Tuleap\AgileDashboard\REST\v1\MilestoneRepresentation |
|
254 | + * @return MilestoneRepresentation |
|
255 | 255 | * |
256 | 256 | * @throws 403 |
257 | 257 | * @throws 404 |
@@ -800,6 +800,9 @@ discard block |
||
800 | 800 | $this->sendAllowHeaderForBacklog(); |
801 | 801 | } |
802 | 802 | |
803 | + /** |
|
804 | + * @param PFUser $user |
|
805 | + */ |
|
803 | 806 | private function getBacklogItemAsArtifact($user, $artifact_id) { |
804 | 807 | $artifact = $this->tracker_artifact_factory->getArtifactById($artifact_id); |
805 | 808 | |
@@ -894,6 +897,9 @@ discard block |
||
894 | 897 | return $burndown; |
895 | 898 | } |
896 | 899 | |
900 | + /** |
|
901 | + * @param integer $id |
|
902 | + */ |
|
897 | 903 | private function getMilestoneById(PFUser $user, $id) { |
898 | 904 | try { |
899 | 905 | $milestone = $this->milestone_factory->getValidatedBareMilestoneByArtifactId($user, $id); |
@@ -917,6 +923,10 @@ discard block |
||
917 | 923 | return UserManager::instance()->getCurrentUser(); |
918 | 924 | } |
919 | 925 | |
926 | + /** |
|
927 | + * @param \Planning_ArtifactMilestone $milestone |
|
928 | + * @param \AgileDashboard_Milestone_Backlog_DescendantBacklogStrategy $strategy |
|
929 | + */ |
|
920 | 930 | private function getMilestoneContentItems($milestone, $strategy) { |
921 | 931 | return $this->backlog_item_collection_factory->getAllCollection( |
922 | 932 | $this->getCurrentUser(), |
@@ -926,6 +936,9 @@ discard block |
||
926 | 936 | ); |
927 | 937 | } |
928 | 938 | |
939 | + /** |
|
940 | + * @param integer $limit |
|
941 | + */ |
|
929 | 942 | private function checkContentLimit($limit) { |
930 | 943 | if (! $this->limitValueIsAcceptable($limit)) { |
931 | 944 | throw new RestException(406, 'Maximum value for limit exceeded'); |
@@ -940,6 +953,10 @@ discard block |
||
940 | 953 | Header::allowOptionsGetPutPatch(); |
941 | 954 | } |
942 | 955 | |
956 | + /** |
|
957 | + * @param integer $limit |
|
958 | + * @param integer $offset |
|
959 | + */ |
|
943 | 960 | private function sendPaginationHeaders($limit, $offset, $size) { |
944 | 961 | Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT); |
945 | 962 | } |
@@ -952,6 +969,9 @@ discard block |
||
952 | 969 | Header::allowOptionsGetPut(); |
953 | 970 | } |
954 | 971 | |
972 | + /** |
|
973 | + * @param \Planning_ArtifactMilestone $milestone |
|
974 | + */ |
|
955 | 975 | private function sendAllowHeadersForMilestone($milestone) { |
956 | 976 | $date = $milestone->getLastModifiedDate(); |
957 | 977 | Header::allowOptionsGet(); |
@@ -96,6 +96,9 @@ discard block |
||
96 | 96 | return $this->validateArtifactsForMilestoneContent($ids, $milestone, $user, true); |
97 | 97 | } |
98 | 98 | |
99 | + /** |
|
100 | + * @param boolean $accept_closed |
|
101 | + */ |
|
99 | 102 | private function validateArtifactsForMilestoneContent(array $ids, Planning_Milestone $milestone, PFUser $user, $accept_closed) { |
100 | 103 | $open_unplanned = null; |
101 | 104 | $todo = null; |
@@ -274,6 +277,9 @@ discard block |
||
274 | 277 | return $index; |
275 | 278 | } |
276 | 279 | |
280 | + /** |
|
281 | + * @param Planning_Milestone $milestone |
|
282 | + */ |
|
277 | 283 | private function getMilestoneBacklogItems(PFUser $user, $milestone) { |
278 | 284 | return $this->backlog_item_collection_factory->getUnplannedOpenCollection( |
279 | 285 | $user, |
@@ -117,6 +117,9 @@ discard block |
||
117 | 117 | return $planning; |
118 | 118 | } |
119 | 119 | |
120 | + /** |
|
121 | + * @param integer $limit |
|
122 | + */ |
|
120 | 123 | private function limitValueIsAcceptable($limit) { |
121 | 124 | return $limit <= self::MAX_LIMIT; |
122 | 125 | } |
@@ -125,6 +128,10 @@ discard block |
||
125 | 128 | return UserManager::instance()->getCurrentUser(); |
126 | 129 | } |
127 | 130 | |
131 | + /** |
|
132 | + * @param integer $limit |
|
133 | + * @param integer $offset |
|
134 | + */ |
|
128 | 135 | private function getMilestonesByPlanning(Planning $planning, $limit, $offset) { |
129 | 136 | $all_milestones = array(); |
130 | 137 | $milestones = $this->milestone_factory->getAllBareMilestones($this->getCurrentUser(), $planning); |
@@ -137,6 +144,9 @@ discard block |
||
137 | 144 | return $milestones_representations; |
138 | 145 | } |
139 | 146 | |
147 | + /** |
|
148 | + * @param integer $size |
|
149 | + */ |
|
140 | 150 | private function sendPaginationHeaders($limit, $offset, $size) { |
141 | 151 | Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT); |
142 | 152 | } |
@@ -233,6 +233,10 @@ discard block |
||
233 | 233 | } |
234 | 234 | } |
235 | 235 | |
236 | + /** |
|
237 | + * @param PFUser $user |
|
238 | + * @param Project $project |
|
239 | + */ |
|
236 | 240 | private function validateArtifactIdsAreInUnassignedTopBacklog($ids, $user, $project) { |
237 | 241 | try { |
238 | 242 | $this->milestone_validator->validateArtifactIdsAreInUnassignedTopBacklog($ids, $user, $project); |
@@ -245,6 +249,9 @@ discard block |
||
245 | 249 | } |
246 | 250 | } |
247 | 251 | |
252 | + /** |
|
253 | + * @param integer $size |
|
254 | + */ |
|
248 | 255 | private function sendPaginationHeaders($limit, $offset, $size) { |
249 | 256 | Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT); |
250 | 257 | } |
@@ -137,6 +137,9 @@ |
||
137 | 137 | $this->sendAllowHeaders(); |
138 | 138 | } |
139 | 139 | |
140 | + /** |
|
141 | + * @param integer $size |
|
142 | + */ |
|
140 | 143 | private function sendPaginationHeaders($limit, $offset, $size) { |
141 | 144 | Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT); |
142 | 145 | } |
@@ -63,6 +63,9 @@ |
||
63 | 63 | $this->sendAllowHeaders(); |
64 | 64 | } |
65 | 65 | |
66 | + /** |
|
67 | + * @param integer $size |
|
68 | + */ |
|
66 | 69 | private function sendPaginationHeaders($limit, $offset, $size) { |
67 | 70 | Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT); |
68 | 71 | } |
@@ -32,6 +32,9 @@ |
||
32 | 32 | return $backlog_item_representation; |
33 | 33 | } |
34 | 34 | |
35 | + /** |
|
36 | + * @param AgileDashboard_Milestone_Backlog_IBacklogItem $backlog_item |
|
37 | + */ |
|
35 | 38 | private function getBacklogItemCardFields($backlog_item) { |
36 | 39 | $current_user = UserManager::instance()->getCurrentUser(); |
37 | 40 | $card_fields_semantic = $this->getCardFieldsSemantic($backlog_item); |
@@ -178,6 +178,9 @@ |
||
178 | 178 | return $this->backlog_item_collection_factory->getUnassignedOpenCollection($user, $top_milestone, $strategy_unassigned, false); |
179 | 179 | } |
180 | 180 | |
181 | + /** |
|
182 | + * @param integer $size |
|
183 | + */ |
|
181 | 184 | private function sendPaginationHeaders($limit, $offset, $size) { |
182 | 185 | Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT); |
183 | 186 | } |
@@ -30,6 +30,7 @@ discard block |
||
30 | 30 | } |
31 | 31 | |
32 | 32 | /** |
33 | + * @param integer $tracker_id |
|
33 | 34 | * @return mixed either false if error or object DataAccessResult |
34 | 35 | */ |
35 | 36 | public function searchByTrackerId($tracker_id) { |
@@ -43,6 +44,7 @@ discard block |
||
43 | 44 | } |
44 | 45 | |
45 | 46 | /** |
47 | + * @param integer $tracker_id |
|
46 | 48 | * @return boolean true if success |
47 | 49 | */ |
48 | 50 | public function save($tracker_id, $field_id) { |