PasswordController::sendPasswordResetLink() - Code Metrics - Inspection of "Upgrade to Laravel 5.4" - AudithSoftworks/Basis - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#47)

by Şəhriyar

created 2017-04-15 12:29 UTC

PasswordController::sendPasswordResetLink() B

↳ Parent: Project

Complexity

Conditions	5
Paths	5

Size

Total Lines	29
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	5.0061

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	29
ccs	15
cts	16
cp	0.9375
rs	8.439
c	1
b	0
f	0
cc	5
eloc	16
nc	5
nop	1
crap	5.0061

<?php namespace App\Http\Controllers\Auth;

use App\Events\Users\RequestedResetPasswordLink;
use App\Events\Users\ResetPassword;
use App\Exceptions\Common\ValidationException;
use App\Exceptions\Users\TokenNotValidException;
use App\Http\Controllers\Controller;
use App\Models\User;
use App\Notifications\ResetPasswordNotification;
use Illuminate\Contracts\Auth\PasswordBroker;
use Illuminate\Database\Eloquent\ModelNotFoundException;
use Illuminate\Foundation\Auth\RedirectsUsers;
use Illuminate\Http\Request;

class PasswordController extends Controller
{
    use RedirectsUsers;

    /**
     * Create a new password controller instance.
     */
    public function __construct()
    {
        $this->middleware('guest');
    }

    /**
     * Display the form to request a password reset link.
     *
     * @return \Illuminate\View\View|\Illuminate\Contracts\View\Factory
     */
    public function requestPasswordResetLink()
    {
        return view('password/email');
    }

