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

Return a lazy collection from PersistentCollection::match($criteria)

    Details

      Description

      In 2.3, PersistentCollection::match() has been implemented by doing the query directly. But sometimes, the only meaningful information about the matched collection would be its length. In this case, it would be great to handle it in the same way than extra lazy collections are handled: the matched collection would be initialized lazily, and could do the count in an extra lazy way (if the original collection was extra lazy).
      This would of course not change anything in the case where the original collection was already initialized.

        Issue Links

          Activity

          Hide
          maciekpak Maciej Klemarczyk added a comment -

          It will be very usefull, for example to compute count of rows.

          In 300 rows, you do not want fetch data from database to compute length of this.

          Function matching(Criteria $criteria) fetch too many data for just compute count of rows.

          Show
          maciekpak Maciej Klemarczyk added a comment - It will be very usefull, for example to compute count of rows. In 300 rows, you do not want fetch data from database to compute length of this. Function matching(Criteria $criteria) fetch too many data for just compute count of rows.
          Hide
          stof Christophe Coevoet added a comment -

          This is exactly the use case I add in mind actually

          Show
          stof Christophe Coevoet added a comment - This is exactly the use case I add in mind actually
          Hide
          bakura Michaël Gallego added a comment -

          +1 on this one! This is absolutely needed when we use the Selectable API as the abstraction for some libraries.

          Show
          bakura Michaël Gallego added a comment - +1 on this one! This is absolutely needed when we use the Selectable API as the abstraction for some libraries.

            People

            • Assignee:
              guilhermeblanco Guilherme Blanco
              Reporter:
              stof Christophe Coevoet
            • Votes:
              8 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: