@@ -96,6 +96,9 @@ |
||
96 | 96 | ); |
97 | 97 | } |
98 | 98 | |
99 | + /** |
|
100 | + * @param Cardwall_FieldProviders_IProvideFieldGivenAnArtifact $field_provider |
|
101 | + */ |
|
99 | 102 | private function getIndexedStatusFieldsOf(array $trackers, $field_provider) { |
100 | 103 | $status_fields = array(); |
101 | 104 |
@@ -39,7 +39,7 @@ |
||
39 | 39 | * |
40 | 40 | * @param array $artifact_ids |
41 | 41 | * @param Cardwall_OnTop_Config_ColumnCollection $columns |
42 | - * @param Cardwall_MappingCollection $mappings_collection |
|
42 | + * @param Cardwall_MappingCollection $mapping_collection |
|
43 | 43 | * @return \Cardwall_Board |
44 | 44 | */ |
45 | 45 | public function getBoard(array $artifact_ids, Cardwall_OnTop_Config_ColumnCollection $columns, Cardwall_MappingCollection $mapping_collection) { |
@@ -50,9 +50,8 @@ |
||
50 | 50 | * |
51 | 51 | * @url GET {id}/cardwall |
52 | 52 | * |
53 | - * @param int $id Id of the milestone |
|
54 | 53 | * |
55 | - * @return \Tuleap\Cardwall\REST\v1\MilestonesCardwallRepresentation |
|
54 | + * @return \AgileDashboard_MilestonesCardwallRepresentation |
|
56 | 55 | * |
57 | 56 | * @throws 403 |
58 | 57 | * @throws 404 |
@@ -45,6 +45,9 @@ discard block |
||
45 | 45 | } |
46 | 46 | |
47 | 47 | /** |
48 | + * @param integer $tracker_id |
|
49 | + * @param integer $field_id |
|
50 | + * @param string $rank |
|
48 | 51 | * @return boolean true if success |
49 | 52 | */ |
50 | 53 | public function add($tracker_id, $field_id, $rank) { |
@@ -59,6 +62,8 @@ discard block |
||
59 | 62 | |
60 | 63 | |
61 | 64 | /** |
65 | + * @param integer $tracker_id |
|
66 | + * @param integer $field_id |
|
62 | 67 | * @return boolean true if success |
63 | 68 | */ |
64 | 69 | public function remove($tracker_id, $field_id) { |
@@ -51,9 +51,8 @@ discard block |
||
51 | 51 | /** |
52 | 52 | * Return a new card controller |
53 | 53 | * |
54 | - * @param Codendi_Request $request |
|
55 | 54 | * |
56 | - * @return Cardwall_CardController |
|
55 | + * @return Cardwall_SingleCard |
|
57 | 56 | * |
58 | 57 | * @throws Exception |
59 | 58 | */ |
@@ -78,6 +77,7 @@ discard block |
||
78 | 77 | } |
79 | 78 | |
80 | 79 | /** |
80 | + * @param Cardwall_CardInCellPresenterFactory $presenter_factory |
|
81 | 81 | * @return Cardwall_CardInCellPresenter |
82 | 82 | */ |
83 | 83 | protected function getCardInCellPresenter($presenter_factory, PFUser $user, Tracker_Artifact $artifact, Cardwall_CardFields $card_fields, Cardwall_UserPreferences_UserPreferencesDisplayUser $display_preferences) { |
@@ -24,8 +24,8 @@ |
||
24 | 24 | class Cardwall_SwimlineSolo extends Cardwall_Swimline{ |
25 | 25 | |
26 | 26 | /** |
27 | - * @param string $title |
|
28 | 27 | * @param array $cells |
28 | + * @param integer $swimline_id |
|
29 | 29 | */ |
30 | 30 | public function __construct($swimline_id, array $cells) { |
31 | 31 | $this->swimline_id = $swimline_id; |
@@ -30,7 +30,6 @@ |
||
30 | 30 | public $swimline_id = self::FAKE_SWIMLINE_ID_FOR_TRACKER_RENDERER; |
31 | 31 | |
32 | 32 | /** |
33 | - * @param string $title |
|
34 | 33 | * @param array $cells |
35 | 34 | */ |
36 | 35 | public function __construct(array $cells) { |
@@ -74,14 +74,24 @@ |
||
74 | 74 | $this->assertNotIn($column); |
75 | 75 | } |
76 | 76 | |
77 | + /** |
|
78 | + * @param Cardwall_Column $column |
|
79 | + */ |
|
77 | 80 | private function assertIn($column) { |
78 | 81 | $this->assertTrue($this->config->isInColumn($this->artifact, $this->field_provider, $column)); |
79 | 82 | } |
80 | 83 | |
84 | + /** |
|
85 | + * @param Cardwall_Column $column |
|
86 | + */ |
|
81 | 87 | private function assertNotIn($column) { |
82 | 88 | $this->assertFalse($this->config->isInColumn($this->artifact, $this->field_provider, $column)); |
83 | 89 | } |
84 | 90 | |
91 | + /** |
|
92 | + * @param integer $id |
|
93 | + * @param string $label |
|
94 | + */ |
|
85 | 95 | public function newCardwall_Column($id, $label) { |
86 | 96 | $bgcolor = $fgcolor = 0; |
87 | 97 | return new Cardwall_Column($id, $label, $bgcolor, $fgcolor); |
@@ -117,6 +117,9 @@ discard block |
||
117 | 117 | } |
118 | 118 | } |
119 | 119 | |
120 | + /** |
|
121 | + * @param DOMElement $mdNode |
|
122 | + */ |
|
120 | 123 | protected function getNodeForMetadataValues($mdValues, $mdNode) { |
121 | 124 | foreach($mdValues as $val) { |
122 | 125 | if($val->getId() != 100) { |
@@ -223,6 +226,9 @@ discard block |
||
223 | 226 | return $this->userCache[$userId]; |
224 | 227 | } |
225 | 228 | |
229 | + /** |
|
230 | + * @param integer $statusId |
|
231 | + */ |
|
226 | 232 | protected function getNormalizedStatus($statusId) { |
227 | 233 | switch($statusId) { |
228 | 234 | case PLUGIN_DOCMAN_ITEM_STATUS_NONE: |
@@ -242,6 +248,9 @@ discard block |
||
242 | 248 | } |
243 | 249 | } |
244 | 250 | |
251 | + /** |
|
252 | + * @param string $label |
|
253 | + */ |
|
245 | 254 | protected function appendChild(DOMElement $node, $label, $value) { |
246 | 255 | if($value != '') { |
247 | 256 | $subNode = $this->doc->createElement($label); |