Enalean / tuleap

plugins/agiledashboard/include/Planning/PlanningPermissionsManager.class.php

Doc Comments +5 added lines

@@ -22,6 +22,11 @@
 
     const PERM_PRIORITY_CHANGE = 'PLUGIN_AGILEDASHBOARD_PLANNING_PRIORITY_CHANGE';
 
+    /**
+     * @param integer $planning_id
+     * @param integer $group_id
+     * @param string $html_element_name
+     */
     public function getPlanningPermissionForm($planning_id, $group_id, $permission, $html_element_name) {
         return permission_fetch_selection_field($permission, $planning_id, $group_id, $html_element_name);
     }

plugins/agiledashboard/include/Planning/Presenters/LastLevelMilestone.class.php

Doc Comments +4 added lines

@@ -29,6 +29,10 @@
     /** @var string */
     public $milestone_type_name;
 
+    /**
+     * @param Planning_Presenter_MilestoneSummaryPresenter[] $milestone_summary_presenters
+     * @param string $tracker_name
+     */
     public function __construct($milestone_summary_presenters, $tracker_name) {
         $this->milestone_summary_presenters = $milestone_summary_presenters;
         $this->milestone_type_name          = $tracker_name;

plugins/agiledashboard/include/Planning/Presenters/MilestoneAccessPresenter.class.php

Doc Comments +3 added lines

@@ -35,6 +35,9 @@
      */
     public $milestone_type;
 
+    /**
+     * @param Planning_Milestone[] $milestones
+     */
     public function __construct($milestones, $milestone_type) {
         $this->milestones     = $milestones;
         $this->milestone_type = $milestone_type;

plugins/agiledashboard/include/Planning/Presenters/MilestoneSummaryPresenter.class.php

Doc Comments +4 added lines

@@ -31,6 +31,10 @@
      */
     private $status_count;
 
+    /**
+     * @param string $plugin_path
+     * @param boolean $has_cardwall
+     */
     public function __construct(Planning_Milestone $milestone, $plugin_path, $has_cardwall, $status_count) {
         parent::__construct($milestone, $plugin_path, $has_cardwall);
 

plugins/agiledashboard/include/Planning/ShortAccessFactory.class.php

Doc Comments +1 added lines

@@ -46,6 +46,7 @@
      *
      * @param PFUser $user     The user who will see the planning
      * @param int  $group_id
+     * @param string $theme_path
      *
      * @return array of Planning_ShortAccess
      */

plugins/agiledashboard/include/Planning/ShortAccessMilestonePresenter.class.php

Doc Comments +3 added lines

@@ -40,6 +40,9 @@
     /** @var AgileDashboard_PaneIconLinkPresenter[] */
     private $pane_icon_list;
 
+    /**
+     * @param string $theme_path
+     */
     public function __construct(
         Planning_ShortAccess $short_access,
         Planning_Milestone $milestone,

plugins/agiledashboard/include/Planning/VirtualTopMilestoneController.class.php

Doc Comments -1 removed lines

@@ -43,7 +43,6 @@
      *   - pass $request to actions (e.g. show).
      *
      * @param Codendi_Request           $request
-     * @param PlanningFactory           $planning_factory
      * @param Planning_MilestoneFactory $milestone_factory
      */
     public function __construct(

plugins/agiledashboard/include/Planning/VirtualTopMilestonePaneFactory.class.php

Doc Comments +4 added lines, -5 removed lines

@@ -89,8 +89,7 @@ 
     }
 
     /**
-     * @param Planning_Milestone $milestone
-     * @return int
+     * @return string
      */
     private function getMilestoneArtifactId() {
          return self::TOP_MILESTONE_DUMMY_ARTIFACT_ID;
@@ -107,7 +106,7 @@ 
         return $this->active_pane[$milestone_artifact_id];
     }
 
-    /** @return AgileDashboard_PaneInfo[] */
+    /** @return AgileDashboard_PaneInfo */
     public function getListOfPaneInfo(Planning_Milestone $milestone) {
         $milestone_artifact_id = $this->getMilestoneArtifactId();
 
@@ -150,7 +149,7 @@ 
     }
 
     /**
-     * @return \AgileDashboard_Milestone_Pane_Planning_PlanningPaneInfo
+     * @return null|AgileDashboard_Milestone_Pane_TopPlanning_TopPlanningPaneInfo
      */
     private function getTopPlanningPaneInfo(Planning_Milestone $milestone) {
         $milestone_artifact_id = $this->getMilestoneArtifactId();
@@ -174,7 +173,7 @@ 
     }
 
     /**
-     * @return \AgileDashboard_Milestone_Pane_Planning_PlanningPaneInfo
+     * @return null|AgileDashboard_Milestone_Pane_TopPlanning_TopPlanningV2PaneInfo
      */
     private function getTopPlanningV2PaneInfo(Planning_Milestone $milestone) {
         $milestone_artifact_id = $this->getMilestoneArtifactId();

plugins/agiledashboard/tests/builders/aMilestone.php

Doc Comments +3 added lines

@@ -70,6 +70,9 @@
         return $this;
     }
     
+    /**
+     * @param integer $planning_id
+     */
     public function withPlanningId($planning_id) {
         $this->withPlanning(aPlanning()->withId($planning_id)->build());
         return $this;