ProfileUpdateController::processProfileUpdate() - Code Metrics - Inspection of "Moving frontend controllers to `Frontend` namespac..." - rinvex/laravel-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 03b333...573f2e )

by Abdelrahman

created 2016-09-04 23:28 UTC

ProfileUpdateController::processProfileUpdate() C

↳ Parent: ProfileUpdateController

Complexity

Conditions	8
Paths	16

Size

Total Lines	35
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	8
eloc	21
c	1
b	0
f	0
nc	16
nop	1
dl	0
loc	35
rs	5.3846

<?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\Frontend;

use Rinvex\Country\Models\Country;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Lang;
use Rinvex\Fort\Http\Requests\ProfileUpdate;
use Rinvex\Fort\Contracts\UserRepositoryContract;
use Rinvex\Fort\Http\Controllers\AbstractController;

class ProfileUpdateController extends AbstractController
{
    /**
     * The users repository instance.
     *
     * @var \Rinvex\Fort\Contracts\UserRepositoryContract
     */
    protected $users;

    /**
     * Create a new account controller instance.
     *
     * @param \Rinvex\Fort\Contracts\UserRepositoryContract $users
     *
     * @return void

     */
    public function __construct(UserRepositoryContract $users)
    {
        $this->users = $users;

        $this->middleware($this->getAuthMiddleware(), ['except' => $this->middlewareWhitelist]);
    }

    /**
     * Show the account update form.
     *
     * @return \Illuminate\Http\Response

     */
    public function showProfileUpdate(Country $country)
    {
        $twoFactor = $this->currentUser()->getTwoFactor();
        $countries = $country->findAll()->pluck('name.common', 'iso_3166_1_alpha2');

        return view('rinvex.fort::profile.page', compact('twoFactor', 'countries'));
    }

