Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mismatched function signatures #153

Closed
bram-pkg opened this issue Jun 18, 2021 · 3 comments
Closed

Mismatched function signatures #153

bram-pkg opened this issue Jun 18, 2021 · 3 comments
Assignees
@mr-feek
Labels
bug Something isn't working

Comments

@bram-pkg
Copy link

bram-pkg commented Jun 18, 2021 •
edited
Loading

Describe the bug

When running Psalm with the Laravel plugin enabled, this error occurs:

Declaration of Psalm\LaravelPlugin\FakeMetaCommand::registerClassAutoloadExceptions() must be compatible 
with Barryvdh\LaravelIdeHelper\Console\MetaCommand::registerClassAutoloadExceptions(): callable

Impacted Versions
Please paste the output of composer show | grep -E 'psalm|laravel' so we can see what versions are impacted

barryvdh/laravel-dompdf               v0.9.0    A DOMPDF Wrapper for Laravel
barryvdh/laravel-ide-helper           v2.10.0   Laravel IDE Helper, generates correct PHPDocs ...
elhebert/laravel-sri                  3.0.1     Subresource Integrity hash generator for laravel
fruitcake/laravel-cors                v2.0.4    Adds CORS (Cross-Origin Resource Sharing) head...
laravel/framework                     v8.47.0   The Laravel Framework.
laravel/sail                          v1.8.1    Docker files for running a basic Laravel appli...
laravel/tinker                        v2.6.1    Powerful REPL for the Laravel framework.
laravel/ui                            v3.3.0    Laravel UI utilities and presets.
psalm/plugin-laravel                  v1.4.6    A Laravel plugin for Psalm
spatie/laravel-ray                    1.20.1    Easily debug Laravel apps
vimeo/psalm                           4.7.3     A static analysis tool for finding errors in P...
@bram-pkg bram-pkg added the bug Something isn't working label Jun 18, 2021
@mr-feek
Copy link
Collaborator

mr-feek commented Jun 21, 2021

@bramceulemans this is due to a breaking change in ide-helper. You need to downgrade ide-helper to one listed within the constraint here: https://github.com/psalm/psalm-plugin-laravel/blob/master/composer.json#L8

See barryvdh/laravel-ide-helper#945 (comment)

@mr-feek mr-feek closed this as completed Jun 21, 2021
@bram-pkg
Copy link
Author

Alright, any hope on fixing this in the future?

@mr-feek
Copy link
Collaborator

mr-feek commented Jun 21, 2021

It's really up to ide-helper, they're the ones who created the breaking change in a non-major version release

bereket-WMDE added a commit to wbstack/api that referenced this issue Jun 28, 2021
@bereket-WMDE
downgrade laravel-de-helper to resolve Mismatched function signatures
3e1e388 
see: psalm/psalm-plugin-laravel#153
bereket-WMDE added a commit to wbstack/api that referenced this issue Jun 28, 2021
@bereket-WMDE
downgrade laravel-de-helper to resolve Mismatched function signatures
a5dd677 
see: psalm/psalm-plugin-laravel#153
addshore pushed a commit to wbstack/api that referenced this issue Jun 30, 2021
@bereket-WMDE @dati18
Laravel update (#113)
cc8323a 
* WIP

* Update Laravel from 7.x to 8.x

- remove legacy factory helper fuctions
- Add new factory implementations for Models accoring to laravel 8.x
- Add psr-4 class mapping for factories and seeders

Bug: T284314

* downgrade laravel-de-helper to resolve Mismatched function signatures

see: psalm/psalm-plugin-laravel#153

* fixed psalm erros

Co-authored-by: Dat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mr-feek mr-feek

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mr-feek @bram-pkg