@@ -24,12 +24,24 @@ discard block |
||
24 | 24 | */ |
25 | 25 | interface Cardwall_OnTop_IConfig { |
26 | 26 | |
27 | + /** |
|
28 | + * @return Tracker|null |
|
29 | + */ |
|
27 | 30 | public function getTracker(); |
28 | 31 | |
32 | + /** |
|
33 | + * @return boolean |
|
34 | + */ |
|
29 | 35 | public function isEnabled(); |
30 | 36 | |
37 | + /** |
|
38 | + * @return boolean |
|
39 | + */ |
|
31 | 40 | public function enable(); |
32 | 41 | |
42 | + /** |
|
43 | + * @return boolean |
|
44 | + */ |
|
33 | 45 | public function disable(); |
34 | 46 | |
35 | 47 | public function getDashboardColumns(); |
@@ -40,14 +52,23 @@ discard block |
||
40 | 52 | |
41 | 53 | public function getTrackers(); |
42 | 54 | |
55 | + /** |
|
56 | + * @return Cardwall_OnTop_Config_TrackerMapping|null |
|
57 | + */ |
|
43 | 58 | public function getMappingFor(Tracker $mapping_tracker); |
44 | 59 | |
60 | + /** |
|
61 | + * @return boolean |
|
62 | + */ |
|
45 | 63 | public function isInColumn(Tracker_Artifact $artifact, |
46 | 64 | Cardwall_FieldProviders_IProvideFieldGivenAnArtifact $field_provider, |
47 | 65 | Cardwall_Column $column); |
48 | 66 | |
49 | 67 | public function getCardwallMappings(array $fields, Cardwall_OnTop_Config_ColumnCollection $cardwall_columns); |
50 | 68 | |
69 | + /** |
|
70 | + * @return void |
|
71 | + */ |
|
51 | 72 | public function fillMappingsWithOnTopMappings(Cardwall_MappingCollection $mappings, |
52 | 73 | Cardwall_OnTop_Config_ColumnCollection $columns); |
53 | 74 | } |
@@ -143,6 +143,9 @@ |
||
143 | 143 | return $this->returnRelevantProgressBarValue($completion); |
144 | 144 | } |
145 | 145 | |
146 | + /** |
|
147 | + * @param double $value |
|
148 | + */ |
|
146 | 149 | private function returnRelevantProgressBarValue($value) { |
147 | 150 | if ($value < 0) { |
148 | 151 | return 0; |
@@ -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) { |