Enalean / tuleap

plugins/git/include/Git/ReferenceManager.class.php

Doc Comments +3 added lines

@@ -64,6 +64,9 @@
         return $reference;
     }
 
+    /**
+     * @param string $value
+     */
     private function splitRepositoryAndSha1($value) {
         $last_slash_position  = strrpos($value, '/');
         $repository_name      = substr($value, 0, $last_slash_position);

plugins/git/include/Git/RemoteServer/Dao.class.php

Doc Comments +17 added lines

@@ -20,6 +20,9 @@ 
 require_once 'common/dao/include/DataAccessObject.class.php';
 class Git_RemoteServer_Dao extends DataAccessObject {
 
+    /**
+     * @param integer $id
+     */
     public function searchById($id) {
         $id = $this->da->escapeInt($id);
         $sql = "SELECT *
@@ -36,6 +39,7 @@ 
     /**
      * This sql request returns for a given project all the servers
      * where its git repositories are migrated
+     * @param integer $project_id
      */
     public function searchAllByProjectId($project_id) {
         $project_id = $this->da->escapeInt($project_id);
@@ -46,6 +50,9 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $user_id
+     */
     public function searchAllRemoteServersForUserId($user_id) {
         $sql = "SELECT DISTINCT pgrs.*
                 FROM plugin_git_remote_servers pgrs
@@ -68,6 +75,10 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param integer $project_id
+     * @param integer $ugroup_id
+     */
     public function searchAllByUGroupId($project_id, $ugroup_id) {
         $project_id = $this->da->escapeInt($project_id);
         $ugroup_id  = $this->da->escapeInt($ugroup_id);
@@ -79,6 +90,12 @@ 
         return $this->retrieve($sql);
     }
 
+    /**
+     * @param string $replication_key
+     * @param boolean $use_ssl
+     * @param string $gerrit_version
+     * @param string $http_password
+     */
     public function save(
         $id,
         $host,

plugins/git/include/Git/RemoteServer/Gerrit/ReplicationSSHKey.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
     /**
      * @param int $id
-     * @return \Git_RemoteServer_GerritReplicationSSHKey
+     * @return Git_RemoteServer_Gerrit_ReplicationSSHKey
      */
     public function setGerritHostId($id) {
         $this->host_id = $id;

plugins/git/include/Git/RemoteServer/GerritServerPresenter.class.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
 
 class Git_RemoteServer_GerritServerPresenter {
 
+    /**
+     * @param boolean $is_used
+     */
     public function __construct(Git_RemoteServer_GerritServer $server, $is_used) {
         $this->id               = $server->getId();
         $this->host             = $server->getHost();

plugins/git/include/Git/RemoteServer/NotFoundException.class.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
 
 class Git_RemoteServer_NotFoundException extends Exception {
 
+    /**
+     * @param integer $id
+     */
     public function __construct($id) {
         parent::__construct("No server found with the id: $id");
     }

plugins/git/include/Git/SystemEventManager.class.php

Doc Comments +6 added lines

@@ -112,6 +112,9 @@ 
         );
     }
 
+    /**
+     * @param integer $remote_server_id
+     */
     public function queueMigrateToGerrit(GitRepository $repository, $remote_server_id, $gerrit_template_id) {
         $this->system_event_manager->createEvent(
             SystemEvent_GIT_GERRIT_MIGRATION::NAME,
@@ -161,6 +164,9 @@ 
         );
     }
 
+    /**
+     * @param string $repository_path
+     */
     public function queueGrokMirrorManifestRepoDelete($repository_path) {
         $this->system_event_manager->createEvent(
             SystemEvent_GIT_GROKMIRROR_MANIFEST_REPODELETE::NAME,

plugins/git/include/Git/UserAccountManager.class.php

Doc Comments +6 added lines, -1 removed lines

@@ -50,6 +50,9 @@ 
         $this->synchroniseSSHKeysWithGerrit($original_keys, $new_keys, $user);
     }
 
+    /**
+     * @param PFUser $user
+     */
     private function synchroniseSSHKeysWithGerrit($original_keys, $new_keys, $user) {
         $this->getGerritUserAccountManager()
             ->synchroniseSSHKeys(
@@ -61,7 +64,6 @@ 
 
     /**
      *
-     * @param Git_RemoteServer_GerritServerFactory $factory
      * @return void
      * @throws Git_UserSynchronisationException
      */
@@ -69,6 +71,9 @@ 
         $this->pushSSHKeysToGerrit($user);
     }
 
+    /**
+     * @param PFUser $user
+     */
     private function pushSSHKeysToGerrit($user) {
         $this->getGerritUserAccountManager()->pushSSHKeys($user);
     }

plugins/git/include/Git_Backend_Gitolite.class.php

Doc Comments +5 added lines, -1 removed lines

@@ -68,6 +68,7 @@ 
     }
 
     /**
+     * @param GitRepository $repository
      * @return bool
      */
     public function updateRepoConf($repository) {
@@ -136,6 +137,9 @@ 
         }
     }
 
+    /**
+     * @param string $key
+     */
     private function getConfigurationParameter($key) {
         $value = $this->getGitPlugin()->getConfigurationParameter($key);
         if ($value !== false && $value !== null) {
@@ -283,7 +287,7 @@ 
      * @param Project $project The project to rename
      * @param string  $newName The new name of the project
      *
-     * @return true if success, false otherwise
+     * @return boolean if success, false otherwise
      */
     public function renameProject(Project $project, $newName) {
         if (is_dir($this->driver->getRepositoriesPath() .'/'. $project->getUnixName())) {

plugins/git/include/Git_Backend_Interface.php

Doc Comments +7 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      * Verify if the repository as already some content within
      *
      * @see    plugins/git/include/Git_Backend_Interface::isInitialized()
-     * @param  GitRepository $repository
+     * @param  GitRepository $respository
      * @return Boolean
      */
     public function isInitialized(GitRepository $respository);
@@ -35,7 +35,7 @@ 
      * Verify if the repository has been created
      *
      * @see    plugins/git/include/Git_Backend_Interface::isCreated()
-     * @param  GitRepository $repository
+     * @param  GitRepository $respository
      * @return Boolean
      */
     public function isCreated(GitRepository $respository);
@@ -105,7 +105,7 @@ 
      * @param Project $project The project to rename
      * @param string  $newName The new name of the project
      *
-     * @return true if success, false otherwise
+     * @return boolean if success, false otherwise
      */
     public function renameProject(Project $project, $newName);
 
@@ -120,6 +120,7 @@ 
      * Perform logical deletion repository in DB
      *
      * @param GitRepository $repository
+     * @return void
      */
     public function markAsDeleted(GitRepository $repository);
 
@@ -127,6 +128,7 @@ 
      * Physically delete a repository already marked for deletion
      *
      * @param GitRepository $repository
+     * @return void
      */
     public function delete(GitRepository $repository);
 
@@ -134,6 +136,7 @@ 
      * Purge archived repository
      *
      * @param GitRepository $repository
+     * @return void
      */
     public function deleteArchivedRepository(GitRepository $repository);
 
@@ -141,6 +144,7 @@ 
      * Move the archived gitolite repositories to the archiving area before purge
      *
      * @param GitRepository $repository
+     * @return boolean
      */
     public function archiveBeforePurge(GitRepository $repository);
 }