Backpack\Base\app\Http\Controllers\Auth\ResetPasswordController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	66
Duplicated Lines	16.67 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	11	11	3
A	guard()	0	8	2
A	showResetForm()	0	8	1

<?php

namespace Backpack\Base\app\Http\Controllers\Auth;

use Backpack\Base\app\Http\Controllers\Controller;
use Illuminate\Foundation\Auth\ResetsPasswords;
use Illuminate\Http\Request;

class ResetPasswordController extends Controller
{
    protected $data = []; // the information we send to the view

    /*
    |--------------------------------------------------------------------------
    | Password Reset Controller
    |--------------------------------------------------------------------------
    |
    | This controller is responsible for handling password reset requests
    | and uses a simple trait to include this behavior. You're free to
    | explore this trait and override any methods you wish to tweak.
    |
    */

    use ResetsPasswords;

    /**
     * Create a new controller instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()

This method seems to be duplicated in your project.

    {
        if (config('backpack.base.separate_admin_session')) {
            $this->middleware('guest:'.config('backpack.base.admin_guard.name'), ['except' => 'logout']);
        } else {
            $this->middleware('guest');
        }

        // where to redirect after password was reset
        $this->redirectTo = property_exists($this, 'redirectTo') ? $this->redirectTo : config('backpack.base.route_prefix', 'admin').'/dashboard';

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

    }

    public function guard()
    {
        if (config('backpack.base.separate_admin_session')) {
            return \Auth::guard(config('backpack.base.admin_guard.name'));
        } else {
            return \Auth::guard();
        }
    }

    // -------------------------------------------------------
    // Laravel overwrites for loading backpack views
    // -------------------------------------------------------

    /**
     * Display the password reset view for the given token.
     *
     * If no token is present, display the link request form.
     *
     * @param \Illuminate\Http\Request $request
     * @param string|null              $token
     *
     * @return \Illuminate\Http\Response
     */
    public function showResetForm(Request $request, $token = null)
    {
        $this->data['title'] = trans('backpack::base.reset_password'); // set the page title

        return view('backpack::auth.passwords.reset', $this->data)->with(

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

            ['token' => $token, 'email' => $request->email]
        );
    }
}