App\Http\Controllers\Users\PasswordController

Complexity

Total Complexity

19

Size/Duplication

Total Lines	103
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

48.28%

Importance

Changes	16
Bugs	4	Features	9

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	redirectPath()	0	8	3
A	requestPasswordResetLink()	0	8	3
A	sendPasswordResetLink()	0	10	3
B	showPasswordResetForm()	0	16	6
A	resetPassword()	0	10	3

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

use App\Contracts\Registrar;
use App\Exceptions\Users\TokenNotValidException;
use App\Http\Controllers\Controller;
use Illuminate\Contracts\Auth\PasswordBroker;
use Illuminate\Http\Request;

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

    /**
     * Display the form to request a password reset link.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\View\View|\Illuminate\Contracts\View\Factory
     */
    public function requestPasswordResetLink(Request $request)
    {
        if ($request->ajax() || $request->wantsJson()) {
            return response()->json([]);
        }

        return view('password/email');
    }

    /**
     * Send a password reset link to the given email's owner, via email.
     *
     * @param \Illuminate\Http\Request $request
     * @param \App\Contracts\Registrar $registrar
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function sendPasswordResetLink(Request $request, Registrar $registrar)
    {
        $registrar->sendResetPasswordLinkViaEmail();

        if ($request->ajax() || $request->wantsJson()) {
            return response()->json(['message' => trans('passwords.sent')]);
        }

        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->ajax() || $request->wantsJson()) {
                throw new TokenNotValidException();
            }

            return view('password/reset')->withErrors(['token' => trans(PasswordBroker::INVALID_TOKEN)]);

The method withErrors does only exist in Illuminate\View\View, but not in Illuminate\Contracts\View\Factory.

        }

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

        return view('password/reset')->with('token', $token);

The method with does only exist in Illuminate\View\View, but not in Illuminate\Contracts\View\Factory.

    }

    /**
     * Reset the password through password-reset-token and email provided.
     *
     * @param \Illuminate\Http\Request $request
     * @param \App\Contracts\Registrar $registrar
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function resetPassword(Request $request, Registrar $registrar)
    {
        $registrar->resetPassword();

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

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

    /**
     * Get the post-register/-login redirect path.
     *
     * @return string
     */
    private function redirectPath()
    {
        if (isset($this->redirectPath)) {
            return $this->redirectPath;

The property redirectPath does not exist. Did you maybe forget to declare it?

        }

        return isset($this->redirectTo) ? $this->redirectTo : '/login';

The property redirectTo does not exist. Did you maybe forget to declare it?

    }
}