RolesController::delete() - Code Metrics - Inspection of "Merge branch 'release/v0.0.3'" - rinvex/laravel-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f9e78a...edb43e )

by Abdelrahman

created 2017-02-11 17:09 UTC

RolesController::delete() A

↳ Parent: RolesController

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	5
nc	1
nop	1
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0

<?php

/*
 * NOTICE OF LICENSE
 *
 * Part of the Rinvex Fort Package.
 *
 * This source file is subject to The MIT License (MIT)
 * that is bundled with this package in the LICENSE file.
 *
 * Package: Rinvex Fort Package
 * License: The MIT License (MIT)
 * Link:    https://rinvex.com
 */

namespace Rinvex\Fort\Http\Controllers\Backend;

use Illuminate\Http\Request;
use Rinvex\Fort\Models\Role;
use Rinvex\Fort\Models\Ability;
use Rinvex\Fort\Http\Controllers\AuthorizedController;

class RolesController extends AuthorizedController
{
    /**
     * {@inheritdoc}
     */
    protected $resource = 'roles';

    /**
     * {@inheritdoc}
     */
    protected $resourceActionWhitelist = ['assign'];


    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response

     */
    public function index()
    {
        $roles = Role::paginate(config('rinvex.fort.backend.items_per_page'));

        return view('rinvex/fort::backend/roles.index', compact('roles'));
    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response

     */
    public function create()
    {
        return $this->form('create', 'store', new Role());
    }

    /**
     * Show the form for editing the given resource.
     *
     * @param \Rinvex\Fort\Models\Role $role
     *
     * @return \Illuminate\Http\Response

     */
    public function edit(Role $role)
    {
        return $this->form('edit', 'update', $role);
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\Response

     */
    public function store(Request $request)
    {
        return $this->process($request, new Role());
    }

    /**
     * Update the given resource in storage.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Rinvex\Fort\Models\Role $role
     *
     * @return \Illuminate\Http\Response

     */
    public function update(Request $request, Role $role)
    {
        return $this->process($request, $role);
    }

    /**
     * Delete the given resource from storage.
     *
     * @param \Rinvex\Fort\Models\Role $role
     *
     * @return \Illuminate\Http\Response

     */
    public function delete(Role $role)
    {
        $role->delete();

        return intend([
            'route' => 'rinvex.fort.backend.roles.index',
            'with'  => ['warning' => trans('rinvex/fort::messages.role.deleted', ['roleId' => $role->id])],
        ]);
    }

    /**
     * Show the form for create/update of the given resource.
     *
     * @param string                   $mode
     * @param string                   $action
     * @param \Rinvex\Fort\Models\Role $role
     *
     * @return \Illuminate\Http\Response

     */
    protected function form($mode, $action, Role $role)
    {
        $abilityList = Ability::all()->groupBy('resource')->map(function ($ability) {
            return $ability->pluck('name', 'id');
        })->toArray();

        return view('rinvex/fort::backend/roles.form', compact('role', 'abilityList', 'mode', 'action'));
    }

    /**
     * Process the form for store/update of the given resource.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Rinvex\Fort\Models\Role $role
     *
     * @return \Illuminate\Http\Response

     */
    protected function process(Request $request, Role $role)
    {
        // Prepare required input fields
        $input = $request->all();

        // Save role
        ! $role->exists ? $role = $role->create($input) : $role->update($input);

        // Sync abilities
        if ($request->user($this->getGuard())->can('grant-abilities')) {
            $role->abilities()->sync((array) array_pull($input, 'abilityList'));
        }

        return intend([
            'route' => 'rinvex.fort.backend.roles.index',
            'with'  => ['success' => trans('rinvex/fort::messages.role.saved', ['roleId' => $role->id])],
        ]);
    }
}


1			<?php
2
3			/*
4			* NOTICE OF LICENSE
5			*
6			* Part of the Rinvex Fort Package.
7			*
8			* This source file is subject to The MIT License (MIT)
9			* that is bundled with this package in the LICENSE file.
10			*
11			* Package: Rinvex Fort Package
12			* License: The MIT License (MIT)
13			* Link: https://rinvex.com
14			*/
15
16			namespace Rinvex\Fort\Http\Controllers\Backend;
17
18			use Illuminate\Http\Request;
19			use Rinvex\Fort\Models\Role;
20			use Rinvex\Fort\Models\Ability;
21			use Rinvex\Fort\Http\Controllers\AuthorizedController;
22
23			class RolesController extends AuthorizedController
24			{
25			/**
26			* {@inheritdoc}
27			*/
28			protected $resource = 'roles';
29
30			/**
31			* {@inheritdoc}
32			*/
33			protected $resourceActionWhitelist = ['assign'];
			0 ignored issues – show Comprehensibility Naming introduced 2016-10-24 15:12 UTC by Report Bug Copy Issue Report The variable name `$resourceActionWhitelist` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
34
35			/**
36			* Display a listing of the resource.
37			*
38			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\View\View\|\I...\Contracts\View\Factory`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
39			*/
40			public function index()
41			{
42			$roles = Role::paginate(config('rinvex.fort.backend.items_per_page'));
43
44			return view('rinvex/fort::backend/roles.index', compact('roles'));
45			}
46
47			/**
48			* Show the form for creating a new resource.
49			*
50			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\View\View\|\I...\Contracts\View\Factory`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
51			*/
52			public function create()
53			{
54			return $this->form('create', 'store', new Role());
55			}
56
57			/**
58			* Show the form for editing the given resource.
59			*
60			* @param \Rinvex\Fort\Models\Role $role
61			*
62			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\View\View\|\I...\Contracts\View\Factory`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
63			*/
64			public function edit(Role $role)
65			{
66			return $this->form('edit', 'update', $role);
67			}
68
69			/**
70			* Store a newly created resource in storage.
71			*
72			* @param \Illuminate\Http\Request $request
73			*
74			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\Http\Redirec...inate\Http\JsonResponse`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
75			*/
76			public function store(Request $request)
77			{
78			return $this->process($request, new Role());
79			}
80
81			/**
82			* Update the given resource in storage.
83			*
84			* @param \Illuminate\Http\Request $request
85			* @param \Rinvex\Fort\Models\Role $role
86			*
87			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\Http\Redirec...inate\Http\JsonResponse`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
88			*/
89			public function update(Request $request, Role $role)
90			{
91			return $this->process($request, $role);
92			}
93
94			/**
95			* Delete the given resource from storage.
96			*
97			* @param \Rinvex\Fort\Models\Role $role
98			*
99			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\Http\Redirec...inate\Http\JsonResponse`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
100			*/
101			public function delete(Role $role)
102			{
103			$role->delete();
104
105			return intend([
106			'route' => 'rinvex.fort.backend.roles.index',
107			'with' => ['warning' => trans('rinvex/fort::messages.role.deleted', ['roleId' => $role->id])],
108			]);
109			}
110
111			/**
112			* Show the form for create/update of the given resource.
113			*
114			* @param string $mode
115			* @param string $action
116			* @param \Rinvex\Fort\Models\Role $role
117			*
118			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\View\View\|\I...\Contracts\View\Factory`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
119			*/
120			protected function form($mode, $action, Role $role)
121			{
122			$abilityList = Ability::all()->groupBy('resource')->map(function ($ability) {
123			return $ability->pluck('name', 'id');
124			})->toArray();
125
126			return view('rinvex/fort::backend/roles.form', compact('role', 'abilityList', 'mode', 'action'));
127			}
128
129			/**
130			* Process the form for store/update of the given resource.
131			*
132			* @param \Illuminate\Http\Request $request
133			* @param \Rinvex\Fort\Models\Role $role
134			*
135			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2017-02-11 17:18 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\Http\Redirec...inate\Http\JsonResponse`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
136			*/
137			protected function process(Request $request, Role $role)
138			{
139			// Prepare required input fields
140			$input = $request->all();
141
142			// Save role
143			! $role->exists ? $role = $role->create($input) : $role->update($input);
144
145			// Sync abilities
146			if ($request->user($this->getGuard())->can('grant-abilities')) {
147			$role->abilities()->sync((array) array_pull($input, 'abilityList'));
148			}
149
150			return intend([
151			'route' => 'rinvex.fort.backend.roles.index',
152			'with' => ['success' => trans('rinvex/fort::messages.role.saved', ['roleId' => $role->id])],
153			]);
154			}
155			}
156

rinvex / laravel-auth

Push — master ( f9e78a...edb43e )

RolesController::delete() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like