Enalean / tuleap

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

Doc Comments +4 added lines

@@ -69,6 +69,9 @@ 
     /** @var AgileDashboard_HierarchyChecker */
     private $hierarchy_checker;
 
+    /**
+     * @param string $plugin_path
+     */
     public function __construct(
         Codendi_Request $request,
         PlanningFactory $planning_factory,
@@ -196,6 +199,7 @@ 
     }
 
     /**
+     * @param Planning[] $plannings
      * @return Planning_Presenter_MilestoneAccessPresenter
      */
     private function getMilestoneAccessPresenters($plannings) {

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

Doc Comments +18 added lines

@@ -28,6 +28,9 @@ 
         return new TrackerDao();
     }
     
+    /**
+     * @param integer $group_id
+     */
     function createPlanning($group_id, PlanningParameters $planning_parameters) {
         $planning_name       = $this->da->quoteSmart($planning_parameters->name);
         $backlog_title       = $this->da->quoteSmart($planning_parameters->backlog_title);
@@ -56,6 +59,9 @@ 
         $this->update($sql);
     }
     
+    /**
+     * @param integer $group_id
+     */
     function searchPlannings($group_id){
         $group_id = $this->da->escapeInt($group_id);
         $sql = "SELECT * 
@@ -90,6 +96,9 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $backlog_tracker_id
+     */
     public function searchByBacklogTrackerId($backlog_tracker_id) {
         $backlog_tracker_id = $this->da->escapeInt($backlog_tracker_id);
         $sql = "
@@ -153,6 +162,9 @@ 
         return $ids;
     }
     
+    /**
+     * @param integer $group_id
+     */
     public function searchNonPlanningTrackersByGroupId($group_id) {
         $planning_tracker_ids = $this->searchPlanningTrackerIdsByGroupId($group_id);
         $tracker_dao          = $this->getTrackerDao();
@@ -160,6 +172,9 @@ 
         return $tracker_dao->searchByGroupIdWithExcludedIds($group_id, $planning_tracker_ids);
     }
     
+    /**
+     * @param integer $planning_id
+     */
     function updatePlanning($planning_id, PlanningParameters $planning_parameters) {
         $planning_id         = $this->da->escapeInt($planning_id);
         $planning_name       = $this->da->quoteSmart($planning_parameters->name);
@@ -178,6 +193,9 @@ 
         $this->updateBacklogTrackers($planning_id, $planning_parameters);
     }
 
+    /**
+     * @param string $planning_id
+     */
     private function updateBacklogTrackers($planning_id, PlanningParameters $planning_parameters) {
         $this->deletePlanningBacklogTrackers($planning_id);
         $this->createBacklogTrackers($planning_id, $planning_parameters);

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

Doc Comments +7 added lines, -5 removed lines

@@ -86,6 +86,9 @@ 
         }
     }
 
+    /**
+     * @param integer $group_id
+     */
     protected function duplicatePriorityChangePermission($group_id, $source_planning_id, $new_planning_id, array $ugroups_mapping) {
         $source_planning                       = $this->getPlanning($source_planning_id);
         $priority_change_permission_ugroup_ids = $this->planning_permissions_manager->getGroupIdsWhoHasPermissionOnPlanning(
@@ -284,7 +287,6 @@ 
      *
      * @param PFUser $user     The user who will see the planning
      * @param int  $group_id
-     * @param PlanningFactory $planning_factory
      *
      * @return Planning[]
      */
@@ -338,7 +340,6 @@ 
      *
      * @param PFUser $user     The user who will see the planning
      * @param int  $group_id
-     * @param PlanningFactory $planning_factory
      *
      * @return Planning[]
      */
@@ -378,7 +379,6 @@ 
     /**
      * Get a planning
      *
-     * @param int $group_id
      *
      * @return Planning
      */
@@ -548,6 +548,9 @@ 
         return $this->dao->searchPlanningTrackerIdsByGroupId($group_id);
     }
 
+    /**
+     * @param integer $group_id
+     */
     public function getBacklogTrackerIdsByGroupId($group_id) {
         return $this->dao->searchBacklogTrackerIdsByGroupId($group_id);
     }
@@ -592,7 +595,7 @@ 
      *
      * @param $planning_id the id of the planning
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deletePlanning($planning_id) {
         return $this->dao->deletePlanning($planning_id);
@@ -617,7 +620,6 @@ 
     /**
      * Retrieve the project trackers that can be used as planning trackers.
      *
-     * @param Planning $planning The planning for which we want to know the available trackers.
      *
      * @return Array of Tracker
      */

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/Presenters/PHP51HomePresenter.class.php

Doc Comments +3 added lines

@@ -29,6 +29,9 @@
     /** @var Planning_ShortAccess[] */
     private $short_access;
 
+    /**
+     * @param string $plugin_theme_path
+     */
     public function __construct(array $short_access, $plugin_theme_path, $project_id) {
         $this->short_access      = $short_access;
         $this->plugin_theme_path = $plugin_theme_path;

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
      */