Laravel constructor redirect is not working?

Question

I have a controller with several methods and I need to add a specific authorization check. If authorization failed then redirect login page. So for this reason i have created one private function and this function call in constructor.

class AdminController extends Controller
{
    public function __construct()
    {
        $this->middleware('web');
        $this->isLogin();
    }

    private function isLogin()
    {
        if (!empty(Auth::user())) {
            echo "Hello";
        } else {
            echo "Fasd";
            return Redirect::to('/login');
        }
    }
}

If auth is not found is does not redirect to login. What i write extra code for this?

You should batter to use middleware: [link](https://laravel.com/docs/master/middleware) [link](http://stackoverflow.com/questions/32164695/using-laravel-auth-middleware) — Rahman Qaiser, Jun 04 '16 at 11:25
I don't want use middleware. Can you suggest me what i write wrong in my code can please help me @RahmanQaiser — sandip kakade, Jun 04 '16 at 11:28
Let us [continue this discussion in chat](http://chat.stackoverflow.com/rooms/113803/discussion-between-dhanashri-and-rahman-qaiser). — sandip kakade, Jun 04 '16 at 11:50
I am using this return redirect('/login'); but not redirect to login — sandip kakade, Jun 04 '16 at 11:53
To check if user is logged in use `Auth::check()` or `Auth::guest()` to check if is not logged in. — Pyton, Jun 04 '16 at 11:59

score 4 · Accepted Answer · answered Jun 04 '16 at 11:55

4

Doing login page redirect use redirect::route into login page alias name routes.php.

public function isLogin()
{
    if (!empty(Auth::user())) {
        echo "Hello";
    } else {
        echo "Fasd";
        return Redirect::route('login');
    }
}

answered Jun 04 '16 at 11:55

sebahattin çatal

241
2
5

getting error Route [login] not defined. @sebahattin – sandip kakade Jun 04 '16 at 12:02
The route method here is used to redirect to [named route](https://laravel.com/docs/5.2/routing#named-routes). So make sure to name it first. See [`Illuminate\Routing\Redirector`](https://laravel.com/api/5.2/Illuminate/Routing/Redirector.html#method_route) – Chay22 Jun 04 '16 at 12:15
Route::get('/login', ['as' => 'login', 'uses' => 'Auth\AuthController@login']); – sebahattin çatal Jun 04 '16 at 12:44
@sebahattinçatal Thanks:) – sandip kakade Jun 22 '16 at 10:54

Laravel constructor redirect is not working?

1 Answers1