search

psalm / psalm-plugin-laravel

A Psalm plugin for Laravel
MIT License
307 stars 72 forks source link

Fixes #184 add UserHandler #217

Closed tm1000 closed 1 year ago

tm1000 commented 2 years ago

This fixes #184. I dont think you actually need to look at the config object because in Laravel 8.x documentation it's stated that:

By default, Laravel includes an App\Models\User Eloquent model in your app/Models directory.

https://laravel.com/docs/8.x/authentication#introduction-database-considerations

mr-feek commented 2 years ago

Would you be able to add a corresponding test for this by chance?

tm1000 commented 2 years ago

@mr-feek yes I can do tests for this. Just need some time so if you want you can draft this PR

alies-dev commented 1 year ago

FYI: I'm working on alternative, more comprehensive solution

tm1000 commented 1 year ago

@lptn you can close this then no worries. Looking forward to your solution!

alies-dev commented 1 year ago

@tm1000 I just created a draft PR, feel free to review and leave some feedback: https://github.com/psalm/psalm-plugin-laravel/pull/326