RepositoryController::recurseDirectory() - Code Metrics - gitaminhq/Gitamin - Measure and Improve Code Quality continuously with Scrutinizer

RepositoryController::recurseDirectory() C
last analyzed 2015-12-17 14:45 UTC

↳ Parent: RepositoryController

Complexity

Conditions	12
Paths	15

Size

Total Lines	61
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	61
rs	6.2857
cc	12
eloc	36
nc	15
nop	2

How to fix Long Method Complexity

<?php

/*
 * This file is part of Gitamin.
 *
 * Copyright (C) 2015-2016 The Gitamin Team
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Gitamin\Http\Controllers;

use Gitamin\Facades\Setting;
use Gitamin\Models\Project;
use Gitter\Client;
use Illuminate\Support\Facades\View;

class RepositoryController extends Controller
{
    protected $client;
    protected $hidden;

    public function __construct()
    {
        $this->hidden = [];
        $this->client = new Client(Setting::get('git_client_path'));
    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index($project = null)
    {
        //
        var_dump($project);

    }

    public function showRepo($team, $project)
    {
        return $this->showTree($team.'/'.$project, '');
    }

    public function showTree($repo, $path)
    {
        $repository = $this->getRepositoryFromName([Setting::get('git_repositories_path')], $repo);

        if (! $path) {
            $path = $repository->getHead();
        }

        if (strpos($path, '/') !== false) {
            $branch = strstr($path, '/', true);
            $tree = str_replace($branch.'/', '', $path);
        } else {
            $branch = $path;
            $tree = '';
        }

        $parent = null;
        if (($slash = strrpos($tree, '/')) !== false) {
            $parent = substr($tree, 0, $slash);
        } elseif (! empty($tree)) {
            $parent = '';
        }

        $files = $repository->getTree($tree ? "$branch:\"$tree\"/" : $branch);

        $pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));

        return View::make('dashboard.repositories.tree')
            ->withPageTitle($pageTitle)
            ->withRepository($repository)
            ->withRepo($repo)
            ->withCurrentBranch($branch)
            ->withBranches($repository->getBranches())
            ->withPath($tree ? $tree.'/' : $tree)
            ->withParentPath($parent)
            ->withFiles($files->output());
    }

    public function showBlob($repo, $path)

    {
        $pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));

        return View::make('dashboard.repositories.blob')
            ->withPageTitle($pageTitle)
            ->withProject([])
            ->withGroups([]);
    }

    public function showCommits($repo, $path)

    {
        $pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));

        return View::make('dashboard.repositories.commits')
            ->withPageTitle($pageTitle)
            ->withProject([])
            ->withGroups([]);
    }

    public function showRaw($repo, $path)

    {
        $pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));

        return View::make('dashboard.repositories.raw')
            ->withPageTitle($pageTitle)
            ->withProject([])
            ->withGroups([]);
    }

    public function showBlame($repo, $path)

    {
        $pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));

        return View::make('dashboard.repositories.blame')
            ->withPageTitle($pageTitle)
            ->withProject([])
            ->withGroups([]);
    }

    public function getRepositoryFromName($paths, $repo)
    {
        $repositories = $this->getRepositories($paths);
        $path = $repositories[$repo]['path'];

        return $this->client->getRepository($path);
    }

    /**
     * Searches for valid repositories on the specified path.
     *
     * @param array $paths Array of paths where repositories will be searched
     *
     * @return array Found repositories, containing their name, path and description sorted
     *               by repository name
     */
    public function getRepositories($paths)
    {
        $allRepositories = [];

        foreach ($paths as $path) {
            $repositories = $this->recurseDirectory($path);

            if (empty($repositories)) {
                throw new \RuntimeException('There are no GIT repositories in '.$path);
            }

            $allRepositories = $allRepositories + $repositories;
        }

        $allRepositories = array_unique($allRepositories, SORT_REGULAR);
        uksort($allRepositories, function ($k1, $k2) {
            return strtolower($k2) < strtolower($k1);
        });

        return $allRepositories;
    }

    private function recurseDirectory($path, $topLevel = true)
    {
        $dir = new \DirectoryIterator($path);

        $repositories = [];

        foreach ($dir as $file) {
            if ($file->isDot()) {
                continue;
            }

            if (strrpos($file->getFilename(), '.') === 0) {
                continue;
            }

            if (! $file->isReadable()) {
                continue;
            }

            if ($file->isDir()) {
                $isBare = file_exists($file->getPathname().'/HEAD');
                $isRepository = file_exists($file->getPathname().'/.git/HEAD');

                if ($isRepository || $isBare) {
                    if (in_array($file->getPathname(), $this->getHidden(), true)) {
                        continue;
                    }

                    if ($isBare) {
                        $description = $file->getPathname().'/description';
                    } else {
                        $description = $file->getPathname().'/.git/description';
                    }

                    if (file_exists($description)) {
                        $description = file_get_contents($description);
                    } else {
                        $description = null;
                    }

                    if (! $topLevel) {
                        $repoName = $file->getPathInfo()->getFilename().'/'.$file->getFilename();
                    } else {
                        $repoName = $file->getFilename();
                    }

                    $repositories[$repoName] = [
                        'name' => $repoName,
                        'path' => $file->getPathname(),
                        'description' => $description,
                    ];

                    continue;
                } else {
                    $repositories = array_merge($repositories, $this->recurseDirectory($file->getPathname(), false));
                }
            }
        }

        return $repositories;
    }

    /**
     * Get hidden repository list.
     *
     * @return array List of repositories to hide
     */
    protected function getHidden()
    {
        return $this->hidden;
    }
}