    /**
     * Send a password reset link to the given email's owner, via email.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     * @throws \App\Exceptions\Common\ValidationException
     */
    public function sendPasswordResetLink(Request $request)
    {
        $validator = app('validator')->make($request->all(), [
            'email' => 'required|email|max:255'
        ]);
        if ($validator->fails()) {
            throw new ValidationException($validator);
        }

        $user = User::whereEmail($request->only('email'))->first();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        if (is_null($user)) {
            throw new ModelNotFoundException(trans('passwords.user'));
        }

        $user->notify(new ResetPasswordNotification($token = app('auth.password.broker')->createToken($user)));

        event(new RequestedResetPasswordLink($user));

        if ($request->expectsJson()) {
            $response = ['message' => trans('passwords.sent')];
            if (env('APP_ENV') == 'testing') {
                $response['token'] = $token;
            }

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

        return redirect()->back()->with('message', trans('passwords.sent'));
    }

    /**
     * Display the password reset view for the given token.
     *
     * @param \Illuminate\Http\Request $request
     * @param string                   $token
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\View\View|\Illuminate\Contracts\View\Factory
     */
    public function showPasswordResetForm(Request $request, $token = null)
    {
        if (is_null($token)) {
            if ($request->expectsJson()) {
                throw new TokenNotValidException();
            }

            return view('password/reset')->withErrors(['token' => trans(PasswordBroker::INVALID_TOKEN)]);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        }

        if ($request->expectsJson()) {
            return response()->json(['token' => $token]);
        }

        return view('password/reset')->with('token', $token);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }

    /**
     * Reset the password through password-reset-token and email provided.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     * @throws \App\Exceptions\Common\ValidationException
     */
    public function resetPassword(Request $request)
    {
        $validator = app('validator')->make($request->all(), [
            'token' => 'required|string',
            'email' => 'required|email|max:255',
            'password' => 'required|confirmed|min:' . app('config')->get('auth.passwords.users.min_length')
        ]);
        if ($validator->fails()) {
            throw new ValidationException($validator);
        }

        $credentials = $request->only('email', 'password', 'password_confirmation', 'token');

        $passwordBroker = app('auth.password.broker');
        $response = $passwordBroker->reset(
            $credentials, function (User $user, $password) {
            $user->password = app('hash')->make($password);
            $user->save();
            app('auth.driver')->login($user);
        });

        switch ($response) {
            case $passwordBroker::INVALID_USER:
                throw new ModelNotFoundException(trans($response));
                break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
            case $passwordBroker::INVALID_TOKEN:
                throw new TokenNotValidException(trans($response));
                break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
        }

        event(new ResetPassword(app('auth.driver')->user()));

        if ($request->expectsJson()) {
            return response()->json(['message' => trans('passwords.reset')]);
        }

        return redirect($this->redirectPath())->with('message', trans('passwords.reset'));
    }
}


1		<?php namespace App\Http\Controllers\Auth;
2
3		use App\Events\Users\RequestedResetPasswordLink;
4		use App\Events\Users\ResetPassword;
5		use App\Exceptions\Common\ValidationException;
6		use App\Exceptions\Users\TokenNotValidException;
7		use App\Http\Controllers\Controller;
8		use App\Models\User;
9		use App\Notifications\ResetPasswordNotification;
10		use Illuminate\Contracts\Auth\PasswordBroker;
11		use Illuminate\Database\Eloquent\ModelNotFoundException;
12		use Illuminate\Foundation\Auth\RedirectsUsers;
13		use Illuminate\Http\Request;
14
15		class PasswordController extends Controller
16		{
17		use RedirectsUsers;
18
19		/**
20		* Create a new password controller instance.
21		*/
22	4	public function __construct()
23		{
24	4	$this->middleware('guest');
25	4	}
26
27		/**
28		* Display the form to request a password reset link.
29		*
30		* @return \Illuminate\View\View\|\Illuminate\Contracts\View\Factory
31		*/
32		public function requestPasswordResetLink()
33		{
34		return view('password/email');
35		}
36
37		/**
38		* Send a password reset link to the given email's owner, via email.
39		*
40		* @param \Illuminate\Http\Request $request
41		*
42		* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
43		* @throws \App\Exceptions\Common\ValidationException
44		*/
45	2	public function sendPasswordResetLink(Request $request)
46		{
47	2	$validator = app('validator')->make($request->all(), [
48		'email' => 'required\|email\|max:255'
49	2	]);
50	2	if ($validator->fails()) {
51	1	throw new ValidationException($validator);
52		}
53
54	2	$user = User::whereEmail($request->only('email'))->first();
		0 ignored issues – show Bug introduced 2016-09-14 12:56 UTC by Report Bug Copy Issue Report The method `first` does only exist in `Illuminate\Database\Query\Builder`, but not in `App\Models\User`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
55	2	if (is_null($user)) {
56	1	throw new ModelNotFoundException(trans('passwords.user'));
57		}
58
59	1	$user->notify(new ResetPasswordNotification($token = app('auth.password.broker')->createToken($user)));
60
61	1	event(new RequestedResetPasswordLink($user));
62
63	1	if ($request->expectsJson()) {
64	1	$response = ['message' => trans('passwords.sent')];
65	1	if (env('APP_ENV') == 'testing') {
66	1	$response['token'] = $token;
67		}
68
69	1	return response()->json($response);
70		}
71
72		return redirect()->back()->with('message', trans('passwords.sent'));
73		}
74
75		/**
76		* Display the password reset view for the given token.
77		*
78		* @param \Illuminate\Http\Request $request
79		* @param string $token
80		*
81		* @return \Illuminate\Http\JsonResponse\|\Illuminate\View\View\|\Illuminate\Contracts\View\Factory
82		*/
83		public function showPasswordResetForm(Request $request, $token = null)
84		{
85		if (is_null($token)) {
86		if ($request->expectsJson()) {
87		throw new TokenNotValidException();
88		}
89
90		return view('password/reset')->withErrors(['token' => trans(PasswordBroker::INVALID_TOKEN)]);
		0 ignored issues – show Bug introduced 2016-09-14 12:56 UTC by Report Bug Copy Issue Report The method `withErrors` does only exist in `Illuminate\View\View`, but not in `Illuminate\Contracts\View\Factory`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
91		}
92
93		if ($request->expectsJson()) {
94		return response()->json(['token' => $token]);
95		}
96
97		return view('password/reset')->with('token', $token);
		0 ignored issues – show Bug introduced 2016-09-14 12:56 UTC by Report Bug Copy Issue Report The method `with` does only exist in `Illuminate\View\View`, but not in `Illuminate\Contracts\View\Factory`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
98		}
99
100		/**
101		* Reset the password through password-reset-token and email provided.
102		*
103		* @param \Illuminate\Http\Request $request
104		*
105		* @return \Illuminate\Http\JsonResponse\|\Illuminate\Http\RedirectResponse
106		* @throws \App\Exceptions\Common\ValidationException
107		*/
108	2	public function resetPassword(Request $request)
109		{
110	2	$validator = app('validator')->make($request->all(), [
111	2	'token' => 'required\|string',
112	2	'email' => 'required\|email\|max:255',
113	2	'password' => 'required\|confirmed\|min:' . app('config')->get('auth.passwords.users.min_length')
114		]);
115	2	if ($validator->fails()) {
116	1	throw new ValidationException($validator);
117		}
118
119	2	$credentials = $request->only('email', 'password', 'password_confirmation', 'token');
120
121	2	$passwordBroker = app('auth.password.broker');
122	2	$response = $passwordBroker->reset(
123	2	$credentials, function (User $user, $password) {
124	1	$user->password = app('hash')->make($password);
125	1	$user->save();
126	1	app('auth.driver')->login($user);
127	2	});
128
129		switch ($response) {
130	2	case $passwordBroker::INVALID_USER:
131	1	throw new ModelNotFoundException(trans($response));
132		break;
		0 ignored issues – show Unused Code introduced 2017-02-15 08:46 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
133	2	case $passwordBroker::INVALID_TOKEN:
134	1	throw new TokenNotValidException(trans($response));
135		break;
		0 ignored issues – show Unused Code introduced 2017-02-15 08:46 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
136		}
137
138	1	event(new ResetPassword(app('auth.driver')->user()));
139
140	1	if ($request->expectsJson()) {
141	1	return response()->json(['message' => trans('passwords.reset')]);
142		}
143
144		return redirect($this->redirectPath())->with('message', trans('passwords.reset'));
145		}
146		}
147

AudithSoftworks / Basis

Pull Request — master (#47)

PasswordController::sendPasswordResetLink() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like