Enalean / tuleap

plugins/cardwall/include/Cardwall_Renderer.class.php

Doc Comments +3 added lines

@@ -101,6 +101,9 @@
         return $this->fetchCards($matching_ids, $user, $form);
     }
     
+    /**
+     * @param Cardwall_Form $form
+     */
     private function fetchCards($matching_ids, PFUser $user, $form = null) {
         $total_rows = $matching_ids['id'] ? substr_count($matching_ids['id'], ',') + 1 : 0;
         if (!$total_rows) {

plugins/cardwall/include/Cardwall_RendererDao.class.php

Doc Comments +3 added lines

@@ -44,6 +44,9 @@
         return $this->update($sql);
     }
     
+    /**
+     * @param integer $renderer_id
+     */
     function save($renderer_id, $field_id) {
         $renderer_id = $this->da->escapeInt($renderer_id);
         $field_id     = $this->da->escapeInt($field_id);

plugins/cardwall/include/CardwallPluginInfo.class.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
  */
 class CardwallPluginInfo extends PluginFileInfo {
     
+    /**
+     * @param cardwallPlugin $plugin
+     */
     function __construct($plugin) {
         parent::__construct($plugin, 'config');
         $this->setPluginDescriptor(new CardwallPluginDescriptor());

plugins/cardwall/include/Column.class.php

Doc Comments +6 added lines

@@ -64,6 +64,9 @@ 
         $this->fgcolor   = $fgcolor;
     }
 
+    /**
+     * @param boolean|string $value
+     */
     public function setAutostack($value) {
         $this->autostack = $value;
         return $this;
@@ -108,6 +111,9 @@ 
         return $is_mapped || $this->matchesStatus($artifact_status);
     }
 
+    /**
+     * @param string $artifact_status
+     */
     private function matchesStatus($artifact_status) {
         return $this->matchesLabel($artifact_status) || $this->matchesTheNoneColumn($artifact_status);
     }

plugins/cardwall/include/Form.class.php

Doc Comments +4 added lines

@@ -63,6 +63,10 @@
      */
     public $submit;
 
+    /**
+     * @param integer $renderer_id
+     * @param Tracker_FormElement_Field|null $field
+     */
     public function __construct($report_id, $renderer_id, $printer_version, $field, array $selectboxes) {
         $this->report_id           = (int)$report_id;
         $this->renderer_id         = (int)$renderer_id;

plugins/cardwall/include/MappingCollection.class.php

Doc Comments +2 added lines, -1 removed lines

@@ -34,7 +34,7 @@ 
     private $mappings_by_field_id = array();
 
     /**
-     * @return Iterator
+     * @return ArrayObject
      */
     public function getIterator() {
         return new ArrayObject($this->mappings);
@@ -61,6 +61,7 @@ 
     }
 
     /**
+     * @param integer $field_id
      * @return array of Cardwall_Mapping
      */
     private function getMappingsByFieldId($field_id) {

plugins/cardwall/include/OnTop/ColumnMappingFieldDao.class.php

Doc Comments +19 added lines

@@ -21,6 +21,9 @@ 
 
 class Cardwall_OnTop_ColumnMappingFieldDao extends DataAccessObject {
 
+    /**
+     * @param integer $cardwall_tracker_id
+     */
     public function searchMappingFields($cardwall_tracker_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $sql = "SELECT t1.id AS tracker_id, m.field_id AS field_id
@@ -36,6 +39,10 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     * @param integer $tracker_id
+     */
     public function create($cardwall_tracker_id, $tracker_id, $field_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $tracker_id          = $this->da->escapeInt($tracker_id);
@@ -49,6 +56,11 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     * @param integer $tracker_id
+     * @param integer $field_id
+     */
     public function save($cardwall_tracker_id, $tracker_id, $field_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $tracker_id          = $this->da->escapeInt($tracker_id);
@@ -64,6 +76,10 @@ 
         }
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     * @param integer $tracker_id
+     */
     public function delete($cardwall_tracker_id, $tracker_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $tracker_id          = $this->da->escapeInt($tracker_id);
@@ -73,6 +89,9 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     */
     public function deleteCardwall($cardwall_tracker_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $sql = "DELETE FROM plugin_cardwall_on_top_column_mapping_field

plugins/cardwall/include/OnTop/ColumnMappingFieldValueDao.class.php

Doc Comments +27 added lines

@@ -21,6 +21,9 @@ 
 
 class Cardwall_OnTop_ColumnMappingFieldValueDao extends DataAccessObject {
 
+    /**
+     * @param integer $cardwall_tracker_id
+     */
     public function searchMappingFieldValues($cardwall_tracker_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $sql = "SELECT *
@@ -29,6 +32,12 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     * @param integer $tracker_id
+     * @param integer $field_id
+     * @param integer $value_id
+     */
     public function save($cardwall_tracker_id, $tracker_id, $field_id, $value_id, $column_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $tracker_id          = $this->da->escapeInt($tracker_id);
@@ -44,6 +53,9 @@ 
         }
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     */
     public function deleteForColumn($cardwall_tracker_id, $column_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $column_id           = $this->da->escapeInt($column_id);
@@ -53,6 +65,11 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     * @param integer $tracker_id
+     * @param integer $field_id
+     */
     public function deleteAllFieldValues($cardwall_tracker_id, $tracker_id, $field_id, $column_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $tracker_id          = $this->da->escapeInt($tracker_id);
@@ -66,6 +83,10 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $cardwall_tracker_id
+     * @param integer $tracker_id
+     */
     public function delete($cardwall_tracker_id, $tracker_id) {
         $cardwall_tracker_id = $this->da->escapeInt($cardwall_tracker_id);
         $tracker_id          = $this->da->escapeInt($tracker_id);
@@ -108,6 +129,9 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param string $field_name
+     */
     private function associativeToSQLCase(array $mapping, $field_name) {
         $when_then = $this->associativeToSQLWhenThen($mapping);
         return $this->getSQLCase($field_name, $when_then);
@@ -123,6 +147,9 @@ 
         return $stmt;
     }
 
+    /**
+     * @param string $when_then
+     */
     private function getSQLCase($field_name, $when_then) {
         $stmt = " CASE $field_name
                   $when_then

plugins/cardwall/include/OnTop/Config.class.php

Doc Comments -1 removed lines

@@ -84,7 +84,6 @@
     /**
      * Get Frestyle columns for Cardwall_OnTop, or status columns if none
      *
-     * @param Tracker $tracker
      * @return Cardwall_OnTop_Config_ColumnCollection
      */
     public function getDashboardColumns() {