UsersController::destroy() - Code Metrics - Inspection of "Refactor both cortex/fort and rinvex/fort" - rinvex/cortex-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( cc9a49...c1329f )

by Abdelrahman

created 2018-02-11 18:33 UTC

UsersController::destroy() A

↳ Parent: UsersController

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

declare(strict_types=1);

namespace Cortex\Fort\Http\Controllers\Managerarea;

use Illuminate\Http\Request;
use Cortex\Fort\Models\User;
use Illuminate\Foundation\Http\FormRequest;
use Cortex\Foundation\DataTables\LogsDataTable;
use Cortex\Foundation\DataTables\ActivitiesDataTable;
use Cortex\Fort\DataTables\Managerarea\UsersDataTable;
use Cortex\Fort\Http\Requests\Managerarea\UserFormRequest;
use Cortex\Foundation\Http\Controllers\AuthorizedController;
use Cortex\Fort\Http\Requests\Managerarea\UserAttributesFormRequest;

class UsersController extends AuthorizedController
{
    /**
     * {@inheritdoc}
     */
    protected $resource = 'user';

    /**
     * List all users.
     *
     * @param \Cortex\Fort\DataTables\Managerarea\UsersDataTable $usersDataTable
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\View\View
     */
    public function index(UsersDataTable $usersDataTable)
    {
        return $usersDataTable->with([
            'id' => 'managerarea-users-index-table',
            'phrase' => trans('cortex/fort::common.users'),
        ])->render('cortex/tenants::managerarea.pages.datatable');
    }

    /**
     * List user logs.
     *
     * @param \Cortex\Fort\Models\User                    $user
     * @param \Cortex\Foundation\DataTables\LogsDataTable $logsDataTable
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse

     */
    public function logs(User $user, LogsDataTable $logsDataTable)
    {
        return $logsDataTable->with([
            'resource' => $user,
            'tabs' => 'managerarea.users.tabs',
            'phrase' => trans('cortex/fort::common.users'),
            'id' => "managerarea-users-{$user->getKey()}-logs-table",
        ])->render('cortex/tenants::managerarea.pages.datatable-logs');
    }

    /**
     * Get a listing of the resource activities.
     *
     * @param \Cortex\Fort\Models\User                          $user
     * @param \Cortex\Foundation\DataTables\ActivitiesDataTable $activitiesDataTable
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse

     */
    public function activities(User $user, ActivitiesDataTable $activitiesDataTable)
    {
        return $activitiesDataTable->with([
            'resource' => $user,
            'tabs' => 'managerarea.users.tabs',
            'phrase' => trans('cortex/fort::common.users'),
            'id' => "managerarea-users-{$user->getKey()}-activities-table",
        ])->render('cortex/tenants::managerarea.pages.datatable-logs');
    }

    /**
     * Show the form for create/update of the given resource attributes.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Cortex\Fort\Models\User $user
     *
     * @return \Illuminate\View\View
     */
    public function attributes(Request $request, User $user)

    {
        return view('cortex/fort::managerarea.pages.user-attributes', compact('user'));
    }

    /**
     * Process the account update form.
     *
     * @param \Cortex\Fort\Http\Requests\Managerarea\UserAttributesFormRequest $request
     * @param \Cortex\Fort\Models\User                                         $user
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function updateAttributes(UserAttributesFormRequest $request, User $user)
    {
        $data = $request->validated();

        // Update profile
        $user->fill($data)->save();

        return intend([
            'back' => true,
            'with' => ['success' => trans('cortex/fort::messages.account.updated_attributes')],
        ]);
    }

    /**
     * Create new user.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Cortex\Fort\Models\User $user
     *
     * @return \Illuminate\View\View
     */
    public function create(Request $request, User $user)
    {
        return $this->form($request, $user);
    }

    /**
     * Edit given user.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Cortex\Fort\Models\User $user
     *
     * @return \Illuminate\View\View
     */
    public function edit(Request $request, User $user)
    {
        return $this->form($request, $user);
    }

