Enalean / tuleap

plugins/git/include/Git/Ci/Launcher.class.php

Doc Comments -1 removed lines

@@ -49,7 +49,6 @@
     /**
      * Trigger jobs corresponding to the Git repository
      *
-     * @param GitRepository $repository_location Name of the git repository
      */
     public function executeForRepository(GitRepository $repository) {
         if ($repository->getProject()->usesService('hudson')) {

plugins/git/include/Git/Driver/Gerrit/ExecFetch.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      * Update a local branch from remote
      *
      * @param String $branch_name
-     * @return type
+     * @return boolean
      */
     public function updateRef($branch_name) {
         return $this->gitCmd("git update-ref refs/heads/$branch_name refs/remotes/$this->remote_name/$branch_name");

plugins/git/include/Git/Driver/Gerrit/MembershipDao.class.php

Doc Comments +4 added lines

@@ -26,6 +26,10 @@
 
 class Git_Driver_Gerrit_MembershipDao extends DataAccessObject {
 
+    /**
+     * @param integer $group_id
+     * @param integer $ugroup_id
+     */
     public function addReference($group_id, $ugroup_id, $remote_server_id) {
         $group_id = $this->da->escapeInt($group_id);
         $ugroup_id = $this->da->escapeInt($ugroup_id);

plugins/git/include/Git/Driver/Gerrit/MembershipManager.class.php

Doc Comments -1 removed lines

@@ -310,7 +310,6 @@
      * Returns gerrit user group uuid
      *
      * @param Git_RemoteServer_GerritServer $server
-     * @param String                        $ugroup
      * @return String
      * @throws Exception
      */

plugins/git/include/Git/Driver/Gerrit/ProjectCreator.class.php

Doc Comments +9 added lines

@@ -84,6 +84,9 @@ 
     /** @var Git_Driver_Gerrit_Template_TemplateFactory */
     private $template_factory;
 
+    /**
+     * @param string $dir
+     */
     public function __construct(
         $dir,
         Git_Driver_Gerrit_GerritDriverFactory $driver_factory,
@@ -204,6 +207,9 @@ 
         $executor->exportBranchesAndTags($gerrit_project_url);
     }
 
+    /**
+     * @param string $replication_group
+     */
     private function pushFullTuleapAccessRightsToGerrit(GitRepository $repository, Git_RemoteServer_GerritServer $gerrit_server, array $ugroups, $replication_group, $template_id) {
         foreach ($ugroups as $ugroup) {
             $this->addGroupToGroupFile($gerrit_server, $repository->getProject()->getUnixName().'/'.$ugroup->getNormalizedName());
@@ -263,6 +269,9 @@ 
         $this->addGroupDefinitionToGroupFile('global:Registered-Users', self::GROUP_REGISTERED_USERS);
     }
 
+    /**
+     * @param string $uuid
+     */
     private function addGroupDefinitionToGroupFile($uuid, $group_name) {
         file_put_contents("$this->dir/groups", "$uuid\t$group_name\n", FILE_APPEND);
     }

plugins/git/include/Git/Driver/Gerrit/RemoteSSHCommandFailure.class.php

Doc Comments +4 added lines

@@ -24,6 +24,10 @@
     private $std_out;
     private $std_err;
 
+    /**
+     * @param string $std_out
+     * @param string $std_err
+     */
     function __construct($exit_code, $std_out, $std_err) {
         parent::__construct(implode(PHP_EOL, array("exit_code: $exit_code", "std_err: $std_err", "std_out: $std_out")));
         $this->exit_code = $exit_code;

plugins/git/include/Git/Driver/Gerrit/Template/Template.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
     /**
      * @param int $group_id
      *
-     * @return True if this template belongs to the given project
+     * @return boolean if this template belongs to the given project
      */
     public function belongsToProject($group_id) {
         return $this->group_id == $group_id;

plugins/git/include/Git/Driver/Gerrit/Template/TemplateDao.class.php

Doc Comments +19 added lines

@@ -27,6 +27,9 @@ 
  */
 class Git_Driver_Gerrit_Template_TemplateDao extends DataAccessObject {
 
+    /**
+     * @param integer $project_id
+     */
     public function getAllTemplatesOfProject($project_id) {
         $project_id = $this->da->escapeInt($project_id);
 
@@ -37,6 +40,9 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $template_id
+     */
     public function getTemplate($template_id) {
         $template_id = $this->da->escapeInt($template_id);
 
@@ -47,6 +53,11 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $project_id
+     * @param string $name
+     * @param string $content
+     */
     public function addTemplate($project_id, $name, $content) {
         $project_id = $this->da->escapeInt($project_id);
         $name       = $this->da->quoteSmart($name);
@@ -65,6 +76,11 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $template_id
+     * @param string $name
+     * @param string $content
+     */
     public function updateTemplate($template_id, $name, $content) {
         $template_id = $this->da->escapeInt($template_id);
         $name        = $this->da->quoteSmart($name);
@@ -77,6 +93,9 @@ 
         return $this->update($sql);
     }
 
+    /**
+     * @param integer $template_id
+     */
     public function deleteTemplate($template_id) {
         $template_id = $this->da->escapeInt($template_id);
 

plugins/git/include/Git/Driver/Gerrit/Template/TemplateFactory.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
      * Instatiate a Template from a SQL row
      *
      * @param array
-     * @return Git_Driver_Gerrit_Template_Template[] | false -where the array is in DAR format
+     * @return Git_Driver_Gerrit_Template_Template | false -where the array is in DAR format
      */
     private function instantiateTemplateFromRow(array $row) {
         return new Git_Driver_Gerrit_Template_Template(