search

TomasVotruba / bladestan

PHPStan analysis for Blade templates
https://tomasvotruba.com/blog/introducing-bladestan-phpstan-analysis-of-blade-templates/
MIT License
286 stars 13 forks source link

Add section about limitations and how to apply workarounds #51

Closed AJenbo closed 1 year ago

AJenbo commented 1 year ago

These are notes based on issue and solutions from when I was getting the project I work on to be fully processed.

TomasVotruba commented 1 year ago

Thanks for the detailed info about weak parts.

Those should be handled by the compiler, as the context is known and we should handle it for the user instead to privde the best user experience possible :)

AJenbo commented 1 year ago

Those should be handled by the compiler, as the context is known and we should handle it for the user instead to privde the best user experience possible :)

I much agree, but solving this will take some work so I'm thinking this would be useful for early adopters :)

TomasVotruba commented 1 year ago

Indeed. It would also state it's the intended and lead us astray from solution :)

Could you share a reproducible test case instead?

AJenbo commented 1 year ago

Turns out the first case isn't actually an issue, but I was getting misleading error message because of an issue with how includes where being processed. That one is now fixed in #52, and I created a test case for the other one in #54

Also note that #45 is now read to be merged