Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Call to a member function setCookie() on null

I am trying to finish this middleware in larval that checks to make sure is subscribed to a subscription plan. If the user is not it redirects to the payment page.

public function handle($request, Closure $next)
{
    if(Auth::check()){
        if (Auth::user()->subscribed('main')) {
            return true;
        }else{
            return view('payments.payment')->with('user',Auth::user());
        }
    }else{
        abort(403, 'Unauthorized action.');
    }
    return $next($request);
}

I am getting this error with not much luck finding a solution Call to a member function setCookie() on null.

like image 970
John Freedom Avatar asked Feb 25 '18 04:02

John Freedom


4 Answers

change

return view('payments.payment')

to

return response()->view('payments.payment')
like image 106
عبدالرزاق أوكسا Avatar answered Nov 17 '22 15:11

عبدالرزاق أوكسا


i solve by redirect to other routereturn redirect()->to('route');

like image 23
asghar Avatar answered Nov 17 '22 15:11

asghar


The problem is where you are returning true. A middleware should return a response-style object, not a boolean.

Since that is your "good" path and you want to proceed with your application logic, you should replace return true; with return $next($request);

public function handle($request, Closure $next)
{
    if(Auth::check()){
        if (Auth::user()->subscribed('main')) {
            return $next($request);
        }else{
            return view('payments.payment')->with('user',Auth::user());
        }
    }else{
        abort(403, 'Unauthorized action.');
    }
}

On an unrelated recommendation, you can clean up your conditional logic a bit to make your code easier to read/follow:

public function handle($request, Closure $next)
{
    // If the user is not logged in, respond with a 403 error.
    if ( ! Auth::check()) {
        abort(403, 'Unauthorized action.');
    }

    // If the user is not subscribed, show a different payments page.
    if ( ! Auth::user()->subscribed('main')) {
        return view('payments.payment')->with('user',Auth::user());
    }

    // The user is subscribed; continue with the request.
    return $next($request);
}
like image 30
Aken Roberts Avatar answered Nov 17 '22 16:11

Aken Roberts


return response(view('payments.payment')->with('user',Auth::user()));

like image 27
Nayan Zala Avatar answered Nov 17 '22 17:11

Nayan Zala