@@ -108,6 +108,9 @@ discard block |
||
108 | 108 | return $this->done_collection[$milestone->getArtifactId()]; |
109 | 109 | } |
110 | 110 | |
111 | + /** |
|
112 | + * @param false|string $redirect_to_self |
|
113 | + */ |
|
111 | 114 | public function getUnassignedOpenCollection( |
112 | 115 | PFUser $user, |
113 | 116 | Planning_Milestone $milestone, |
@@ -119,6 +122,9 @@ discard block |
||
119 | 122 | return $this->filterOutAssignedBacklogItems($collection, $user); |
120 | 123 | } |
121 | 124 | |
125 | + /** |
|
126 | + * @param boolean $redirect_to_self |
|
127 | + */ |
|
122 | 128 | public function getUnassignedCollection( |
123 | 129 | PFUser $user, |
124 | 130 | Planning_Milestone $milestone, |
@@ -221,6 +227,9 @@ discard block |
||
221 | 227 | return $collection; |
222 | 228 | } |
223 | 229 | |
230 | + /** |
|
231 | + * @param string $redirect_to_self |
|
232 | + */ |
|
224 | 233 | public function getInconsistentCollection( |
225 | 234 | PFUser $user, |
226 | 235 | Planning_Milestone $milestone, |
@@ -232,6 +241,9 @@ discard block |
||
232 | 241 | return $this->inconsistent_collection[$milestone->getArtifactId()]; |
233 | 242 | } |
234 | 243 | |
244 | + /** |
|
245 | + * @param string $redirect_to_self |
|
246 | + */ |
|
235 | 247 | public function getAllCollection( |
236 | 248 | PFUser $user, |
237 | 249 | Planning_Milestone $milestone, |
@@ -471,6 +483,9 @@ discard block |
||
471 | 483 | } |
472 | 484 | } |
473 | 485 | |
486 | + /** |
|
487 | + * @param integer $artifact_id |
|
488 | + */ |
|
474 | 489 | private function pushItemInDoneCollection(Planning_Milestone $milestone, array $semantics, $artifact_id, AgileDashboard_Milestone_Backlog_IBacklogItem $backlog_item) { |
475 | 490 | $this->setInitialEffort($backlog_item, $semantics[$artifact_id]); |
476 | 491 |
@@ -24,12 +24,21 @@ |
||
24 | 24 | |
25 | 25 | interface AgileDashboard_Milestone_Backlog_BacklogRowPresenter { |
26 | 26 | |
27 | + /** |
|
28 | + * @return string |
|
29 | + */ |
|
27 | 30 | public function title(); |
28 | 31 | |
29 | 32 | public function url(); |
30 | 33 | |
34 | + /** |
|
35 | + * @return integer |
|
36 | + */ |
|
31 | 37 | public function points(); |
32 | 38 | |
39 | + /** |
|
40 | + * @return string|null |
|
41 | + */ |
|
33 | 42 | public function parent_title(); |
34 | 43 | |
35 | 44 | public function parent_url(); |
@@ -160,6 +160,9 @@ |
||
160 | 160 | return $trackers_without_initial_effort_defined; |
161 | 161 | } |
162 | 162 | |
163 | + /** |
|
164 | + * @param string $redirect_to_self |
|
165 | + */ |
|
163 | 166 | public function getPresenter( |
164 | 167 | PFUser $user, |
165 | 168 | Planning_ArtifactMilestone $milestone, |
@@ -275,6 +275,7 @@ discard block |
||
275 | 275 | * We only want the section of the tree that goes from a |
276 | 276 | * tracker in a milestone to a tracker in a submilestone. |
277 | 277 | * |
278 | + * @param integer[] $submilestone_tracker_hierarchy_tree |
|
278 | 279 | */ |
279 | 280 | private function extractRelevantSubmilestoneBacklogTrackerHierarchyTreeSection(&$submilestone_tracker_hierarchy_tree) { |
280 | 281 | foreach ($submilestone_tracker_hierarchy_tree as $key => $value) { |
@@ -299,6 +300,7 @@ discard block |
||
299 | 300 | |
300 | 301 | /** |
301 | 302 | * get all children of a given type of any of the parent items |
303 | + * @param string $ids_of_parent_items_to_search_in |
|
302 | 304 | */ |
303 | 305 | private function getIdsOfChildItemsOfOneTrackerType($ids_of_parent_items_to_search_in, $tracker_id) { |
304 | 306 | $search = $this->artifact_dao->getLinkedArtifactsOfArtifactsOfTrackerTypeAsString( |
@@ -309,6 +311,9 @@ discard block |
||
309 | 311 | return (string) $search['artifact_ids']; |
310 | 312 | } |
311 | 313 | |
314 | + /** |
|
315 | + * @param string $children |
|
316 | + */ |
|
312 | 317 | private function canChildrenBePlanned($children, $tracker_id) { |
313 | 318 | return $children && in_array($tracker_id, $this->descendant_tracker_ids); |
314 | 319 | } |
@@ -78,6 +78,9 @@ discard block |
||
78 | 78 | return $this->retrieve($sql); |
79 | 79 | } |
80 | 80 | |
81 | + /** |
|
82 | + * @param integer $milestone_tracker_id |
|
83 | + */ |
|
81 | 84 | public function searchPaginatedTopMilestones( |
82 | 85 | $milestone_tracker_id, |
83 | 86 | Tuleap\AgileDashboard\Milestone\Criterion\ISearchOnStatus $criterion, |
@@ -157,6 +160,9 @@ discard block |
||
157 | 160 | return array($from_status_statement, $where_status_statement); |
158 | 161 | } |
159 | 162 | |
163 | + /** |
|
164 | + * @param integer $milestone_artifact_id |
|
165 | + */ |
|
160 | 166 | public function searchSubMilestones($milestone_artifact_id) { |
161 | 167 | $limit = null; |
162 | 168 | $offset = null; |
@@ -100,6 +100,9 @@ |
||
100 | 100 | return $options; |
101 | 101 | } |
102 | 102 | |
103 | + /** |
|
104 | + * @param string $content |
|
105 | + */ |
|
103 | 106 | private function getOptionForSelectBox($selected_milestone_id, $milestone_id, $content) { |
104 | 107 | $selected = ''; |
105 | 108 |
@@ -69,6 +69,11 @@ |
||
69 | 69 | return $milestone_representation; |
70 | 70 | } |
71 | 71 | |
72 | + /** |
|
73 | + * @param integer $limit |
|
74 | + * @param integer $offset |
|
75 | + * @param string $order |
|
76 | + */ |
|
72 | 77 | public function getPaginatedSubMilestonesRepresentations( |
73 | 78 | Planning_Milestone $milestone, |
74 | 79 | PFUser $user, |
@@ -41,6 +41,7 @@ |
||
41 | 41 | * Tracker_Artifact::STATUS_CLOSED => no_of_closed, |
42 | 42 | * ) |
43 | 43 | * |
44 | + * @param integer|null $milestone_artifact_id |
|
44 | 45 | * @return array |
45 | 46 | */ |
46 | 47 | public function getStatus(PFUser $user, $milestone_artifact_id) { |
@@ -24,6 +24,9 @@ |
||
24 | 24 | /** @var string */ |
25 | 25 | private $theme_path; |
26 | 26 | |
27 | + /** |
|
28 | + * @param string $theme_path |
|
29 | + */ |
|
27 | 30 | public function __construct(Planning_Milestone $milestone, $theme_path) { |
28 | 31 | parent::__construct($milestone); |
29 | 32 | $this->theme_path = $theme_path; |