Feature #12759

Sorting of csv relation is always ordered by uid

Added by Kevin Ulrich Moschallski over 2 years ago. Updated about 1 month ago.

Status:Rejected Start date:2011-02-03
Priority:-- undefined -- Due date:
Assignee:- % Done:

0%
Category:Extbase: Generic Persistence
Target version:Extbase 6.1
Has patch:Yes Tags:
Votes: 5 (View)

Description

Hi,

when a relation is of type csv the order of the objectStorage is always by uid even if the order is changed manualy in the BE. The problem is that you can't set a order field with csv relations.

The solution is to add ORDER BY FIELD to the sql query.

I made a patch to add this order statement if no order is set for the query. Please have a look.

12759.patch (2.1 kB) Kevin Ulrich Moschallski, 2011-02-03 15:29

History

#2
Updated by Matthias Krappitz over 1 year ago

We can confirm this and would love to see this patch in extbase. This is a bug, that should affect many more people that just us.

#3
Updated by Alexander Claes over 1 year ago

Same problem here! We tried this bugfix and it works very well for us.

#4
Updated by Gerrit Code Review 8 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14452

#5
Updated by Alexander Schnitzler 6 months ago

  • Target version set to Extbase 6.1
  • Has patch set to No

#6
Updated by Alexander Schnitzler 6 months ago

  • Tracker changed from Bug to Feature

May change orderings in some environments. So this is a feature and we should think about making this configurable.

#7
Updated by Anja Leichsenring about 1 month ago

  • Has patch changed from No to Yes

#8
Updated by Gerrit Code Review about 1 month ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/14452

#9
Updated by Marc Bastian Heinrichs about 1 month ago

  • Status changed from Under Review to Rejected
  • Priority changed from Must have to -- undefined --

Patch works only if you have one comparison constraint. Patch needs to have massive improvement to get in.

please reopen, if you have a working patch.

Also available in: Atom PDF