DDC-1148: [Patch] Auto infer parameter type in DQL #1443

Closed
opened 2026-01-22 13:14:34 +01:00 by admin · 4 comments
Owner

Originally created by @doctrinebot on GitHub (May 11, 2011).

Originally assigned to: @guilhermeblanco on GitHub.

Jira issue originally created by user @guilhermeblanco:

Here is the patch:

http://pastie.org/1891023
http://pastie.org/1891030

Originally created by @doctrinebot on GitHub (May 11, 2011). Originally assigned to: @guilhermeblanco on GitHub. Jira issue originally created by user @guilhermeblanco: Here is the patch: http://pastie.org/1891023 http://pastie.org/1891030
admin added the Improvement label 2026-01-22 13:14:34 +01:00
admin closed this issue 2026-01-22 13:14:34 +01:00
Author
Owner

@doctrinebot commented on GitHub (May 12, 2011):

Comment created by @beberlei:

instanceof DateTimeZone is just wrong. drop that whole branch, TZ only saves the timezone from a DateTime object, infering TYPE::DECIMAL sounds dangerous to me, i would drop that one aswell. The array check has a bug, you call key() and then do nothing with the $key instaed check is_integer on the $value again, which is always an array from the previous check.

@doctrinebot commented on GitHub (May 12, 2011): Comment created by @beberlei: instanceof DateTimeZone is just wrong. drop that whole branch, TZ only saves the timezone from a DateTime object, infering TYPE::DECIMAL sounds dangerous to me, i would drop that one aswell. The array check has a bug, you call key() and then do nothing with the $key instaed check is_integer on the $value again, which is always an array from the previous check.
Author
Owner

@doctrinebot commented on GitHub (May 12, 2011):

Comment created by @beberlei:

I thought you wanted to infer the types from the left hand sides of the DQL conditions?

@doctrinebot commented on GitHub (May 12, 2011): Comment created by @beberlei: I thought you wanted to infer the types from the left hand sides of the DQL conditions?
Author
Owner

@doctrinebot commented on GitHub (May 13, 2011):

Comment created by @guilhermeblanco:

Implemented: 8e3fdc5adc

@doctrinebot commented on GitHub (May 13, 2011): Comment created by @guilhermeblanco: Implemented: https://github.com/doctrine/doctrine2/commit/8e3fdc5adc0facd4b44f9054ab3681e46131ba8a
Author
Owner

@doctrinebot commented on GitHub (May 13, 2011):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (May 13, 2011): Issue was closed with resolution "Fixed"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#1443