Trying to get property of non-object in middleware - laravel 5

Asked Apr 09 '15 at 12:10

Active Apr 09 '15 at 12:10

Viewed 41 times

I have this middleware:

public function handle($request, Closure $next)
{
    $response = $next($request);

    $user = $this->auth->getUser(); 
    if ($user->role_id == 1) {
        return $response;
    } else {
        return response('Unauthorized.', 401);
    }
}

i want to use this middleware when call profile page and the code in route is:

Route::get('profile', ['middleware' => 'superAdmin', 'uses' => 'users\UserController@index']);

and I added it to $routeMiddleware array in kernel.php

when call the page:

http://localhost:8080/public/profile

got this error:

ErrorException in SuperAdminRoleMiddleware.php line 43: Trying to get property of non-object

line 43 is: if ($user->role_id == 1) {

what is the issue:

asked Apr 09 '15 at 12:10

Ayman Hussein

3,817
7
28
48

Where are you getting `$this->auth->getUser()` from? The Auth class in Laravel provides `Auth::user()` – Joe Apr 09 '15 at 12:15
I have already included Auth at top of file - use Auth; – Ayman Hussein Apr 09 '15 at 12:17
That still doesn't explain where $this->auth->getUser() comes from. – Joe Apr 09 '15 at 12:21
I am using it in another middleware and works fine it is a method in auth class. – Ayman Hussein Apr 09 '15 at 12:23
in this middleware works fine public function handle($request, Closure $next) { $response = $next($request); $user = $this->auth->getUser(); if ($user->status == 1) { return Redirect::to('/home'); } else { Auth::logout(); return Redirect::to('/auth/login')->withInput($request->only('username', 'remember')) ->withErrors([ 'username' => 'Your account still NOT active', ]); } return $response; } – Ayman Hussein Apr 09 '15 at 12:23
`var_dump($user)` - what value does it hold? – deceze Apr 09 '15 at 12:25
$user object is null – Ayman Hussein Apr 09 '15 at 12:29
Then figure out why `$this->auth->getUser()` returns `null`. If necessary rephrase your question into this direction. – deceze Apr 09 '15 at 12:31
but i used another middleware like this and it's work, may be somthing not included – Ayman Hussein Apr 09 '15 at 12:34
this middleware (status) works fine: Route::post('/auth/login', ['middleware' => 'status', 'uses' => 'Auth\AuthController@postLogin']); – Ayman Hussein Apr 09 '15 at 12:35
Post the entire middleware so we can see what's happening – Matthew Brown Apr 09 '15 at 13:02
"It works just fine elsewhere" is not helping anyone solve this. It doesn't work *here*, figure out why. – deceze Apr 09 '15 at 13:04

Trying to get property of non-object in middleware - laravel 5

0 Answers0