Create an accessible variable in all controllers that use the same middleware

Asked

Viewed 116 times

0

I’m creating an EAD application, and I need all controllers that use Middleware Students to receive from the middleware itself, the variable with the user data

I tried to define the variable in the middleware, using $request->merge(['user'] => $user);

but when I gave dd($request->user); in the controller (in __Construct), it returned null

Controller:


namespace App\Http\Controllers;

use Illuminate\Support\Facades\Session;
use Illuminate\Http\Request;
use App\Helpers\Helper;

use App\Models\Students;

class StudentController extends Controller
{
    /** Currenct Student logged **/
    protected $information;

    public function __construct(Request $request)
    {
        $this->middleware(\App\Http\Middleware\Students::class);
        dd( $request->user ); // <- Isso retorna null, quando na verdade deveria retornar o user
        $this->information = $request->user;
    }

    public function index()
    {
        return view('dashboard.views.home', [
            'user' => $this->information
        ]);
    }
}

Middleware:

namespace App\Http\Middleware;

use Closure;
use App\Helpers\Helper;
use Illuminate\Support\Facades\Redirect;

class Students
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if($user = Helper::verifyOrGet()) {
            $request = $request->merge(['user' => $user]);
            return $next($request);
        } else {
            return Redirect::to('login')->with('msg', '1');
        }
    }
}

The curious thing is that if I give a dd($request->user); in the middleware itself, it returns me the normal user, but in the controller no, has anyone already gone through it? Is there any solution?

php laravel framework

  • Do you want to get the user logged in? Why not use the Auth::user() within its functions and outside the constructor ? It’s just a hint.

  • 2

    You don’t have to do any of that using Auth::user() if you are logged in returns the user ...

1 answer

0

You don’t need to create a middleware for this.

If this route is only accessed by authenticated users just have to check it with the native middleware of the Laravel.

To recover user according to documentation you can use $request->user()

Your controller should look like:

namespace App\Http\Controllers;

use Illuminate\Http\Request;

class StudentController extends Controller
{
    public function __construct()
    {
        $this->middleware('auth');
    }

    public function index(Request $request)
    {
        return view('dashboard.views.home', [
            'user' => $request->user()
        ]);
    }
}

Note: Use this practice of setting the middleware in the controller constructor only if you have not set the middleware in the route statement.

Browser other questions tagged php laravel framework

You are not signed in. Login or sign up in order to post.