1		<?php
2
3		/*
4		* This file is part of Gitamin.
5		*
6		* Copyright (C) 2015-2016 The Gitamin Team
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Gitamin\Http\Controllers;
13
14		use Gitamin\Facades\Setting;
15		use Gitamin\Models\Project;
16		use Gitter\Client;
17		use Illuminate\Support\Facades\View;
18
19		class RepositoryController extends Controller
20		{
21		protected $client;
22		protected $hidden;
23
24		public function __construct()
25		{
26		$this->hidden = [];
27		$this->client = new Client(Setting::get('git_client_path'));
28		}
29
30		/**
31		* Display a listing of the resource.
32		*
33		* @return \Illuminate\Http\Response
34		*/
35		public function index($project = null)
36		{
37		//
38		var_dump($project);
		0 ignored issues – show Security Debugging Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report `var_dump($project);` looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data. Loading history...
39		}
40
41		public function showRepo($team, $project)
42		{
43		return $this->showTree($team.'/'.$project, '');
44		}
45
46		public function showTree($repo, $path)
47		{
48		$repository = $this->getRepositoryFromName([Setting::get('git_repositories_path')], $repo);
49
50		if (! $path) {
51		$path = $repository->getHead();
52		}
53
54		if (strpos($path, '/') !== false) {
55		$branch = strstr($path, '/', true);
56		$tree = str_replace($branch.'/', '', $path);
57		} else {
58		$branch = $path;
59		$tree = '';
60		}
61
62		$parent = null;
63		if (($slash = strrpos($tree, '/')) !== false) {
64		$parent = substr($tree, 0, $slash);
65		} elseif (! empty($tree)) {
66		$parent = '';
67		}
68
69		$files = $repository->getTree($tree ? "$branch:\"$tree\"/" : $branch);
70
71		$pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));
72
73		return View::make('dashboard.repositories.tree')
74		->withPageTitle($pageTitle)
75		->withRepository($repository)
76		->withRepo($repo)
77		->withCurrentBranch($branch)
78		->withBranches($repository->getBranches())
79		->withPath($tree ? $tree.'/' : $tree)
80		->withParentPath($parent)
81		->withFiles($files->output());
82		}
83
84	View Code Duplication	public function showBlob($repo, $path)
		0 ignored issues – show Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$repo` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Duplication introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
85		{
86		$pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));
87
88		return View::make('dashboard.repositories.blob')
89		->withPageTitle($pageTitle)
90		->withProject([])
91		->withGroups([]);
92		}
93
94	View Code Duplication	public function showCommits($repo, $path)
		0 ignored issues – show Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$repo` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Duplication introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
