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.

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]: Mark macro/mixin methods as non-static #1090

Open
mortenscheel opened this issue Oct 24, 2024 · 1 comment
Open

[Feature Request]: Mark macro/mixin methods as non-static #1090

mortenscheel opened this issue Oct 24, 2024 · 1 comment

Comments

@mortenscheel
Copy link

Feature Description

When adding macro/mixin methods added to a class like Illuminate\Console\Command, the helper code will contain definitions of both a static and non-static version of the method. But the static version doesn't make any sense for a command, and I have an inspection that complains that $this->theMacro() should be changed to self::theMacro().

Is it possible to tell laravel-idea that it shouldn't define a static version of a macro? If not, could it be added?

@adelf
Copy link
Member

adelf commented Jan 22, 2025

We will remove static methods from the helper code soon. In one of the next versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants