AccountController::update() - Code Metrics - wingsline/laravel-blog - Measure and Improve Code Quality continuously with Scrutinizer

AccountController::update() A
last analyzed 2020-09-08 17:53 UTC

↳ Parent: AccountController

Complexity

Conditions	3
Paths	4

Size

Total Lines	19
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	11
nc	4
nop	1
dl	0
loc	19
c	1
b	0
f	0
cc	3
rs	9.9

<?php

namespace Wingsline\Blog\Http\Controllers;

use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Routing\Controller as BaseController;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash;
use Wingsline\Blog\Http\Requests\AccountEditRequest;

class AccountController extends BaseController
{
    use AuthorizesRequests;
    use DispatchesJobs;
    use ValidatesRequests;

    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function edit()
    {
        $user = Auth::user();

        return view('blog::account.edit', compact('user'));
    }

    public function update(AccountEditRequest $request)
    {
        $user = Auth::user();
        $validated = $request->validated();
        $user->fill([
            'name' => $validated['name'],
            'email' => $validated['email'],
        ]);

        if (null !== $validated['password']) {
            $user->password = Hash::make($validated['password']);

        }

        $user->save();
        if (\count($user->getChanges())) {
            flash()->success('Account updated.');
        }

        return redirect()->route('admin.account.edit');
    }
}


1			<?php
2
3			namespace Wingsline\Blog\Http\Controllers;
4
5			use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
6			use Illuminate\Foundation\Bus\DispatchesJobs;
7			use Illuminate\Foundation\Validation\ValidatesRequests;
8			use Illuminate\Routing\Controller as BaseController;
9			use Illuminate\Support\Facades\Auth;
10			use Illuminate\Support\Facades\Hash;
11			use Wingsline\Blog\Http\Requests\AccountEditRequest;
12
13			class AccountController extends BaseController
14			{
15			use AuthorizesRequests;
16			use DispatchesJobs;
17			use ValidatesRequests;
18
19			/**
20			* @return \Illuminate\Contracts\View\Factory\|\Illuminate\View\View
21			*/
22			public function edit()
23			{
24			$user = Auth::user();
25
26			return view('blog::account.edit', compact('user'));
27			}
28
29			public function update(AccountEditRequest $request)
30			{
31			$user = Auth::user();
32			$validated = $request->validated();
33			$user->fill([
34			'name' => $validated['name'],
35			'email' => $validated['email'],
36			]);
37
38			if (null !== $validated['password']) {
39			$user->password = Hash::make($validated['password']);
			0 ignored issues – show Bug introduced 2019-11-25 17:57 UTC by Report Bug Copy Issue Report Accessing `password` on the interface `Illuminate\Contracts\Auth\Authenticatable` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
40			}
41
42			$user->save();
43			if (\count($user->getChanges())) {
44			flash()->success('Account updated.');
45			}
46
47			return redirect()->route('admin.account.edit');
48			}
49			}
50

wingsline / laravel-blog

AccountController::update() A last analyzed 2020-09-08 17:53 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

AccountController::update() A
last analyzed 2020-09-08 17:53 UTC