Enalean / tuleap

plugins/agiledashboard/include/AgileDashboard/REST/v1/BacklogItemResource.php

Doc Comments +12 added lines, -1 removed lines

@@ -102,7 +102,7 @@ 
      *
      * @param int $id Id of the Backlog Item
      *
-     * @return array {@type Tuleap\AgileDashboard\REST\v1\BacklogItemRepresentation}
+     * @return BacklogItemRepresentation {@type Tuleap\AgileDashboard\REST\v1\BacklogItemRepresentation}
      *
      * @throws 403
      * @throws 404
@@ -317,6 +317,9 @@ 
         }
     }
 
+    /**
+     * @param integer $id
+     */
     private function getArtifact($id) {
         $artifact     = $this->artifact_factory->getArtifactById($id);
         $current_user = $this->getCurrentUser();
@@ -353,6 +356,9 @@ 
         return new Tracker_SlicedArtifactsBuilder(new Tracker_ArtifactDao());
     }
 
+    /**
+     * @param integer $limit
+     */
     private function checkContentLimit($limit) {
         if (! $this->limitValueIsAcceptable($limit)) {
             throw new RestException(406, 'Maximum value for limit exceeded');
@@ -371,6 +377,11 @@ 
         Header::allowOptionsGetPatch();
     }
 
+    /**
+     * @param integer $limit
+     * @param integer $offset
+     * @param integer $size
+     */
     private function sendPaginationHeaders($limit, $offset, $size) {
         Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT);
     }

Unused Use Statements -1 removed lines

@@ -36,7 +36,6 @@
 use TrackerFactory;
 use Tracker_SemanticManager;
 use Tracker_SemanticCollection;
-use Tracker_FormElementFactory;
 use Tracker_Semantic_Title;
 use Tracker_Semantic_Status;
 use AgileDashBoard_Semantic_InitialEffort;

plugins/agiledashboard/include/AgileDashboard/REST/v1/Kanban/KanbanColumnsResource.php

Doc Comments +2 added lines, -2 removed lines

@@ -112,7 +112,6 @@ 
      *
      * @param int                             $id        Id of the column
      * @param int                             $kanban_id Id of the Kanban {@from query}
-     * @param KanbanColumnPATCHRepresentation $column    The kanban column {@from body} {@type Tuleap\AgileDashboard\REST\v1\Kanban\KanbanColumnPATCHRepresentation}
      *
      * @throws 401
      * @throws 404
@@ -182,7 +181,8 @@ 
         }
     }
 