    /**
     * Process the account update form.
     *
     * @param \Rinvex\Fort\Http\Requests\ProfileUpdate $request
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function processProfileUpdate(ProfileUpdate $request)
    {
        $currentUser = $this->currentUser();
        $data        = $request->except(['_token', 'id']);
        $twoFactor   = $currentUser->getTwoFactor();

        if (isset($data['password'])) {
            $data['password'] = bcrypt($data['password']);
        }

        $emailVerification = $data['email'] != $currentUser->email ? [
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            'email_verified'    => false,
            'email_verified_at' => null,
        ] : [];

        $phoneVerification = $data['phone'] != $currentUser->phone ? [
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            'phone_verified'    => false,
            'phone_verified_at' => null,
        ] : [];

        $countryVerification = $data['country'] !== $currentUser->country;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

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

        if ($phoneVerification || $countryVerification) {

            array_set($twoFactor, 'phone.enabled', false);
        }

        $this->users->update($request->get('id'), $data + $emailVerification + $phoneVerification + $twoFactor);

        return intend([
            'back' => true,
            'with' => [
                          'rinvex.fort.alert.success' => Lang::get('rinvex.fort::message.account.'.(! empty($emailVerification) ? 'reverify' : 'updated')),

                      ] + ($twoFactor !== $currentUser->getTwoFactor() ? ['rinvex.fort.alert.warning' => Lang::get('rinvex.fort::message.verification.twofactor.phone.auto_disabled')] : []),

        ]);
    }

    /**
     * Get current user.
     *
     * @return \Rinvex\Fort\Contracts\AuthenticatableContract
     */
    protected function currentUser()
    {
        return Auth::guard($this->getGuard())->user();
    }
}


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\Frontend;
17
18			use Rinvex\Country\Models\Country;
19			use Illuminate\Support\Facades\Auth;
20			use Illuminate\Support\Facades\Lang;
21			use Rinvex\Fort\Http\Requests\ProfileUpdate;
22			use Rinvex\Fort\Contracts\UserRepositoryContract;
23			use Rinvex\Fort\Http\Controllers\AbstractController;
24
25			class ProfileUpdateController extends AbstractController
26			{
27			/**
28			* The users repository instance.
29			*
30			* @var \Rinvex\Fort\Contracts\UserRepositoryContract
31			*/
32			protected $users;
33
34			/**
35			* Create a new account controller instance.
36			*
37			* @param \Rinvex\Fort\Contracts\UserRepositoryContract $users
38			*
39			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-09-04 23:31 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
40			*/
41			public function __construct(UserRepositoryContract $users)
42			{
43			$this->users = $users;
44
45			$this->middleware($this->getAuthMiddleware(), ['except' => $this->middlewareWhitelist]);
46			}
47
48			/**
49			* Show the account update form.
50			*
51			* @return \Illuminate\Http\Response
			0 ignored issues – show Documentation introduced 2016-09-03 21:04 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...
52			*/
53			public function showProfileUpdate(Country $country)
54			{
55			$twoFactor = $this->currentUser()->getTwoFactor();
56			$countries = $country->findAll()->pluck('name.common', 'iso_3166_1_alpha2');
57
58			return view('rinvex.fort::profile.page', compact('twoFactor', 'countries'));
59			}
60
61			/**
62			* Process the account update form.
63			*
64			* @param \Rinvex\Fort\Http\Requests\ProfileUpdate $request
65			*
66			* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
67			*/
68			public function processProfileUpdate(ProfileUpdate $request)
69			{
70			$currentUser = $this->currentUser();
71			$data = $request->except(['_token', 'id']);
72			$twoFactor = $currentUser->getTwoFactor();
73
74			if (isset($data['password'])) {
75			$data['password'] = bcrypt($data['password']);
76			}
77
78			$emailVerification = $data['email'] != $currentUser->email ? [
			0 ignored issues – show Bug introduced 2016-09-04 23:31 UTC by Report Bug Copy Issue Report Accessing `email` on the interface `Rinvex\Fort\Contracts\AuthenticatableContract` 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...
79			'email_verified' => false,
80			'email_verified_at' => null,
81			] : [];
82
83			$phoneVerification = $data['phone'] != $currentUser->phone ? [
			0 ignored issues – show Bug introduced 2016-09-04 23:31 UTC by Report Bug Copy Issue Report Accessing `phone` on the interface `Rinvex\Fort\Contracts\AuthenticatableContract` 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...
84			'phone_verified' => false,
85			'phone_verified_at' => null,
86			] : [];
87
88			$countryVerification = $data['country'] !== $currentUser->country;
			0 ignored issues – show Bug introduced 2016-09-04 23:31 UTC by Report Bug Copy Issue Report Accessing `country` on the interface `Rinvex\Fort\Contracts\AuthenticatableContract` 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...
89
90			if ($phoneVerification \|\| $countryVerification) {
			0 ignored issues – show Bug Best Practice introduced 2016-09-04 23:31 UTC by Report Bug Copy Issue Report The expression `$phoneVerification` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
91			array_set($twoFactor, 'phone.enabled', false);
92			}
93
94			$this->users->update($request->get('id'), $data + $emailVerification + $phoneVerification + $twoFactor);
95
96			return intend([
97			'back' => true,
98			'with' => [
99			'rinvex.fort.alert.success' => Lang::get('rinvex.fort::message.account.'.(! empty($emailVerification) ? 'reverify' : 'updated')),
			0 ignored issues – show Coding Style introduced 2016-09-04 23:31 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 155 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...
100			] + ($twoFactor !== $currentUser->getTwoFactor() ? ['rinvex.fort.alert.warning' => Lang::get('rinvex.fort::message.verification.twofactor.phone.auto_disabled')] : []),
			0 ignored issues – show Coding Style introduced 2016-08-25 14:01 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 189 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...
101			]);
102			}
103
104			/**
105			* Get current user.
106			*
107			* @return \Rinvex\Fort\Contracts\AuthenticatableContract
108			*/
109			protected function currentUser()
110			{
111			return Auth::guard($this->getGuard())->user();
112			}
113			}
114

rinvex / laravel-auth

Push — master ( 03b333...573f2e )

ProfileUpdateController::processProfileUpdate() C

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like