Enalean / tuleap

plugins/docman/include/ApprovalTable/Reviewer/ReviewerDao.class.php

Doc Comments +11 added lines

@@ -76,6 +76,9 @@ 
         return parent::prepareRanking($userId, $tableId, $rank, 'reviewer_id', 'table_id');
     }
 
+    /**
+     * @param integer $userId
+     */
     function addUser($tableId, $userId) {
         $newRank = $this->prepareUserRanking($tableId, $userId, 'end');
         $sql = sprintf('INSERT INTO plugin_docman_approval_user'.
@@ -146,6 +149,11 @@ 
         return $this->_copyReviewers($srcTableId, $dstTableId, 'NULL', 0, "''", 'NULL');
     }
 
+    /**
+     * @param string $date
+     * @param string $comment
+     * @param string $version
+     */
     function _copyReviewers($srcTableId, $dstTableId, $date, $state, $comment, $version) {
         $sql = 'INSERT INTO plugin_docman_approval_user'.
             '(table_id, reviewer_id, rank, date, state, comment, version) '.
@@ -158,6 +166,9 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $state
+     */
     function getAllReviewsForUserByState($userId, $state) {
         // Item
         $sql_item = 'SELECT u.table_id, i.item_id, i.group_id, t.date, i.title, g.group_name'.

plugins/docman/include/ApprovalTable/Reviewer/ReviewerFactory.class.php

Doc Comments +3 added lines, -1 removed lines

@@ -237,7 +237,7 @@ 
     /**
      * Add members of the given ugroup to the reviewer list.
      *
-     * @return true if at least one user was added to the list.
+     * @return boolean if at least one user was added to the list.
      */
     function addUgroup($ugroupId) {
         $nbUserAdded = 0;
@@ -316,6 +316,7 @@ 
 
     /**
      *
+     * @param integer $newTableId
      */
     function newTableCopy($newTableId) {
         $dao = $this->_getDao();
@@ -324,6 +325,7 @@ 
 
     /**
      *
+     * @param integer $newTableId
      */
     function newTableReset($newTableId) {
         $dao = $this->_getDao();

plugins/docman/include/ApprovalTable/Wiki/WikiDao.class.php

Doc Comments +2 added lines

@@ -105,6 +105,8 @@
 
     /**
      * Did user access the wiki since the given version was published.
+     * @param integer $groupId
+     * @param integer $versionId
      */
     function userAccessedSince($userId, $pageName, $groupId, $versionId) {
         $sql  = 'SELECT NULL'.

plugins/docman/include/ApprovalTable/Wiki/WikiFactory.class.php

Doc Comments +4 added lines

@@ -45,6 +45,7 @@ 
      * If there is no approval table linked to the item, pick-up the last
      * version id of the wiki page.
      * If there is no version for the given wiki page, default to 0.
+     * @param Docman_Wiki $item
      */
     function __construct($item, $versionNumber=null) {
         parent::__construct($item);
@@ -106,6 +107,9 @@ 
         return $this->getTableFromVersion($this->item->getId(), $this->wikiVersionId);
     }
 
+    /**
+     * @param integer $version
+     */
     function getTableFromVersion($itemId, $version) {
         $table = null;
         if($version !== null) {

plugins/docman/include/Docman_Actions.class.php

Doc Comments +9 added lines, -2 removed lines

@@ -76,6 +76,9 @@ 
         return $ret_id;
     }
 
+    /**
+     * @param string $field
+     */
     private function _raiseMetadataChangeEvent(&$user, &$item, $group_id, $old, $new, $field) {
         $logEventParam = array('group_id' => $group_id,
                                'item'     => &$item,
@@ -1173,8 +1176,8 @@ 
     * The force parameter must be set to true if you want to bypass permissions checking (@see permission_add_ugroup).
     * Pretty difficult to know if a user can update the permissions which does not exist for a new item...
     *
-    * @param $group_id integer The id of the project
-    * @param $item_id integer The id of the item
+    * @param integer $group_id integer The id of the project
+    * @param integer $item_id integer The id of the item
     * @param $permission_definition array The definission of the permission (pretty name, relations between perms, internal name, ...)
     * @param $old_permissions array The permissions before
     * @param &$done_permissions array The permissions after
@@ -1862,6 +1865,7 @@ 
 
     /**
      * @access private
+     * @param Docman_ApprovalTableReviewerFactory $atrf
      */
     function _approval_update_add_users($atrf, $usUserList, $sUgroups) {
         $noError = true;
@@ -1929,6 +1933,7 @@ 
 
     /**
      * @access private
+     * @param Docman_ApprovalTableReviewerFactory $atrf
      */
     function _approval_update_del_users($atrf, $selectedUsers) {
         $deletedUsers = 0;
@@ -1947,6 +1952,7 @@ 
 
     /**
      * @access private
+     * @param Docman_ApprovalTableReviewerFactory $atrf
      */
     function _approval_update_notify_users($atrf, $selectedUsers) {
         $notifiedUsers = 0;
@@ -1972,6 +1978,7 @@ 
 
     /**
      * @access private
+     * @param Docman_ApprovalTableReviewerFactory $atrf
      */
     function _approval_update_notif_resend($atrf) {
         $res = $atrf->notifyReviewers();

plugins/docman/include/Docman_ActionsDeleteVisitor.class.php

Doc Comments +3 added lines

@@ -145,6 +145,9 @@
         return $this->visitDocument($item, $params);
     }
 
+    /**
+     * @param Docman_Item $item
+     */
     public function restrictAccess($item, $params = array()) {
         // Check whether there is other references to this wiki page.
         $dao = $this->_getItemDao();

plugins/docman/include/Docman_BuildItemMappingVisitor.class.php

Doc Comments +2 added lines

@@ -164,6 +164,7 @@ 
      * When a child of the source item is not found in the destination item,
      * the search stops, the mapping is (obviously) not done and the source
      * child and its future brothers are discared.
+     * @param Docman_Folder $dstItem
      */
     function compareFolderChildren($srcItem, $dstItem) {
         $nodesToInspect = array();
@@ -210,6 +211,7 @@ 
 
     /**
      * Search if there is an equivalent of $item in $parentId.
+     * @param integer $parentId
      */
     function searchMatchingItem($item, $parentId) {
         $dao =& $this->getItemDao();

plugins/docman/include/Docman_CloneItemsVisitor.class.php

Doc Comments +3 added lines

@@ -228,6 +228,9 @@
         }
     }
     
+    /**
+     * @param string $mdLabel
+     */
     function _metadataEnabled($srcGroupId, $mdLabel) {
         if(!isset($this->_cacheMetadataUsage[$mdLabel])) {
             $srcSettingsBo =& $this->_getSettingsBo($srcGroupId);

plugins/docman/include/Docman_Controller.class.php

Doc Comments +12 added lines

@@ -125,6 +125,7 @@ 
     /**
      * Wrapper to i18n string call for docman.
      * static
+     * @param string $key
      */
     function txt($key, $vars = array()) {
         return $GLOBALS['Language']->getText('plugin_docman', $key, $vars);
@@ -147,6 +148,9 @@ 
         }
     }
 
+    /**
+     * @param integer $srcGroupId
+     */
     function _cloneDocman($srcGroupId, $dstGroupId, $ugroupsMapping) {
         $user = $this->getUser();
 
@@ -1550,6 +1554,9 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     */
     function getProperty($name) {
         $info =& $this->plugin->getPluginInfo();
         return $info->getPropertyValueForName($name);
@@ -1647,6 +1654,7 @@ 
     * Checks that the new property have a non-empty name,
     * and also checks that the same name is not already taken by
     * another property
+    * @param string $name
     */
     private function validateNewMetadata($name) {
         $name = trim($name);
@@ -1671,6 +1679,7 @@ 
     * Checks that the updating property have a non-empty name,
     * and if the name have been changed, also checks that the same
     * name is not already taken by another property
+    * @param string $name
     */
     private function validateUpdateMetadata($name, $label) {
         $name = trim($name);
@@ -1697,6 +1706,9 @@ 
         return $valid;
     }
     
+    /**
+     * @param integer $loveId
+     */
     function validateLove($loveId, $md, &$love) {
         $valid = false;