Call to a member function setCookie() on null

Question

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.

score 29 · Answer 1 · edited Nov 10 '20 at 17:47

29

change

return view('payments.payment')

to

return response()->view('payments.payment')

edited Nov 10 '20 at 17:47

buddemat

4,552
14
29
49

answered Nov 10 '20 at 15:54

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

481
5
6

2

In case of returning a `view`, this is the correct answer. – Pars May 06 '21 at 14:06
I had the same problem. I was returning a view. This solution worked for me. – Rupesh Bhurke Feb 05 '22 at 15:32

Aken Roberts · Answer 2 · 2018-11-29T19:47:20.597

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);
}

Note that trying to return a view from middleware will still cause the error. It is better to return a redirect response to the route that handles the view instead of trying to return the view itself: `return redirect()->route('route.to.payments.payment');` — newUserName02, Mar 20 '19 at 18:04

score 5 · Answer 3 · answered Apr 16 '20 at 10:07

5

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

answered Apr 16 '20 at 10:07

Nayan Zala

61
1
1

score 3 · Answer 4 · answered Mar 04 '20 at 12:46

3

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

answered Mar 04 '20 at 12:46

asghar

437
5
5

Call to a member function setCookie() on null

4 Answers4

Linked