-    /** @return AgileDashboard_Kanban */
+    /** @param integer $id
+/** @return AgileDashboard_Kanban */
     private function getKanban(PFUser $user, $id) {
         try {
             $kanban = $this->kanban_factory->getKanban($user, $id);

plugins/agiledashboard/include/AgileDashboard/REST/v1/Kanban/KanbanItemCollectionRepresentation.php

Doc Comments +5 added lines

@@ -32,6 +32,11 @@
     /** @var int */
     public $total_size;
 
+    /**
+     * @param integer $column_id
+     * @param integer $limit
+     * @param integer $offset
+     */
     public function build(PFUser $user, AgileDashboard_Kanban $kanban, $column_id, $limit, $offset) {
         $dao             = new AgileDashboard_KanbanItemDao();
         $factory         = Tracker_ArtifactFactory::instance();

plugins/agiledashboard/include/AgileDashboard/REST/v1/Kanban/KanbanItemsResource.php

Doc Comments +3 added lines, -2 removed lines

@@ -229,7 +229,7 @@ 
      * @access hybrid
      *
      * @param  int  $id     Id of the artifact
-     * @return Tuleap\AgileDashboard\REST\v1\Kanban\KanbanRepresentation
+     * @return KanbanItemRepresentation
      */
     protected function get($id) {
         $this->checkAccess();
@@ -325,7 +325,8 @@ 
 
     }
 
-    /** @return AgileDashboard_Kanban */
+    /** @param integer $id
+/** @return AgileDashboard_Kanban */
     private function getKanban(PFUser $user, $id) {
         try {
             $kanban = $this->kanban_factory->getKanban($user, $id);

Unused Use Statements +4 added lines, -4 removed lines

@@ -42,10 +42,10 @@
 use PFUser;
 use Tracker_FormElement_Field_List;
 use Tracker_Semantic_Status;
-use Tracker_REST_Artifact_ArtifactCreator                 as ArtifactCreator;
-use Tracker_REST_Artifact_ArtifactValidator               as ArtifactValidator;
-use Tuleap\Tracker\REST\TrackerReference                  as TrackerReference;
-use Tuleap\Tracker\REST\v1\ArtifactValuesRepresentation   as ArtifactValuesRepresentation;
+use Tracker_REST_Artifact_ArtifactCreator as ArtifactCreator;
+use Tracker_REST_Artifact_ArtifactValidator as ArtifactValidator;
+use Tuleap\Tracker\REST\TrackerReference as TrackerReference;
+use Tuleap\Tracker\REST\v1\ArtifactValuesRepresentation as ArtifactValuesRepresentation;
 use AgileDashboard_KanbanUserPreferences;
 use AgileDashboard_KanbanItemManager;
 use Tuleap\RealTime\NodeJSClient;

plugins/agiledashboard/include/AgileDashboard/REST/v1/Kanban/KanbanRepresentation.php

Doc Comments +5 added lines

@@ -85,6 +85,11 @@
      */
     public $user_can_reorder_columns;
 
+    /**
+     * @param boolean $user_can_add_columns
+     * @param boolean $user_can_reorder_columns
+     * @param boolean $user_can_add_in_place
+     */
     public function build(
         AgileDashboard_Kanban $kanban,
         AgileDashboard_KanbanColumnFactory $column_factory,

plugins/agiledashboard/include/AgileDashboard/REST/v1/Kanban/KanbanRepresentationBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     }
 
     /**
-     * @return Tuleap\AgileDashboard\REST\v1\Kanban\KanbanRepresentation
+     * @return KanbanRepresentation
      */
     public function build(AgileDashboard_Kanban $kanban, PFUser $user) {
 

plugins/agiledashboard/include/AgileDashboard/REST/v1/MilestoneRepresentation.class.php

Unused Use Statements -2 removed lines

@@ -26,9 +26,7 @@
 use Tuleap\Tracker\REST\Artifact\BurndownRepresentation;
 use Tuleap\REST\JsonCast;
 use PlanningFactory;
-
 use AgileDashboard_MilestonesCardwallRepresentation;
-use Tuleap\Tracker\REST\TrackerRepresentation;
 
 /**
  * Representation of a milestone

plugins/agiledashboard/include/AgileDashboard/REST/v1/MilestoneResource.class.php

Doc Comments +21 added lines, -1 removed lines

@@ -251,7 +251,7 @@ 
      *
      * @param int $id Id of the milestone
      *
-     * @return Tuleap\AgileDashboard\REST\v1\MilestoneRepresentation
+     * @return MilestoneRepresentation
      *
      * @throws 403
      * @throws 404
@@ -800,6 +800,9 @@ 
         $this->sendAllowHeaderForBacklog();
     }
 
+    /**
+     * @param PFUser $user
+     */
     private function getBacklogItemAsArtifact($user, $artifact_id) {
         $artifact = $this->tracker_artifact_factory->getArtifactById($artifact_id);
 
@@ -894,6 +897,9 @@ 
         return $burndown;
     }
 
+    /**
+     * @param integer $id
+     */
     private function getMilestoneById(PFUser $user, $id) {
         try {
             $milestone = $this->milestone_factory->getValidatedBareMilestoneByArtifactId($user, $id);
@@ -917,6 +923,10 @@ 
         return UserManager::instance()->getCurrentUser();
     }
 
+    /**
+     * @param \Planning_ArtifactMilestone $milestone
+     * @param \AgileDashboard_Milestone_Backlog_DescendantBacklogStrategy $strategy
+     */
     private function getMilestoneContentItems($milestone, $strategy) {
         return $this->backlog_item_collection_factory->getAllCollection(
             $this->getCurrentUser(),
@@ -926,6 +936,9 @@ 
         );
     }
 
+    /**
+     * @param integer $limit
+     */
     private function checkContentLimit($limit) {
         if (! $this->limitValueIsAcceptable($limit)) {
              throw new RestException(406, 'Maximum value for limit exceeded');
@@ -940,6 +953,10 @@ 
         Header::allowOptionsGetPutPatch();
     }
 
+    /**
+     * @param integer $limit
+     * @param integer $offset
+     */
     private function sendPaginationHeaders($limit, $offset, $size) {
         Header::sendPaginationHeaders($limit, $offset, $size, self::MAX_LIMIT);
     }
@@ -952,6 +969,9 @@ 
         Header::allowOptionsGetPut();
     }
 
+    /**
+     * @param \Planning_ArtifactMilestone $milestone
+     */
     private function sendAllowHeadersForMilestone($milestone) {
         $date = $milestone->getLastModifiedDate();
         Header::allowOptionsGet();

plugins/agiledashboard/include/AgileDashboard/REST/v1/MilestoneResourceValidator.class.php

Doc Comments +6 added lines

@@ -96,6 +96,9 @@ 
         return $this->validateArtifactsForMilestoneContent($ids, $milestone, $user, true);
     }
 
+    /**
+     * @param boolean $accept_closed
+     */
     private function validateArtifactsForMilestoneContent(array $ids, Planning_Milestone $milestone, PFUser $user, $accept_closed) {
         $open_unplanned = null;
         $todo           = null;
@@ -274,6 +277,9 @@ 
         return $index;
     }
 
+    /**
+     * @param Planning_Milestone $milestone
+     */
     private function getMilestoneBacklogItems(PFUser $user, $milestone) {
         return $this->backlog_item_collection_factory->getUnplannedOpenCollection(
             $user,