-
Notifications
You must be signed in to change notification settings - Fork 44
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
Return a generic array in Query #115
Conversation
This may not be a list, it may be an indexed array. Example: https://github.com/doctrine/orm/blob/bfed8cb6ed448f4ab1ea3fff06e4d6c44439e4ef/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php#L491
Can the type of the index be determined somehow? |
The type is taken from DB, it's a type of the field. It's possible to parametrize |
Thanks! |
…return-type Fix templates on QueryBuilder::getQuery return type (broken by #115).
Is the commit a3a972c really useful ? Changing the generic signature is kinda a BC break for all static analysis users. And it create conflict with others tools like phpstan (as showed here phpstan/phpstan-doctrine#377). This shouldn't be done so easily. I dunno how it works but array can be written with one template |
This may not be a list, it may be an indexed array. Example:
https://github.com/doctrine/orm/blob/bfed8cb6ed448f4ab1ea3fff06e4d6c44439e4ef/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php#L491
This is a potential error like