ProjectController::getDownloadExport() - Code Metrics - Inspection of "Misc: increment version & update changes doc" - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 91f8d9...64265e )

by Mohamed

created 2016-05-23 07:25 UTC

ProjectController::getDownloadExport() A

↳ Parent: ProjectController

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	8
ccs	3
cts	3
cp	1
rs	9.4285
cc	1
eloc	3
nc	1
nop	2
crap	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\FilterIssue as FilterForm;
use Tinyissue\Form\Note as NoteForm;
use Tinyissue\Form\Project as Form;
use Tinyissue\Http\Requests\FormRequest;
use Tinyissue\Model\Project;
use Tinyissue\Model\Project\Issue;
use Tinyissue\Model\Project\Note;
use Tinyissue\Services\Exporter;

/**
 * ProjectController is the controller class for managing request related to a project.
 *
 * @author Mohamed Alsharaf <[email protected]>
 */
class ProjectController extends Controller
{
    /**
     * Display activity for a project.
     *
     * @param Project $project
     *
     * @return \Illuminate\View\View
     */
    public function getIndex(Project $project)
    {
        $activities = $project->activities()
            ->with('activity', 'issue', 'user', 'assignTo', 'comment', 'note')
            ->orderBy('created_at', 'DESC')
            ->take(10)
            ->get();

        return view('project.index', [

            'tabs'       => $this->projectMainViewTabs($project, 'index'),
            'project'    => $project,
            'active'     => 'activity',
            'activities' => $activities,
            'sidebar'    => 'project',
        ]);
    }

    /**
     * Display issues for a project.
     *
     * @param FilterForm $filterForm
     * @param Request    $request
     * @param Project    $project
     * @param int        $status
     *
     * @return \Illuminate\View\View
     */
    public function getIssues(FilterForm $filterForm, Request $request, Project $project, $status = Issue::STATUS_OPEN)
    {
        $active = $status == Issue::STATUS_OPEN ? 'open_issue' : 'closed_issue';
        $issues = $project->listIssues($status, $request->all());

        return view('project.index', [

            'tabs'       => $this->projectMainViewTabs($project, 'issues', $issues, $status),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            'project'    => $project,
            'active'     => $active,
            'issues'     => $issues,
            'sidebar'    => 'project',
            'filterForm' => $filterForm,
        ]);
    }

    /**
     * Display issues assigned to current user for a project.
     *
     * @param Project $project
     *
     * @return \Illuminate\View\View
     */
    public function getAssigned(Project $project)
    {
        $issues = $project->listAssignedIssues($this->auth->user()->id);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}

