CODEWITHSUNDEEP
laravelstripe-payments
John Freedom
John Freedom

Reputation: 481

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.

Upvotes: 16

Views: 26272

Answers (4)

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

Reputation: 561

change

return view('payments.payment')

to

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

Upvotes: 37

Nayan Zala
Nayan Zala

Reputation: 61

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

Upvotes: 5

asghar
asghar

Reputation: 447

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

Upvotes: 3

Aken Roberts
Aken Roberts

Reputation: 13467

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

Upvotes: 13

Related Questions