-
Notifications
You must be signed in to change notification settings - Fork 7
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.
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
[Feature Request]: Add possibility to disable facades aliases generation to _ide_helper_facades.php #984
Comments
I made some changes in aliases fetching just yesterday) I'll try to fix that somehow. Thank you. |
It stopped working at least several weeks ago. |
Packages can declare facade aliases by the composer.json "extras" section. You don't want to see them too?) |
Yes. Magical global classes is bad for architecture. Actually all magic is bad, but some of it can be tolerated =) |
@adelf I can't seem to find the setting for this? Did I misunderstand and this wasn't added as a setting? 👀 The reason why I want to disable it is that it will block auto-import of the facades, because the global facade has the same name. |
It's not released yet. I hope to do it next week. |
Ah that makes sense, thank you. 👍 You are doing an awesome work! |
Fixed in Laravel Idea 8.2 |
Thank you for this work :) Really nice for auto import! |
@adelf Thank you! Works perfectly! |
Feature Description
Feature Description
Hello.
I have disabled aliases for facades in config/app.php:
But plugin still generates code assists for aliases in _ide_helper_facades.php:
I think that some people also disable some or all aliases and then receive errors like "Class "Str" not found" when code assist suggests \Str instead of \Illuminate\Support\Str.
I asked about this some time ago (Issue #752) but after recent updates it started to generate helpers again. I think the best solution here will be an option in plugin settings like "Do not generate helper code for global Laravel facades".
Thank you for this great plugin!
The text was updated successfully, but these errors were encountered: