Enalean / tuleap

plugins/git/tests/GitActionsTest.php

Doc Comments +4 added lines

@@ -617,6 +617,10 @@
         $this->changeProjectRepositoriesAccess($project_id, $is_private);
     }
 
+    /**
+     * @param integer $project_id
+     * @param boolean $is_private
+     */
     private function changeProjectRepositoriesAccess($project_id, $is_private) {
         return GitActions::changeProjectRepositoriesAccess($project_id, $is_private, $this->dao, $this->factory);
     }

plugins/git/tests/GitRepositoryManagerTest.php

Doc Comments +15 added lines

@@ -94,6 +94,9 @@ 
         $this->manager    = new GitRepositoryManager($this->factory, mock('Git_SystemEventManager'), $this->dao, $this->backup_directory);
     }
 
+    /**
+     * @param string $path
+     */
     private function aRepoWithPath($path) {
         return aGitRepository()->withPath($this->project_name.'/'.$path.'.git')->withProject($this->project)->build();
     }
@@ -229,6 +232,9 @@ 
 class GitRepositoryIdMatchExpectation extends SimpleExpectation {
     private $repository_id;
 
+    /**
+     * @param integer $repository_id
+     */
     public function __construct($repository_id) {
         parent::__construct();
         $this->repository_id = $repository_id;
@@ -397,6 +403,9 @@ 
         $this->manager->forkRepositories($repos, $to_project, $this->user, '', null, $this->forkPermissions);
     }
 
+    /**
+     * @param string[] $repo_ids
+     */
     protected function getRepoCollectionUnreadableFor($repo_ids, $user) {
         $return = array();
         foreach ($repo_ids as $id) {
@@ -487,6 +496,9 @@ 
         $this->forkRepositories(array($this->repository), '^toto/pouet');
     }
 
+    /**
+     * @param integer $id
+     */
     private function GivenARepository($id) {
         $repo = new MockGitRepository();
         $repo->setReturnValue('getId', $id);
@@ -496,6 +508,9 @@ 
         return $repo;
     }
 
+    /**
+     * @param string $namespace
+     */
     private function forkRepositories($repositories, $namespace=null) {
         $this->manager->forkRepositories($repositories, $this->project, $this->user, $namespace, null, $this->forkPermissions);
     }

plugins/git/tests/GitRepositoryTest.php

Doc Comments +4 added lines

@@ -140,6 +140,10 @@
         $this->assertEqual('u/johan/tulip', $repo->getFullName());
     }
 
+    /**
+     * @param string $name
+     * @param null|string $namespace
+     */
     protected function _GivenARepositoryWithNameAndNamespace($name, $namespace) {
         $repo = new GitRepository();
         $repo->setName($name);

plugins/git/tests/GitTest.php

Doc Comments +6 added lines

@@ -229,6 +229,9 @@ 
         $this->assertItNeedsAValidRepoId();
     }
 
+    /**
+     * @param Codendi_Request $request
+     */
     protected function getGit($request, $factory) {
         $git = parent::getGit($request, $factory);
         $git->setAction('disconnect_gerrit');
@@ -332,6 +335,9 @@ 
         $git->request();
     }
 
+    /**
+     * @param Codendi_Request $request
+     */
     protected function getGit($request, $factory, $template_factory = null) {
         $git = parent::getGit($request, $factory, $template_factory);
         $git->setAction('migrate_to_gerrit');

plugins/git/tests/GitViewsRepositoriesTraversalStrategyTest.class.php

Doc Comments +3 added lines

@@ -26,6 +26,9 @@
 
 abstract class GitViewsRepositoriesTraversalStrategyTest extends TuleapTestCase {
     
+    /**
+     * @param string $classname
+     */
     public function __construct($classname) {
         parent::__construct();
         $this->classname = $classname;

plugins/git/tests/GitViewsTest.php

Doc Comments +7 added lines

@@ -86,6 +86,9 @@ 
 
     }
 
+    /**
+     * @param string $unixName
+     */
     private function GivenAProject($id, $name, $unixName = null, $useGit = true) {
         $project = new MockProject();
         $project->setReturnValue('getId', $id);
@@ -95,6 +98,10 @@ 
         return $project;
     }
 
+    /**
+     * @param string $id
+     * @param string $name
+     */
     private function GivenAProjectWithoutGitService($id, $name) {
         return $this->GivenAProject($id, $name, null, false);
     }

plugins/git/tests/GitXmlImporterTest.php

Doc Comments +3 added lines

@@ -245,6 +245,9 @@
         $this->assertEqual(GitRepository::DEFAULT_DESCRIPTION, $this->git_dao->last_saved_repository->getDescription());
     }
 
+    /**
+     * @param SimpleXMLElement $xml
+     */
     private function import($xml) {
         return $this->importer->import($this->project, mock('PFUSer'), $xml, parent::getTmpDir());
     }

plugins/graphontrackers/include/data-access/GraphicReportFactory.class.php

Doc Comments -1 removed lines

@@ -81,7 +81,6 @@
     /**
      * Display the report list
      *
-     * @param : $reports      the list the reports within an artifact to display
      *
      * @return void
      */

plugins/graphontrackers/include/data-access/GraphOnTrackers_ChartDao.class.php

Doc Comments +8 added lines

@@ -34,6 +34,9 @@ 
             $this->table_name = 'plugin_graphontrackers_chart';
         }
         
+        /**
+         * @param string $report_id
+         */
         public function searchByReportId($report_id) {
             $sql = "SELECT * FROM plugin_graphontrackers_chart WHERE report_graphic_id = ";
             $sql .= $this->da->escapeInt($report_id);
@@ -53,6 +56,11 @@ 
             return $this->update($sql);
         }
         
+        /**
+         * @param string $report_id
+         * @param string $title
+         * @param string $description
+         */
         public function create($report_id, $chart_type, $title, $description, $width, $height) {
             $rank = $this->prepareRanking(0, $report_id, 'beginning', 'id', 'report_graphic_id');
             $sql = sprintf("INSERT INTO plugin_graphontrackers_chart(report_graphic_id, rank, chart_type, title, description, width, height) VALUES (%d, %d, %s, %s, %s, %d, %d)",