Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-2838

Leaky abstraction when applying Criteria to hydrated/non-hydrated PersistentCollection

    Details

    • Type: Bug Bug
    • Status: Reopened
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.4.1
    • Fix Version/s: None
    • Component/s: ORM
    • Labels:
      None

      Description

      When applying a Criteria to a PersistentCollection that has been hydrated the field names must be camel case, if the collection has not yet been hydrated the field names must be underscore separated.

      The github repo linked here contains a simplified testcase for the matrix of hydrated/non-hydrated entities and camel case/underscore separated fields.

      https://github.com/ptlis/DoctrineTestcase

        Issue Links

          Activity

          Hide
          Marco Pivetta added a comment -

          We can't check out an entire project just to test a bug.

          Please write an actual functional test case that can be integrated into the Doctrine ORM test suite.

          Show
          Marco Pivetta added a comment - We can't check out an entire project just to test a bug. Please write an actual functional test case that can be integrated into the Doctrine ORM test suite.
          Hide
          brian ridley added a comment -

          Hi,

          i'm happy to do so - i'll take a look at this over the weekend.

          Show
          brian ridley added a comment - Hi, i'm happy to do so - i'll take a look at this over the weekend.
          Hide
          Simon Paridon added a comment -

          brian ridley: Any progress on this? This is currently an issue for us as well and hope to get fixed. I could look into converting it to a test for integration with the test suite if you don't have the time... but it might take a while since I have no experience with the requirements that should be met. (Plus, I am not sure how tightly coupled it is with your project)

          Show
          Simon Paridon added a comment - brian ridley : Any progress on this? This is currently an issue for us as well and hope to get fixed. I could look into converting it to a test for integration with the test suite if you don't have the time... but it might take a while since I have no experience with the requirements that should be met. (Plus, I am not sure how tightly coupled it is with your project)
          Hide
          Simon Paridon added a comment -

          Hi Marco Pivetta, brian ridley,

          I attached a functional Test that integrates with the test suite. Please let me know if I should issue a PR, and I'll do that this evening.

          Show
          Simon Paridon added a comment - Hi Marco Pivetta , brian ridley , I attached a functional Test that integrates with the test suite. Please let me know if I should issue a PR, and I'll do that this evening.

            People

            • Assignee:
              Marco Pivetta
              Reporter:
              brian ridley
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: