UserWithId Strategy calls id() method on User Model about laravel-unleash HOT 6 CLOSED

@mikefrancis We're running into the same issue as @vinnyvimto on our end – my understanding is the framework source link above calls getUserResolver() with the $guard as a parameter, but returns a Illuminate\Foundation\Auth\User, not the Illuminate\Contracts\Auth\Guard. I updated the PR I pushed to use getAuthIdentifier() (from Illuminate\Auth\Authenticatable, which User implements) instead of id(), which is working on our end now and should fix the tests as well.

from laravel-unleash.

@vinnyvimto It seems like you know how to fix the issue, would be great if you could raise a PR instead?

from laravel-unleash.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from laravel-unleash.

Hello,

Why has it been closed ? This bug is preventing this project to work properly with the basic features of unleash. :/

from laravel-unleash.

@rfussien If it's so important, can I ask if there issomething preventing you from PRing a fix?

from laravel-unleash.

@vinnyvimto

$user on this line should be an instance of either Illuminate\Contracts\Auth\Guard or null from looking at the framework source.

The strategy checks for null, and any Guard has to implement the id() method, so this looks like a problem with your guard? Are you using something custom which doesn't implement that method?

from laravel-unleash.