Uploaded image for project: 'Doctrine 2 - ORM'
  1. Doctrine 2 - ORM
  2. DDC-2190

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

    Details

    • Type: Bug
    • Status: Open
    • Priority: 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. DDC2190Test.php
        0.6 kB
        Benjamin Eberlei

        Activity

        Hide
        beberlei 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
        beberlei 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
        beberlei 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
        beberlei 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.

          People

          • Assignee:
            beberlei Benjamin Eberlei
            Reporter:
            stof Christophe Coevoet
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: