Fix Mapper::query() method to pass param types.#208
Fix Mapper::query() method to pass param types.#208andrejs wants to merge 4 commits intospotorm:masterfrom
Conversation
|
Off: The custom query above is just an example, right? Because you don't need to use a custom query for that. You can execute it like this: On: I can't say anything about the usefullness of passing through the custom param type. |
|
Yes, it's just an example for the sake of simplicity. Intended usage of |
|
I agree! |
|
Needs tests, otherwise looks good to me. Valitron bug was already fixed with #217. |
When using custom query, it's not possible to use custom types for query parameters. For example:
This pull request fixes the problem by passing custom param types to \Doctrine\DBAL\Connection::executeQuery() via Mapper::query().