laravel-ide-macros icon indicating copy to clipboard operation
laravel-ide-macros copied to clipboard

Don't throw if macro is not recognizable

Open hackel opened this issue 3 years ago • 1 comments

Currently, if a macro is registered improperly or in a format that this package doesn't recognize, it throws an exception. e.g.

ReflectionException: Method Illuminate\Support\Carbon::__invoke() does not exist

This simply ignores the error and continues.

hackel avatar Aug 06 '20 00:08 hackel

@hackel are you sure it's a great idea to ignore such mistakes? Like, if I made a typo in the macro name, it'll just be ignored, I'll see that the macro helpers are not generated, but will have no idea why.

What's the use case for such change?

KristofMorva avatar Aug 15 '20 08:08 KristofMorva