Enalean / tuleap

plugins/docman/include/ApprovalTable/Reminder.class.php

Doc Comments +2 added lines, -3 removed lines

@@ -98,7 +98,7 @@ 
      *
      * @param Docman_ApprovalTable $table Approval table
      *
-     * @return Boolean
+     * @return null|boolean
      */
     private function notifyNextReviewer(Docman_ApprovalTable $table) {
         $dao = new Docman_ApprovalTableReviewerDao(CodendiDataAccess::instance());
@@ -121,7 +121,7 @@ 
      * @param Docman_ApprovalTable $table      Approval table
      * @param Integer              $reviewerId Id of the reviewer
      *
-     * @return Boolean
+     * @return null|boolean
      */
     private function notifyIndividual(Docman_ApprovalTable $table, $reviewerId) {
         $hp       = Codendi_HTMLPurifier::instance();
@@ -191,7 +191,6 @@ 
     /**
      * Retrieve url to access a given docman item
      *
-     * @param Docman_Item $table The approval table that its reminder notification will be sent
      *
      * @return String
      */

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

Doc Comments +3 added lines

@@ -51,6 +51,9 @@
         return $this->rank;
     }
 
+    /**
+     * @param integer|null $v
+     */
     function setReviewDate($v) {
         $this->reviewDate = $v;
     }

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();