95		{
96		$pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));
97
98		return View::make('dashboard.repositories.commits')
99		->withPageTitle($pageTitle)
100		->withProject([])
101		->withGroups([]);
102		}
103
104	View Code Duplication	public function showRaw($repo, $path)
		0 ignored issues – show Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$repo` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Duplication introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
105		{
106		$pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));
107
108		return View::make('dashboard.repositories.raw')
109		->withPageTitle($pageTitle)
110		->withProject([])
111		->withGroups([]);
112		}
113
114	View Code Duplication	public function showBlame($repo, $path)
		0 ignored issues – show Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$repo` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Duplication introduced 2015-11-30 08:43 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
115		{
116		$pageTitle = sprintf('"%s" - %s - %s', 'project title', trans('dashboard.projects.edit.title'), trans('dashboard.dashboard'));
117
118		return View::make('dashboard.repositories.blame')
119		->withPageTitle($pageTitle)
120		->withProject([])
121		->withGroups([]);
122		}
123
124		public function getRepositoryFromName($paths, $repo)
125		{
126		$repositories = $this->getRepositories($paths);
127		$path = $repositories[$repo]['path'];
128
129		return $this->client->getRepository($path);
130		}
131
132		/**
133		* Searches for valid repositories on the specified path.
134		*
135		* @param array $paths Array of paths where repositories will be searched
136		*
137		* @return array Found repositories, containing their name, path and description sorted
138		* by repository name
139		*/
140		public function getRepositories($paths)
141		{
142		$allRepositories = [];
143
144		foreach ($paths as $path) {
145		$repositories = $this->recurseDirectory($path);
146
147		if (empty($repositories)) {
148		throw new \RuntimeException('There are no GIT repositories in '.$path);
149		}
150
151		$allRepositories = $allRepositories + $repositories;
152		}
153
154		$allRepositories = array_unique($allRepositories, SORT_REGULAR);
155		uksort($allRepositories, function ($k1, $k2) {
156		return strtolower($k2) < strtolower($k1);
157		});
158
159		return $allRepositories;
160		}
161
162		private function recurseDirectory($path, $topLevel = true)
163		{
164		$dir = new \DirectoryIterator($path);
165
166		$repositories = [];
167
168		foreach ($dir as $file) {
169		if ($file->isDot()) {
170		continue;
171		}
172
173		if (strrpos($file->getFilename(), '.') === 0) {
174		continue;
175		}
176
177		if (! $file->isReadable()) {
178		continue;
179		}
180
181		if ($file->isDir()) {
182		$isBare = file_exists($file->getPathname().'/HEAD');
183		$isRepository = file_exists($file->getPathname().'/.git/HEAD');
184
185		if ($isRepository \|\| $isBare) {
186		if (in_array($file->getPathname(), $this->getHidden(), true)) {
187		continue;
188		}
189
190		if ($isBare) {
191		$description = $file->getPathname().'/description';
192		} else {
193		$description = $file->getPathname().'/.git/description';
194		}
195
196		if (file_exists($description)) {
197		$description = file_get_contents($description);
198		} else {
199		$description = null;
200		}
201
202		if (! $topLevel) {
203		$repoName = $file->getPathInfo()->getFilename().'/'.$file->getFilename();
204		} else {
205		$repoName = $file->getFilename();
206		}
207
208		$repositories[$repoName] = [
209		'name' => $repoName,
210		'path' => $file->getPathname(),
211		'description' => $description,
212		];
213
214		continue;
215		} else {
216		$repositories = array_merge($repositories, $this->recurseDirectory($file->getPathname(), false));
217		}
218		}
219		}
220
221		return $repositories;
222		}
223
224		/**
225		* Get hidden repository list.
226		*
227		* @return array List of repositories to hide
228		*/
229		protected function getHidden()
230		{
231		return $this->hidden;
232		}
233		}
234

gitaminhq / Gitamin

RepositoryController::recurseDirectory() C last analyzed 2015-12-17 14:45 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

RepositoryController::recurseDirectory() C
last analyzed 2015-12-17 14:45 UTC