        return view('project.index', [

            'tabs'    => $this->projectMainViewTabs($project, 'assigned', $issues),
            'project' => $project,
            'active'  => 'issue_assigned_to_you',
            'issues'  => $issues,
            'sidebar' => 'project',
        ]);
    }

    /**
     * Display notes for a project.
     *
     * @param Project  $project
     * @param NoteForm $form
     *
     * @return \Illuminate\View\View
     */
    public function getNotes(Project $project, NoteForm $form)
    {
        $notes = $project->notes()->with('createdBy')->get();

        return view('project.index', [

            'tabs'     => $this->projectMainViewTabs($project, 'notes', $notes),
            'project'  => $project,
            'active'   => 'notes',
            'notes'    => $notes,
            'sidebar'  => 'project',
            'noteForm' => $form,
        ]);
    }

    /**
     * @param Project $project
     * @param $view
     * @param null $data
     * @param bool $status
     *
     * @return array
     */
    protected function projectMainViewTabs(Project $project, $view, $data = null, $status = false)
    {
        $notesCount = $view === 'note' ? $data->count() : $project->notes()->count();


        $assignedIssuesCount = 0;
        if ($view !== 'assigned' && !$this->auth->guest()) {
            $assignedIssuesCount = $this->auth->user()->assignedIssuesCount($project->id);
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.
    }
}
        } elseif ($view === 'assigned') {
            $assignedIssuesCount = $data->count();

        }

        if ($view === 'issues') {
            if ($status == Issue::STATUS_OPEN) {
                $closedIssuesCount = $project->closedIssuesCount()->count();
                $openIssuesCount   = $data->count();

            } else {
                $closedIssuesCount = $data->count();

                $openIssuesCount   = $project->openIssuesCount()->count();
            }
        } else {
            $openIssuesCount   = $project->openIssuesCount()->count();
            $closedIssuesCount = $project->closedIssuesCount()->count();
        }

        $tabs   = [];
        $tabs[] = [
            'url'  => $project->to(),
            'page' => 'activity',
        ];
        $tabs[] = [
            'url'    => $project->to('issues'),
            'page'   => 'open_issue',
            'prefix' => $openIssuesCount,
        ];
        $tabs[] = [
            'url'    => $project->to('issues') . '/0',
            'page'   => 'closed_issue',
            'prefix' => $closedIssuesCount,
        ];
        if (!$this->auth->guest()) {
            $tabs[] = [
                'url'    => $project->to('assigned'),
                'page'   => 'issue_assigned_to_you',
                'prefix' => $assignedIssuesCount,
            ];
        }
        $tabs[] = [
            'url'    => $project->to('notes'),
            'page'   => 'notes',
            'prefix' => $notesCount,
        ];

        return $tabs;
    }

    /**
     * Edit the project.
     *
     * @param Project $project
     * @param Form    $form
     *
     * @return \Illuminate\View\View
     */
    public function getEdit(Project $project, Form $form)
    {
        return view('project.edit', [

            'form'    => $form,
            'project' => $project,
            'sidebar' => 'project',
        ]);
    }

    /**
     * To update project details.
     *
     * @param Project             $project
     * @param FormRequest\Project $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function postEdit(Project $project, FormRequest\Project $request)
    {
        // Delete the project
        if ($request->has('delete-project')) {
            $project->delete();

            return redirect('projects')
                ->with('notice', trans('tinyissue.project_has_been_deleted'));
        }

        $project->update($request->all());

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

    /**
     * Ajax: returns list of users that are not in the project.
     *
     * @param Project $project
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function getInactiveUsers(Project $project = null)
    {
        $users = $project->usersNotIn();
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}

        return response()->json($users);
    }

    /**
     * Ajax: add user to the project.
     *
     * @param Project $project
     * @param Request $request
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function postAssign(Project $project, Request $request)

    {
        $status = false;
        if ($request->has('user_id')) {
            $project->assignUser((int) $request->input('user_id'));
            $status = true;
        }

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

    /**
     * Ajax: remove user from the project.
     *
     * @param Project $project
     * @param Request $request
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function postUnassign(Project $project, Request $request)

    {
        $status = false;
        if ($request->has('user_id')) {
            $project->unassignUser((int) $request->input('user_id'));
            $status = true;
        }

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

    /**
     * To add a new note to the project.
     *
     * @param Project          $project
     * @param Note             $note
     * @param FormRequest\Note $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function postAddNote(Project $project, Note $note, FormRequest\Note $request)
    {
        $note->setRelation('project', $project);
        $note->setRelation('createdBy', $this->auth->user());
        $note->createNote($request->all());

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

    /**
     * Ajax: To update project note.
     *
     * @param Project $project
     * @param Note    $note
     * @param Request $request
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function postEditNote(Project $project, Project\Note $note, Request $request)
    {
        $body = '';
        if ($request->has('body')) {
            $note->setRelation('project', $project);
            $note->body = $request->input('body');
            $note->save();
            $body = \Html::format($note->body);
        }

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

    /**
     * Ajax: to delete a project note.
     *
     * @param Project $project
     * @param Note    $note
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function getDeleteNote(Project $project, Project\Note $note)

    {
        $note->delete();

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

    /**
     * Ajax: generate the issues export file.
     *
     * @param Project  $project
     * @param Exporter $exporter
     * @param Request  $request
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function postExportIssues(Project $project, Exporter $exporter, Request $request)
    {
        // Generate export file
        $info = $exporter->exportFile(
            'Project\Issue',
            $request->input('format', Exporter::TYPE_CSV),

            $request->all()
        );

        // Download link
        $link = link_to(
            $project->to('download_export/' . $info['file']),
            trans('tinyissue.download_export'),
            ['class' => 'btn btn-link']
        );

        return response()->json([
            'link'  => $link,
            'title' => $info['title'],
            'file'  => $info['file'],
            'ext'   => $info['ext'],
        ]);
    }

    /**
     * Download and then delete an export file.
     *
     * @param Project $project
     * @param string  $file
     *
     * @return \Symfony\Component\HttpFoundation\BinaryFileResponse
     */
    public function getDownloadExport(Project $project, $file)

    {
        // Filter out any characters that are not in pattern
        $file = preg_replace('/[^a-z0-9\_\.]/mi', '', $file);

        // Download export
        return response()->download(storage_path('exports/' . $file), $file)->deleteFileAfterSend(true);
    }
}


