TaskManager\Http\Controllers\RegisterController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	store()	0	6	1
A	create()	0	3	1

<?php

namespace TaskManager\Http\Controllers;

use Illuminate\Http\Request;
use TaskManager\Http\Requests\RegisterForm;
use TaskManager\Mail\WelcomeNewUser;

class RegisterController extends Controller
{
    public function __construct()
    {
        $this->middleware('guest');
    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        return view('register');

The expression return view('register') returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request $request
     * @param RegisterForm $form
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request, RegisterForm $form)

The parameter $form is not used and could be removed.

    {
       $user = \TaskManager\User::create($request->only('email','name','password'));
       auth()->login($user);

The method login() does not exist on Illuminate\Contracts\Auth\Factory.

       \Mail::to($user)->send(new WelcomeNewUser($user));
       return redirect()->home();

The expression return redirect()->home() returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

    }

}