Doctrine 2 - ORM

findBy() support finding by a single DateTime but not by multiple DateTime

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: ORM
  • Security Level: All
  • Labels:
    None

Description

The following code works:

$repository->findBy(array('date' => new \DateTime()))

but the following code fails as it does not apply the conversion of the date type for each element:

$repository->findBy(array('date' => array(new \DateTime(), new \DateTime('tomorrow')))
  1. File
    DDC2190Test.php
    09/May/13 12:47 PM
    0.6 kB
    Benjamin Eberlei

Activity

Ascending order - Click to sort in descending order
Christophe Coevoet made changes -
Field Original Value New Value
Description The following code works:

{code}$repository->findBy('date' => new \DateTime()){code}

but the following code fails as it does not apply the conversion of the {{date}} type for each element:

{code}$repository->findBy('date' => array(new \DateTime(), new \DateTime('tomorrow')){code} 		The following code works:

{code}$repository->findBy(array('date' => new \DateTime())){code}

but the following code fails as it does not apply the conversion of the {{date}} type for each element:

{code}$repository->findBy(array('date' => array(new \DateTime(), new \DateTime('tomorrow'))){code}
Hide
Permalink
Benjamin Eberlei added a comment -

This is actually very hard to implement, the problem is that we only have ARRAY constants for PDO::PARAM_INT and PDO::PARAM_STR - all the other types would require special handling.

Show
Benjamin Eberlei added a comment - This is actually very hard to implement, the problem is that we only have ARRAY constants for PDO::PARAM_INT and PDO::PARAM_STR - all the other types would require special handling.
Hide
Permalink
Benjamin Eberlei added a comment -

Attaching failing testcase.

The idea is to have something like "datetime[]" as type and detect this in the SQLParserUtils of DBAL.

Another approach would be to convert the values in the ORM already, before passing to the DBAL.

Show
Benjamin Eberlei added a comment - Attaching failing testcase. The idea is to have something like "datetime[]" as type and detect this in the SQLParserUtils of DBAL. Another approach would be to convert the values in the ORM already, before passing to the DBAL.
Benjamin Eberlei made changes -
Attachment DDC2190Test.php [ 11526 ]
This list may be incomplete, as errors occurred whilst retrieving source from linked applications:
  • Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DDC-2190, expand=changesets[-21:-1].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Christophe Coevoet
Vote (0)
Watch (2)

Dates

  • Created:
    Updated: