Use of undefined constant login - assumed 'login' (this will throw an Error in a future version of PHP) when trying to logout

Question

when ever i want to logout, it always show "Use of undefined constant login - assumed 'login'" error. what did i do wrong?

web.php

route::get('/logout','AuthController@logout')->name('logout');

route::get('/','PagesController@home')->name('home');

AuthController.php

public function getLogin(){
        return view('login');
    }

    public function postLogin(Request $request){
       if(!\Auth::attempt(['email' => $request->email, 'password' => $request->password])){
        return redirect()->back();
       }
        return redirect()->route('home');
    }

public function logout(){
        \Auth::logout();
        return redirect()->route(login);
    }

PagesController.php

public function home()
    {
        return view('index') ;
    }

main.blade.php

<a href="{{route('logout')}}">logout</a>

`return redirect()->route(login);` <- you've missed the `'` marks around `login` so technically you're referencing the constant `login` rather than a string `'login'` - and you get the warning (in future it'll be an error). — CD001, Nov 28 '19 at 10:24
Easily done - sometimes it just takes a fresh pair of eyes to look at it. — CD001, Nov 28 '19 at 10:28

score 4 · Accepted Answer · answered Nov 28 '19 at 10:25

4

Put login in qoutation:

public function logout(){
        \Auth::logout();
        return redirect()->route('login');
    }

answered Nov 28 '19 at 10:25

Rouhollah Mazarei

3,969
1
14
20

score 3 · Answer 2 · answered Nov 28 '19 at 10:25

As per your code you need to use quote for it on the below line while using login, so if you do not provide quote it is assuming as constant. And also there should be a route for it in your route file with the name login as per your question that route also do not exists yet

public function logout(){
        \Auth::logout();
        return redirect()->route('login');
    }

score 3 · Answer 3 · answered Nov 28 '19 at 10:31

3

public function logout(){
        \Auth::logout();
        return redirect()->route(login);
    }

change To :

public function logout(){
        \Auth::logout();
        return redirect()->route('login');
    }

answered Nov 28 '19 at 10:31

Ramki

452
2
16

Use of undefined constant login - assumed 'login' (this will throw an Error in a future version of PHP) when trying to logout

3 Answers3