Doctrine 2 - ORM

Add ORDER BY clause to collection persister SQL generation

Details

  • Type: Sub-task Sub-task
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0-ALPHA4
  • Fix Version/s: 2.0-BETA1
  • Component/s: ORM
  • Security Level: All
  • Labels:
    None

Description

OrderBy Mapping information of collections translate to an ORDER BY clause in the Collection Persister for both OneToMany and ManyToMany associations.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Benjamin Eberlei added a comment -

Implemented for both ManyToMany and OneToMany associations.

Show
Benjamin Eberlei added a comment - Implemented for both ManyToMany and OneToMany associations.
Hide
Permalink
Roman S. Borschel added a comment -

Is there a test for class table inheritance already? JoinedSubclassPersister#_getSelectEntitiesSql might need to be adjusted for this feature also. We should add a test to make sure.

Show
Roman S. Borschel added a comment - Is there a test for class table inheritance already? JoinedSubclassPersister#_getSelectEntitiesSql might need to be adjusted for this feature also. We should add a test to make sure.
Hide
Permalink
Benjamin Eberlei added a comment -

Oh no there is no test for class table inheritance,it probably fails.

Show
Benjamin Eberlei added a comment - Oh no there is no test for class table inheritance,it probably fails.
Hide
Permalink
Benjamin Eberlei added a comment -

Wah this could be a considerable issue of the implementation, how to make sure %alias% is replaced with the correct alias in a class table hierachy?

Show
Benjamin Eberlei added a comment - Wah this could be a considerable issue of the implementation, how to make sure %alias% is replaced with the correct alias in a class table hierachy?
Hide
Permalink
Benjamin Eberlei added a comment -

Changed @OrderBy to take a simple DQL snippet and parse this according to very strict rules. This solves the class table inheritance problem as shown by a new test-case

Show
Benjamin Eberlei added a comment - Changed @OrderBy to take a simple DQL snippet and parse this according to very strict rules. This solves the class table inheritance problem as shown by a new test-case

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Benjamin Eberlei
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: