ProjectsController - Code Metrics - Inspection of "Revert "Misc: added .DS_Store to .gitignore"" - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (a31570)

by Mohamed

created 2016-01-10 07:04 UTC

ProjectsController A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	146
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	11

Test Coverage

Coverage

100%

Importance

Changes	9
Bugs	3	Features	2

Metric	Value
wmc	10
c	9
b	3
f	2
lcom	1
cbo	11
dl	0
loc	146
ccs	32
cts	32
cp	1
rs	10

6 Methods

Rating	Name	Size	Complexity
B	getIndex()	34	5
A	getNew()	7	1
A	postNew()	6	1
A	postProgress()	21	1
A	getNewIssue()	7	1
A	postNewIssue()	18	1

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Http\Controllers;

use Illuminate\Http\Request;
use Tinyissue\Form\Project as Form;
use Tinyissue\Form\GlobalIssue as IssueForm;
use Tinyissue\Http\Requests\FormRequest;
use Tinyissue\Model\Project;
use Tinyissue\Model\User;

/**
 * ProjectsController is the controller class for managing request related to projects
 *
 * @author Mohamed Alsharaf <[email protected]>
 */
class ProjectsController extends Controller
{
    /**
     * Display list of active/archived projects
     *
     * @param int $status
     *
     * @return \Illuminate\View\View
     */
    public function getIndex($status = Project::STATUS_OPEN)
    {
        $viewData = [];
        if (!$this->auth->guest()) {
            $projects = $this->auth->user()->projectsWithCountOpenIssues($status)->get();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            if ($status) {
                $countActive = $projects->count();
                $countArchived = $this->auth->user()->projectsWithCountOpenIssues(Project::STATUS_ARCHIVED)->count();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            } else {
                $countActive = $this->auth->user()->projectsWithCountOpenIssues(Project::STATUS_OPEN)->count();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
                $countArchived = $projects->count();
            }
            $viewData['projects'] = $this->auth->user()->projects()->get();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        } else {
            $viewData['sidebar'] = 'public';
            $project = new Project();
            $projects = $project->projectsWithOpenIssuesCount($status, Project::PRIVATE_NO)->get();
            if ($status) {
                $countActive = $projects->count();
                $countArchived = $project->projectsWithOpenIssuesCount(Project::STATUS_ARCHIVED, Project::PRIVATE_NO)->count();
            } else {
                $countActive = $project->projectsWithOpenIssuesCount(Project::STATUS_OPEN, Project::PRIVATE_NO)->count();
                $countArchived = $projects->count();
            }
            $user = new User();
            $viewData['activeUsers'] = $user->activeUsers();
        }
        $viewData['content_projects'] = $projects;
        $viewData['active'] = $status === Project::STATUS_OPEN? 'active' : 'archived';
        $viewData['active_count'] = $countActive;
        $viewData['archived_count'] = $countArchived;

        return view('projects.index', $viewData);
    }

    /**
     * Add new project form
     *
     * @param Form $form
     *
     * @return \Illuminate\View\View
     */
    public function getNew(Form $form)
    {
        return view('projects.new', [
            'form' => $form,
            'projects' => $this->auth->user()->projects()->get(),
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        ]);
    }

    /**
     * To create a new project
     *
     * @param Project             $project
     * @param FormRequest\Project $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function postNew(Project $project, FormRequest\Project $request)
    {
        $project->createProject($request->all());

        return redirect($project->to());
    }

    /**
     * Ajax: Calculate the progress of user projects
     *
     * @param Request $request
     * @param Project $project
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function postProgress(Request $request, Project $project)
    {
        // Get all projects with count of closed and opened issues
        $projects = $project->projectsWithCountIssues((array) $request->input('ids'));

        // The project progress Html and value
        $progress = $projects->transform(function (Project $project) {
            $progress = $project->getProgress();
            $view = view('partials/progress', ['text' => $progress . '%', 'progress' => $progress])->render();

            return [
                'id' => $project->id,
                'progress' => [
                    'html' => $view,
                    'value' => $progress,
                ],
            ];
        })->lists('progress', 'id')->all();

        return response()->json(['status' => true, 'progress' => $progress]);
    }

    /**
     * Add new issue form
     *
     * @param IssueForm $form
     *
     * @return \Illuminate\View\View
     */
    public function getNewIssue(IssueForm $form)
    {
        return view('projects.new-issue', [
            'form' => $form,
            'projects' => $this->auth->user()->projects()->get(),
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        ]);
    }

    /**
     * To create a new issue
     *
     * @param Project\Issue           $issue
     * @param FormRequest\GlobalIssue $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function postNewIssue(Project\Issue $issue, FormRequest\GlobalIssue $request)
    {
        $project = Project::find((int) $request->input('project'));

        $issue->setRelation('project', $project);
        $issue->setRelation('user', $this->auth->user());
        $issue->createIssue([
            'title' => $request->input('title'),
            'body' => $request->input('body'),
            'tag' => $request->input('tag'),
            'upload_token' => $request->input('upload_token'),
            'assigned_to' => (int) $project->default_assignee,
            'time_quote' => 0,
        ]);

        return redirect($issue->to())
            ->with('notice', trans('tinyissue.issue_has_been_created'));
    }
}


1		<?php
2
3		/*
4		* This file is part of the Tinyissue package.
5		*
6		* (c) Mohamed Alsharaf <[email protected]>
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 Tinyissue\Http\Controllers;
13
14		use Illuminate\Http\Request;
15		use Tinyissue\Form\Project as Form;
16		use Tinyissue\Form\GlobalIssue as IssueForm;
17		use Tinyissue\Http\Requests\FormRequest;
18		use Tinyissue\Model\Project;
19		use Tinyissue\Model\User;
20
21		/**
22		* ProjectsController is the controller class for managing request related to projects
23		*
24		* @author Mohamed Alsharaf <[email protected]>
25		*/
26		class ProjectsController extends Controller
27		{
28		/**
29		* Display list of active/archived projects
30		*
31		* @param int $status
32		*
33		* @return \Illuminate\View\View
34		*/
35	3	public function getIndex($status = Project::STATUS_OPEN)
36		{
37	3	$viewData = [];
38	3	if (!$this->auth->guest()) {
39	3	$projects = $this->auth->user()->projectsWithCountOpenIssues($status)->get();
		0 ignored issues – show Bug introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Contracts\Auth\Authenticatable` as the method `projectsWithCountOpenIssues()` does only exist in the following implementations of said interface: `Tinyissue\Model\User`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
40	3	if ($status) {
41	3	$countActive = $projects->count();
42	3	$countArchived = $this->auth->user()->projectsWithCountOpenIssues(Project::STATUS_ARCHIVED)->count();
		0 ignored issues – show Bug introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Contracts\Auth\Authenticatable` as the method `projectsWithCountOpenIssues()` does only exist in the following implementations of said interface: `Tinyissue\Model\User`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
43		} else {
44	2	$countActive = $this->auth->user()->projectsWithCountOpenIssues(Project::STATUS_OPEN)->count();
		0 ignored issues – show Bug introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Contracts\Auth\Authenticatable` as the method `projectsWithCountOpenIssues()` does only exist in the following implementations of said interface: `Tinyissue\Model\User`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
45	2	$countArchived = $projects->count();
46		}
47	3	$viewData['projects'] = $this->auth->user()->projects()->get();
		0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Contracts\Auth\Authenticatable` as the method `projects()` does only exist in the following implementations of said interface: `Tinyissue\Model\User`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
48		} else {
49		$viewData['sidebar'] = 'public';
50		$project = new Project();
51		$projects = $project->projectsWithOpenIssuesCount($status, Project::PRIVATE_NO)->get();
52		if ($status) {
53		$countActive = $projects->count();
54		$countArchived = $project->projectsWithOpenIssuesCount(Project::STATUS_ARCHIVED, Project::PRIVATE_NO)->count();
55		} else {
56		$countActive = $project->projectsWithOpenIssuesCount(Project::STATUS_OPEN, Project::PRIVATE_NO)->count();
57		$countArchived = $projects->count();
58		}
59		$user = new User();
60		$viewData['activeUsers'] = $user->activeUsers();
61		}
62	3	$viewData['content_projects'] = $projects;
63	3	$viewData['active'] = $status === Project::STATUS_OPEN? 'active' : 'archived';
64	3	$viewData['active_count'] = $countActive;
65	3	$viewData['archived_count'] = $countArchived;
66
67	3	return view('projects.index', $viewData);
68		}
69
70		/**
71		* Add new project form
72		*
73		* @param Form $form
74		*
75		* @return \Illuminate\View\View
76		*/
77	1	public function getNew(Form $form)
78		{
79	1	return view('projects.new', [
80	1	'form' => $form,
81	1	'projects' => $this->auth->user()->projects()->get(),
		0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Contracts\Auth\Authenticatable` as the method `projects()` does only exist in the following implementations of said interface: `Tinyissue\Model\User`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
82		]);
83		}
84
85		/**
86		* To create a new project
87		*
88		* @param Project $project
89		* @param FormRequest\Project $request
90		*
91		* @return \Illuminate\Http\RedirectResponse
92		*/
93	1	public function postNew(Project $project, FormRequest\Project $request)
94		{
95	1	$project->createProject($request->all());
96
97	1	return redirect($project->to());
98		}
99
100		/**
101		* Ajax: Calculate the progress of user projects
102		*
103		* @param Request $request
104		* @param Project $project
105		*
106		* @return \Symfony\Component\HttpFoundation\Response
107		*/
108	1	public function postProgress(Request $request, Project $project)
109		{
110		// Get all projects with count of closed and opened issues
111	1	$projects = $project->projectsWithCountIssues((array) $request->input('ids'));
112
113		// The project progress Html and value
114	1	$progress = $projects->transform(function (Project $project) {
115	1	$progress = $project->getProgress();
116	1	$view = view('partials/progress', ['text' => $progress . '%', 'progress' => $progress])->render();
117
118		return [
119	1	'id' => $project->id,
120		'progress' => [
121	1	'html' => $view,
122	1	'value' => $progress,
123		],
124		];
125	1	})->lists('progress', 'id')->all();
126
127	1	return response()->json(['status' => true, 'progress' => $progress]);
128		}
129
130		/**
131		* Add new issue form
132		*
133		* @param IssueForm $form
134		*
135		* @return \Illuminate\View\View
136		*/
137		public function getNewIssue(IssueForm $form)
138		{
139		return view('projects.new-issue', [
140		'form' => $form,
141		'projects' => $this->auth->user()->projects()->get(),
		0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Contracts\Auth\Authenticatable` as the method `projects()` does only exist in the following implementations of said interface: `Tinyissue\Model\User`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
142		]);
143		}
144
145		/**
146		* To create a new issue
147		*
148		* @param Project\Issue $issue
149		* @param FormRequest\GlobalIssue $request
150		*
151		* @return \Illuminate\Http\RedirectResponse
152		*/
153		public function postNewIssue(Project\Issue $issue, FormRequest\GlobalIssue $request)
154		{
155		$project = Project::find((int) $request->input('project'));
156
157		$issue->setRelation('project', $project);
158		$issue->setRelation('user', $this->auth->user());
159		$issue->createIssue([
160		'title' => $request->input('title'),
161		'body' => $request->input('body'),
162		'tag' => $request->input('tag'),
163		'upload_token' => $request->input('upload_token'),
164		'assigned_to' => (int) $project->default_assignee,
165		'time_quote' => 0,
166		]);
167
168		return redirect($issue->to())
169		->with('notice', trans('tinyissue.issue_has_been_created'));
170		}
171		}
172

satrun77 / tinyissue

Branch — develop (a31570)

ProjectsController A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

6 Methods

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like