Auth::login($user) in laravel not able to login the user

All we need is an easy explanation of the problem, so here it is.

I am using laravel 5.4 and Auth::login($user) is showing Type error:

Argument 1 passed to Illuminate\Auth\SessionGuard::login() must
implement interface Illuminate\Contracts\Auth\Authenticatable,
instance of Illuminate\Database\Eloquent\Builder given, called in
/home/vendor/laravel/framework/src/Illuminate/Auth/AuthManager.php on
line 294

My User.php file is:

namespace App;

use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;
class User extends Authenticatable
{
use Notifiable;
/**
 * The attributes that are mass assignable.
 *
 * @var array
 */
protected $fillable = [
    'name', 'email', 'role', 'password',
];

/**
 * The attributes that should be hidden for arrays.
 *
 * @var array
 */
protected $hidden = [
    'password', 'remember_token',
];
}

My AuthController.php is:

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\User;
use Auth;
use Illuminate\Support\Facades\Input;
use \Hash;

class AuthController extends Controller
{
//
public function register(Request $request){
    $this->validate($request, [
        'name' => 'required|max:30',
        'email' => 'required|email|unique:users',
        'regpassword' => 'required|alpha_num|confirmed'
        ]);

    /*$user = new User();
    $user['name'] = $request['name'];
    $user['email'] = $request['email'];
    $user['password'] = bcrypt($request['password']);
    $user['role'] = 'user';
    $user->save();
    */
    $user = User::create(array(
        'email' => Input::get('email'),
        'name' => Input::get('name'),
        'password' => Hash::make(Input::get('password')),
        'role' => 'user'));
    return 'success';


 }

public function userLogin(Request $request){
    $this->validate($request,[
        'email' => 'required|email',
        'password' => 'required' 
        ]);

    $user = User::where('email', '=', $request['email'])-> where('password' ,'=', Hash::make($request['password']))->where('role','=','user');                                                    

    if($user){
        Auth::login($user); 
        return $next($request);
    }
    else{
        return redirect()->back();
    }


}
}

My web.php

Route::get('/', function () {
return view('welcome');
});

Auth::routes();

Route::get('/home', '[email protected]');
Route::post('/register', '[email protected]');
Route::post('/UserLogin','[email protected]');

How to solve :

I know you bored from this bug, So we are here to help you! Take a deep breath and look at the explanation of your problem. We have many solutions to this problem, But we recommend you to use the first method because it is tested & true method that will 100% work for you.

Method 1

Your problem is this:

$user = User::where('email', '=', $request['email'])-> where('password' ,'=', Hash::make($request['password']))->where('role','=','user');

Is lacking a call to get the result set back from the query. Add ->first().

What it’ saying is:

You gave me an instance of the query Builder

But

I wanted a model that extends the authenticatable class

If you open App\User, you’ll see that it does indeed extend this class:

class User extends Authenticatable {

Method 2

This:

$user = User::where('email', '=', $request['email'])-> where('password' ,'=', Hash::make($request['password']))->where('role','=','user'); 

is not the way you will get the user.

Instead of

$user = User::where('email', '=', $request['email'])-> where('password' ,'=', Hash::make($request['password']))->where('role','=','user');                                                    

if($user){
    Auth::login($user); 
    return $next($request);
}
else{
    return redirect()->back();
}

you could use:

   $logged = auth()->attempt(
          ['email' => $request['email'], 
           'password' => $request['password'], 
           'role' => 'user']);

    if($logged) {
        return $next($request); // probably it won't work. This is fine in middleware but not in controller
    }
    else{
        return redirect()->back();
    }

Method 3

Solved my problem as following:

$user = User::where('email', '=' $request['email'])->where('role','=','user')->first();
if(Hash::check($request['password'],$user->password)){
    Auth::login($user);
    return  'success';
}   

Note: Use and implement method 1 because this method fully tested our system.
Thank you 🙂

All methods was sourced from stackoverflow.com or stackexchange.com, is licensed under cc by-sa 2.5, cc by-sa 3.0 and cc by-sa 4.0

Leave a Reply