Ruud Kamphuis

Results 363 comments of Ruud Kamphuis

> I get it, but I'm worried this would make debugging problems harder. Also, a problem in objectManagerLoader would still allow analysis to continue, but with vastly different results, leaving...

@janedbal That approach would still let PHPStan fail / crash hard, making the PHPStorm integration annoying, as it pops up with errors on every file change.

This is the error we're seeing in PHPStorm: ![Screenshot 2025-06-02 at 15 08 27@2x](https://github.com/user-attachments/assets/923319f8-4944-45b6-a86d-5d41b0f67229)

I was just using Xdebug to find the issue: It's throwing here: https://github.com/phpstan/phpstan-doctrine/blob/be66017631e5289e8a3bd8973e59781642da40fd/src/Type/Doctrine/ArgumentsProcessor.php#L61-L66

It seems that `$className` is the alias `ev` now: ![Screenshot 2024-09-12 at 16 39 15@2x](https://github.com/user-attachments/assets/0e3527d4-eff6-4561-a173-cefe7bf445bd)

Interesting: ![Screenshot 2024-09-12 at 16 40 16@2x](https://github.com/user-attachments/assets/20df0c6c-12d1-4b41-ae6c-684b08ca2bfa) It thinks this ConstantStringType is a ClassStringType

It seems that I have a class called `ev` and `event`. The last one makes sense, somewhere. But the first one? Interesting. https://github.com/phpstan/phpstan-src/blob/5480d27383ffee399037c31d461c5a85344128ca/src/Type/Constant/ConstantStringType.php#L83-L92

You're so fast ⚡ I added some debug info in https://github.com/phpstan/phpstan-doctrine/issues/608

For what it's worth, I decided to create a custom schema manager to intercept the `_getPortableTableColumnDefinition` call and choose the correct type. Hope it will help somebody: https://gist.github.com/ruudk/10fd5dc1325492df97b33682b3423f34

@oojacoboo Why not contribute the changes?