Push — master ( 91f8d9...64265e )

ProjectController::getDownloadExport() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

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\FilterIssue as FilterForm;
16			use Tinyissue\Form\Note as NoteForm;
17			use Tinyissue\Form\Project as Form;
18			use Tinyissue\Http\Requests\FormRequest;
19			use Tinyissue\Model\Project;
20			use Tinyissue\Model\Project\Issue;
21			use Tinyissue\Model\Project\Note;
22			use Tinyissue\Services\Exporter;
23
24			/**
25			* ProjectController is the controller class for managing request related to a project.
26			*
27			* @author Mohamed Alsharaf <[email protected]>
28			*/
29			class ProjectController extends Controller
30			{
31			/**
32			* Display activity for a project.
33			*
34			* @param Project $project
35			*
36			* @return \Illuminate\View\View
37			*/
38	11		public function getIndex(Project $project)
39			{
40	11		$activities = $project->activities()
41	11		->with('activity', 'issue', 'user', 'assignTo', 'comment', 'note')
42	11		->orderBy('created_at', 'DESC')
43	11		->take(10)
44	11		->get();
45
46	11		return view('project.index', [
			0 ignored issues – show Bug Compatibility introduced 2016-02-06 09:03 UTC by Report Bug Copy Issue Report The expression `view('project.index', ar...idebar' => 'project'));` of type `Illuminate\View\View\|Ill...\Contracts\View\Factory` adds the type `Illuminate\Contracts\View\Factory` to the return on line 46 which is incompatible with the return type documented by `Tinyissue\Http\Controlle...ectController::getIndex` of type `Illuminate\View\View`. Loading history...
47	11		'tabs' => $this->projectMainViewTabs($project, 'index'),
48	11		'project' => $project,
49	11		'active' => 'activity',
50	11		'activities' => $activities,
51	11		'sidebar' => 'project',
52			]);
53			}
54
55			/**
56			* Display issues for a project.
57			*
58			* @param FilterForm $filterForm
59			* @param Request $request
60			* @param Project $project
61			* @param int $status
62			*
63			* @return \Illuminate\View\View
64			*/
65	2		public function getIssues(FilterForm $filterForm, Request $request, Project $project, $status = Issue::STATUS_OPEN)
66			{
67	2		$active = $status == Issue::STATUS_OPEN ? 'open_issue' : 'closed_issue';
68	2		$issues = $project->listIssues($status, $request->all());
69
70	2		return view('project.index', [
			0 ignored issues – show Bug Compatibility introduced 2016-02-06 09:03 UTC by Report Bug Copy Issue Report The expression `view('project.index', ar...Form' => $filterForm));` of type `Illuminate\View\View\|Ill...\Contracts\View\Factory` adds the type `Illuminate\Contracts\View\Factory` to the return on line 70 which is incompatible with the return type documented by `Tinyissue\Http\Controlle...ctController::getIssues` of type `Illuminate\View\View`. Loading history...
71	2		'tabs' => $this->projectMainViewTabs($project, 'issues', $issues, $status),
			0 ignored issues – show Documentation introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report `$status` is of type `integer`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
72	2		'project' => $project,
73	2		'active' => $active,
74	2		'issues' => $issues,
75	2		'sidebar' => 'project',
76	2		'filterForm' => $filterForm,
77			]);
78			}
79
80			/**
81			* Display issues assigned to current user for a project.
82			*
83			* @param Project $project
84			*
85			* @return \Illuminate\View\View
86			*/
87	1		public function getAssigned(Project $project)
88			{
89	1		$issues = $project->listAssignedIssues($this->auth->user()->id);
			0 ignored issues – show Bug introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report Accessing `id` on the interface `Illuminate\Contracts\Auth\Authenticatable` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
90
91	1		return view('project.index', [
			0 ignored issues – show Bug Compatibility introduced 2016-02-06 09:03 UTC by Report Bug Copy Issue Report The expression `view('project.index', ar...idebar' => 'project'));` of type `Illuminate\View\View\|Ill...\Contracts\View\Factory` adds the type `Illuminate\Contracts\View\Factory` to the return on line 91 which is incompatible with the return type documented by `Tinyissue\Http\Controlle...Controller::getAssigned` of type `Illuminate\View\View`. Loading history...
92	1		'tabs' => $this->projectMainViewTabs($project, 'assigned', $issues),
93	1		'project' => $project,
94	1		'active' => 'issue_assigned_to_you',
95	1		'issues' => $issues,
96	1		'sidebar' => 'project',
97			]);
98			}
99
100			/**
101			* Display notes for a project.
102			*
103			* @param Project $project
104			* @param NoteForm $form
105			*
106			* @return \Illuminate\View\View
107			*/
108	7		public function getNotes(Project $project, NoteForm $form)
109			{
110	7		$notes = $project->notes()->with('createdBy')->get();
111
112	7		return view('project.index', [
			0 ignored issues – show Bug Compatibility introduced 2016-02-06 09:03 UTC by Report Bug Copy Issue Report The expression `view('project.index', ar... 'noteForm' => $form));` of type `Illuminate\View\View\|Ill...\Contracts\View\Factory` adds the type `Illuminate\Contracts\View\Factory` to the return on line 112 which is incompatible with the return type documented by `Tinyissue\Http\Controlle...ectController::getNotes` of type `Illuminate\View\View`. Loading history...
113	7		'tabs' => $this->projectMainViewTabs($project, 'notes', $notes),
114	7		'project' => $project,
115	7		'active' => 'notes',
116	7		'notes' => $notes,
117	7		'sidebar' => 'project',
118	7		'noteForm' => $form,
119			]);
120			}
121
122			/**
123			* @param Project $project
124			* @param $view
125			* @param null $data
126			* @param bool $status
127			*
128			* @return array
129			*/
130	21		protected function projectMainViewTabs(Project $project, $view, $data = null, $status = false)
131			{
132	21		$notesCount = $view === 'note' ? $data->count() : $project->notes()->count();
			0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report The method `count` cannot be called on `$data` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
133
134	21		$assignedIssuesCount = 0;
135	21		if ($view !== 'assigned' && !$this->auth->guest()) {
136	19		$assignedIssuesCount = $this->auth->user()->assignedIssuesCount($project->id);
			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 `assignedIssuesCount()` 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...
137	2		} elseif ($view === 'assigned') {
138	1		$assignedIssuesCount = $data->count();
			0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report The method `count` cannot be called on `$data` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
139			}
140
141	21		if ($view === 'issues') {
142	2		if ($status == Issue::STATUS_OPEN) {
143	2		$closedIssuesCount = $project->closedIssuesCount()->count();
144	2		$openIssuesCount = $data->count();
			0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report The method `count` cannot be called on `$data` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
145			} else {
146	1		$closedIssuesCount = $data->count();
			0 ignored issues – show Bug introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report The method `count` cannot be called on `$data` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
147	2		$openIssuesCount = $project->openIssuesCount()->count();
148			}
149			} else {
150	19		$openIssuesCount = $project->openIssuesCount()->count();
151	19		$closedIssuesCount = $project->closedIssuesCount()->count();
152			}
153
154	21		$tabs = [];
155	21		$tabs[] = [
156	21		'url' => $project->to(),
157	21		'page' => 'activity',
158			];
159	21		$tabs[] = [
160	21		'url' => $project->to('issues'),
161	21		'page' => 'open_issue',
162	21		'prefix' => $openIssuesCount,
163			];
164	21		$tabs[] = [
165	21		'url' => $project->to('issues') . '/0',
166	21		'page' => 'closed_issue',
167	21		'prefix' => $closedIssuesCount,
168			];
169	21		if (!$this->auth->guest()) {
170	20		$tabs[] = [
171	20		'url' => $project->to('assigned'),
172	20		'page' => 'issue_assigned_to_you',
173	20		'prefix' => $assignedIssuesCount,
174			];
175			}
176	21		$tabs[] = [
177	21		'url' => $project->to('notes'),
178	21		'page' => 'notes',
179	21		'prefix' => $notesCount,
180			];
181
182	21		return $tabs;
183			}
184
185			/**
186			* Edit the project.
187			*
188			* @param Project $project
189			* @param Form $form
190			*
191			* @return \Illuminate\View\View
192			*/
193	2		public function getEdit(Project $project, Form $form)
194			{
195	2		return view('project.edit', [
			0 ignored issues – show Bug Compatibility introduced 2016-02-06 09:03 UTC by Report Bug Copy Issue Report The expression `view('project.edit', arr...idebar' => 'project'));` of type `Illuminate\View\View\|Ill...\Contracts\View\Factory` adds the type `Illuminate\Contracts\View\Factory` to the return on line 195 which is incompatible with the return type documented by `Tinyissue\Http\Controlle...jectController::getEdit` of type `Illuminate\View\View`. Loading history...
196	2		'form' => $form,
197	2		'project' => $project,
198	2		'sidebar' => 'project',
199			]);
200			}
201
202			/**
203			* To update project details.
204			*
205			* @param Project $project
206			* @param FormRequest\Project $request
207			*
208			* @return \Illuminate\Http\RedirectResponse
209			*/
210	2		public function postEdit(Project $project, FormRequest\Project $request)
211			{
212			// Delete the project
213	2		if ($request->has('delete-project')) {
214	1		$project->delete();
215
216	1		return redirect('projects')
217	1		->with('notice', trans('tinyissue.project_has_been_deleted'));
218			}
219
220	1		$project->update($request->all());
221
222	1		return redirect($project->to())
223	1		->with('notice', trans('tinyissue.project_has_been_updated'));
224			}
225
226			/**
227			* Ajax: returns list of users that are not in the project.
228			*
229			* @param Project $project
230			*
231			* @return \Symfony\Component\HttpFoundation\Response
232			*/
233	1		public function getInactiveUsers(Project $project = null)
234			{
235	1		$users = $project->usersNotIn();
			0 ignored issues – show Bug introduced 2016-03-20 02:01 UTC by Report Bug Copy Issue Report It seems like `$project` is not always an object, but can also be of type `null`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
236
237	1		return response()->json($users);
238			}
239
240			/**
241			* Ajax: add user to the project.
242			*
243			* @param Project $project
244			* @param Request $request
245			*
246			* @return \Symfony\Component\HttpFoundation\Response
247			*/
248	1	View Code Duplication	public function postAssign(Project $project, Request $request)
			0 ignored issues – show Duplication introduced 2015-12-15 16:27 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...
249			{
250	1		$status = false;
251	1		if ($request->has('user_id')) {
252	1		$project->assignUser((int) $request->input('user_id'));
253	1		$status = true;
254			}
255
256	1		return response()->json(['status' => $status]);
257			}
258
259			/**
260			* Ajax: remove user from the project.
261			*
262			* @param Project $project
263			* @param Request $request
264			*
265			* @return \Symfony\Component\HttpFoundation\Response
266			*/
267	1	View Code Duplication	public function postUnassign(Project $project, Request $request)
			0 ignored issues – show Duplication introduced 2015-12-15 16:27 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...
268			{
269	1		$status = false;
270	1		if ($request->has('user_id')) {
271	1		$project->unassignUser((int) $request->input('user_id'));
272	1		$status = true;
273			}
274
275	1		return response()->json(['status' => $status]);
276			}
277
278			/**
279			* To add a new note to the project.
280			*
281			* @param Project $project
282			* @param Note $note
283			* @param FormRequest\Note $request
284			*
285			* @return \Illuminate\Http\RedirectResponse
286			*/
287	2		public function postAddNote(Project $project, Note $note, FormRequest\Note $request)
288			{
289	2		$note->setRelation('project', $project);
290	2		$note->setRelation('createdBy', $this->auth->user());
291	2		$note->createNote($request->all());
292
293	2		return redirect($note->to())->with('notice', trans('tinyissue.your_note_added'));
294			}
295
296			/**
297			* Ajax: To update project note.
298			*
299			* @param Project $project
300			* @param Note $note
301			* @param Request $request
302			*
303			* @return \Symfony\Component\HttpFoundation\Response
304			*/
305	1		public function postEditNote(Project $project, Project\Note $note, Request $request)
306			{
307	1		$body = '';
308	1		if ($request->has('body')) {
309	1		$note->setRelation('project', $project);
310	1		$note->body = $request->input('body');
311	1		$note->save();
312	1		$body = \Html::format($note->body);
313			}
314
315	1		return response()->json(['status' => true, 'text' => $body]);
316			}
317
318			/**
319			* Ajax: to delete a project note.
320			*
321			* @param Project $project
322			* @param Note $note
323			*
324			* @return \Symfony\Component\HttpFoundation\Response
325			*/
326	1		public function getDeleteNote(Project $project, Project\Note $note)
			0 ignored issues – show Unused Code introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report The parameter `$project` 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...
327			{
328	1		$note->delete();
329
330	1		return response()->json(['status' => true]);
331			}
332
333			/**
334			* Ajax: generate the issues export file.
335			*
336			* @param Project $project
337			* @param Exporter $exporter
338			* @param Request $request
339			*
340			* @return \Symfony\Component\HttpFoundation\Response
341			*/
342	4		public function postExportIssues(Project $project, Exporter $exporter, Request $request)
343			{
344			// Generate export file
345	4		$info = $exporter->exportFile(
346	4		'Project\Issue',
347	4		$request->input('format', Exporter::TYPE_CSV),
			0 ignored issues – show Bug introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report It seems like `$request->input('format'...ces\Exporter::TYPE_CSV)` targeting `Illuminate\Http\Request::input()` can also be of type `array`; however, `Tinyissue\Services\Exporter::exportFile()` does only seem to accept `string`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
348	4		$request->all()
349			);
350
351			// Download link
352	4		$link = link_to(
353	4		$project->to('download_export/' . $info['file']),
354	4		trans('tinyissue.download_export'),
355	4		['class' => 'btn btn-link']
356			);
357
358	4		return response()->json([
359	4		'link' => $link,
360	4		'title' => $info['title'],
361	4		'file' => $info['file'],
362	4		'ext' => $info['ext'],
363			]);
364			}
365
366			/**
367			* Download and then delete an export file.
368			*
369			* @param Project $project
370			* @param string $file
371			*
372			* @return \Symfony\Component\HttpFoundation\BinaryFileResponse
373			*/
374	4		public function getDownloadExport(Project $project, $file)
			0 ignored issues – show Unused Code introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report The parameter `$project` 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...
375			{
376			// Filter out any characters that are not in pattern
377	4		$file = preg_replace('/[^a-z0-9\_\.]/mi', '', $file);
378
379			// Download export
380	4		return response()->download(storage_path('exports/' . $file), $file)->deleteFileAfterSend(true);
381			}
382			}
383

satrun77 / tinyissue

Push — master ( 91f8d9...64265e )

ProjectController::getDownloadExport() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like