    /**
     * Show user create/edit form.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Cortex\Fort\Models\User $user
     *
     * @return \Illuminate\View\View
     */
    protected function form(Request $request, User $user)
    {
        $countries = collect(countries())->map(function ($country, $code) {
            return [
                'id' => $code,
                'text' => $country['name'],
                'emoji' => $country['emoji'],
            ];
        })->values();
        $currentUser = $request->user($this->getGuard());
        $languages = collect(languages())->pluck('name', 'iso_639_1');
        $genders = ['male' => trans('cortex/fort::common.male'), 'female' => trans('cortex/fort::common.female')];

        $roles = $currentUser->can('superadmin')
            ? app('cortex.fort.role')->all()->pluck('name', 'id')->toArray()
            : $currentUser->roles->pluck('name', 'id')->toArray();

        $abilities = $currentUser->can('superadmin')
            ? app('cortex.fort.ability')->all()->pluck('title', 'id')->toArray()
            : $currentUser->abilities->pluck('title', 'id')->toArray();

        return view('cortex/fort::managerarea.pages.user', compact('user', 'abilities', 'roles', 'countries', 'languages', 'genders'));

    }

    /**
     * Store new user.
     *
     * @param \Cortex\Fort\Http\Requests\Managerarea\UserFormRequest $request
     * @param \Cortex\Fort\Models\User                             $user
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function store(UserFormRequest $request, User $user)
    {
        return $this->process($request, $user);
    }

    /**
     * Update given user.
     *
     * @param \Cortex\Fort\Http\Requests\Managerarea\UserFormRequest $request
     * @param \Cortex\Fort\Models\User                             $user
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function update(UserFormRequest $request, User $user)
    {
        return $this->process($request, $user);
    }

    /**
     * Process stored/updated user.
     *
     * @param \Illuminate\Foundation\Http\FormRequest $request
     * @param \Cortex\Fort\Models\User                $user
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    protected function process(FormRequest $request, User $user)
    {
        // Prepare required input fields
        $data = $request->validated();

        ! $request->hasFile('profile_picture')
        || $user->addMediaFromRequest('profile_picture')
                ->sanitizingFileName(function ($fileName) {
                    return md5($fileName).'.'.pathinfo($fileName, PATHINFO_EXTENSION);
                })
                ->toMediaCollection('profile_picture', config('cortex.fort.media.disk'));

        ! $request->hasFile('cover_photo')
        || $user->addMediaFromRequest('cover_photo')
                ->sanitizingFileName(function ($fileName) {
                    return md5($fileName).'.'.pathinfo($fileName, PATHINFO_EXTENSION);
                })
                ->toMediaCollection('cover_photo', config('cortex.fort.media.disk'));

        // Save user
        $user->fill($data)->save();

        return intend([
            'url' => route('managerarea.users.index'),
            'with' => ['success' => trans('cortex/foundation::messages.resource_saved', ['resource' => 'user', 'id' => $user->username])],

        ]);
    }

    /**
     * Destroy given user.
     *
     * @param \Cortex\Fort\Models\User $user
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function destroy(User $user)
    {
        $user->delete();

        return intend([
            'url' => route('managerarea.users.index'),
            'with' => ['warning' => trans('cortex/foundation::messages.resource_deleted', ['resource' => 'user', 'id' => $user->username])],

        ]);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Cortex\Fort\Http\Controllers\Managerarea;
6
7			use Illuminate\Http\Request;
8			use Cortex\Fort\Models\User;
9			use Illuminate\Foundation\Http\FormRequest;
10			use Cortex\Foundation\DataTables\LogsDataTable;
11			use Cortex\Foundation\DataTables\ActivitiesDataTable;
12			use Cortex\Fort\DataTables\Managerarea\UsersDataTable;
13			use Cortex\Fort\Http\Requests\Managerarea\UserFormRequest;
14			use Cortex\Foundation\Http\Controllers\AuthorizedController;
15			use Cortex\Fort\Http\Requests\Managerarea\UserAttributesFormRequest;
16
17			class UsersController extends AuthorizedController
18			{
19			/**
20			* {@inheritdoc}
21			*/
22			protected $resource = 'user';
23
24			/**
25			* List all users.
26			*
27			* @param \Cortex\Fort\DataTables\Managerarea\UsersDataTable $usersDataTable
28			*
29			* @return \Illuminate\Http\JsonResponse\|\Illuminate\View\View
30			*/
31			public function index(UsersDataTable $usersDataTable)
32			{
33			return $usersDataTable->with([
34			'id' => 'managerarea-users-index-table',
35			'phrase' => trans('cortex/fort::common.users'),
36			])->render('cortex/tenants::managerarea.pages.datatable');
37			}
38
39			/**
40			* List user logs.
41			*
42			* @param \Cortex\Fort\Models\User $user
43			* @param \Cortex\Foundation\DataTables\LogsDataTable $logsDataTable
44			*
45			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
			0 ignored issues – show Documentation introduced 2018-01-31 14:25 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\Http\JsonRes...e\|\Illuminate\View\View`? 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...
46			*/
47			public function logs(User $user, LogsDataTable $logsDataTable)
48			{
49			return $logsDataTable->with([
50			'resource' => $user,
51			'tabs' => 'managerarea.users.tabs',
52			'phrase' => trans('cortex/fort::common.users'),
53			'id' => "managerarea-users-{$user->getKey()}-logs-table",
54			])->render('cortex/tenants::managerarea.pages.datatable-logs');
55			}
56
57			/**
58			* Get a listing of the resource activities.
59			*
60			* @param \Cortex\Fort\Models\User $user
61			* @param \Cortex\Foundation\DataTables\ActivitiesDataTable $activitiesDataTable
62			*
63			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
			0 ignored issues – show Documentation introduced 2018-01-31 14:25 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\Http\JsonRes...e\|\Illuminate\View\View`? 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...
64			*/
65			public function activities(User $user, ActivitiesDataTable $activitiesDataTable)
66			{
67			return $activitiesDataTable->with([
68			'resource' => $user,
69			'tabs' => 'managerarea.users.tabs',
70			'phrase' => trans('cortex/fort::common.users'),
71			'id' => "managerarea-users-{$user->getKey()}-activities-table",
72			])->render('cortex/tenants::managerarea.pages.datatable-logs');
73			}
74
75			/**
76			* Show the form for create/update of the given resource attributes.
77			*
78			* @param \Illuminate\Http\Request $request
79			* @param \Cortex\Fort\Models\User $user
80			*
81			* @return \Illuminate\View\View
82			*/
83			public function attributes(Request $request, User $user)
			0 ignored issues – show Unused Code introduced 2018-01-31 14:25 UTC by Report Bug Copy Issue Report The parameter `$request` 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...
84			{
85			return view('cortex/fort::managerarea.pages.user-attributes', compact('user'));
86			}
87
88			/**
89			* Process the account update form.
90			*
91			* @param \Cortex\Fort\Http\Requests\Managerarea\UserAttributesFormRequest $request
92			* @param \Cortex\Fort\Models\User $user
93			*
94			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
95			*/
96			public function updateAttributes(UserAttributesFormRequest $request, User $user)
97			{
98			$data = $request->validated();
99
100			// Update profile
101			$user->fill($data)->save();
102
103			return intend([
104			'back' => true,
105			'with' => ['success' => trans('cortex/fort::messages.account.updated_attributes')],
106			]);
107			}
108
109			/**
110			* Create new user.
111			*
112			* @param \Illuminate\Http\Request $request
113			* @param \Cortex\Fort\Models\User $user
114			*
115			* @return \Illuminate\View\View
116			*/
117			public function create(Request $request, User $user)
118			{
119			return $this->form($request, $user);
120			}
121
122			/**
123			* Edit given user.
124			*
125			* @param \Illuminate\Http\Request $request
126			* @param \Cortex\Fort\Models\User $user
127			*
128			* @return \Illuminate\View\View
129			*/
130			public function edit(Request $request, User $user)
131			{
132			return $this->form($request, $user);
133			}
134
135			/**
136			* Show user create/edit form.
137			*
138			* @param \Illuminate\Http\Request $request
139			* @param \Cortex\Fort\Models\User $user
140			*
141			* @return \Illuminate\View\View
142			*/
143			protected function form(Request $request, User $user)
144			{
145			$countries = collect(countries())->map(function ($country, $code) {
146			return [
147			'id' => $code,
148			'text' => $country['name'],
149			'emoji' => $country['emoji'],
150			];
151			})->values();
152			$currentUser = $request->user($this->getGuard());
153			$languages = collect(languages())->pluck('name', 'iso_639_1');
154			$genders = ['male' => trans('cortex/fort::common.male'), 'female' => trans('cortex/fort::common.female')];
155
156			$roles = $currentUser->can('superadmin')
157			? app('cortex.fort.role')->all()->pluck('name', 'id')->toArray()
158			: $currentUser->roles->pluck('name', 'id')->toArray();
159
160			$abilities = $currentUser->can('superadmin')
161			? app('cortex.fort.ability')->all()->pluck('title', 'id')->toArray()
162			: $currentUser->abilities->pluck('title', 'id')->toArray();
163
164			return view('cortex/fort::managerarea.pages.user', compact('user', 'abilities', 'roles', 'countries', 'languages', 'genders'));
			0 ignored issues – show Coding Style introduced 2018-01-31 14:25 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 135 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
165			}
166
167			/**
168			* Store new user.
169			*
170			* @param \Cortex\Fort\Http\Requests\Managerarea\UserFormRequest $request
171			* @param \Cortex\Fort\Models\User $user
172			*
173			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
174			*/
175			public function store(UserFormRequest $request, User $user)
176			{
177			return $this->process($request, $user);
178			}
179
180			/**
181			* Update given user.
182			*
183			* @param \Cortex\Fort\Http\Requests\Managerarea\UserFormRequest $request
184			* @param \Cortex\Fort\Models\User $user
185			*
186			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
187			*/
188			public function update(UserFormRequest $request, User $user)
189			{
190			return $this->process($request, $user);
191			}
192
193			/**
194			* Process stored/updated user.
195			*
196			* @param \Illuminate\Foundation\Http\FormRequest $request
197			* @param \Cortex\Fort\Models\User $user
198			*
199			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
200			*/
201			protected function process(FormRequest $request, User $user)
202			{
203			// Prepare required input fields
204			$data = $request->validated();
205
206			! $request->hasFile('profile_picture')
207			\|\| $user->addMediaFromRequest('profile_picture')
208			->sanitizingFileName(function ($fileName) {
209			return md5($fileName).'.'.pathinfo($fileName, PATHINFO_EXTENSION);
210			})
211			->toMediaCollection('profile_picture', config('cortex.fort.media.disk'));
212
213			! $request->hasFile('cover_photo')
214			\|\| $user->addMediaFromRequest('cover_photo')
215			->sanitizingFileName(function ($fileName) {
216			return md5($fileName).'.'.pathinfo($fileName, PATHINFO_EXTENSION);
217			})
218			->toMediaCollection('cover_photo', config('cortex.fort.media.disk'));
219
220			// Save user
221			$user->fill($data)->save();
222
223			return intend([
224			'url' => route('managerarea.users.index'),
225			'with' => ['success' => trans('cortex/foundation::messages.resource_saved', ['resource' => 'user', 'id' => $user->username])],
			0 ignored issues – show Coding Style introduced 2018-02-11 18:43 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 138 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
226			]);
227			}
228
229			/**
230			* Destroy given user.
231			*
232			* @param \Cortex\Fort\Models\User $user
233			*
234			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
235			*/
236			public function destroy(User $user)
237			{
238			$user->delete();
239
240			return intend([
241			'url' => route('managerarea.users.index'),
242			'with' => ['warning' => trans('cortex/foundation::messages.resource_deleted', ['resource' => 'user', 'id' => $user->username])],
			0 ignored issues – show Coding Style introduced 2018-02-11 18:43 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 140 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
243			]);
244			}
245			}
246

rinvex / cortex-auth

Push — develop ( cc9a49...c1329f )

UsersController::destroy() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like