Bug #27659

sorting by $TCA foreign_sortby vs. foreign_default_sortby

Added by Peter Niederlag almost 2 years ago. Updated about 1 month ago.

Status:Needs Feedback Start date:2012-07-22
Priority:Should have Due date:
Assignee:Alexander Schnitzler % Done:

0%
Category:Extbase: Generic Persistence
Target version:Extbase 6.2
Has patch:Yes Tags:
Votes: 0

Description

IRRE expects foreign_sortby to be a manual sorting field!
This setting triggers sorting controls in Backend and when saved IRRE tries to update the field values!
This is documented and intended behaviour of IRRE!
http://typo3.org/documentation/document-library/core-documentation/doc_core_api/4.2.0/view/4/2/#id4431730

In extbase we need another way to get proper sorting of child records. To me it seems like support for
'foreign_default_sortby' is the way to go (if we stick with $TCA).

T3X_issue27659-0_1_0-z-201207221401.t3x (99.5 kB) Anja Leichsenring, 2012-07-22 14:03

Related issues

related to Extbase MVC Framework - Bug #27248: Allow descending order with foreign_sortby Rejected 2011-06-06

History

#1
Updated by Thomas Mammitzsch almost 2 years ago

IRRE also hides the field which is used by foreign_sortby, so you can't edit it directly (f.e. if its a date field which you want to sort by).

#2
Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

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

#3
Updated by Felix Oertel 10 months ago

  • Status changed from Under Review to Accepted
  • Assignee set to Felix Oertel
  • Target version set to Extbase 6.0
  • Start date changed from 2011-06-23 to 2012-07-22

foreign_default_sortby can only used with non-MM relations. with MM relations foreign_sorting is always used to manual order the records. for non-MM relations extbase has to support the ordering (asc and desc) of child records of course.

the current patch is abandoned because it tries to solve the rejected issue [1] as well and is too difficult (SQL string split). we will try to come up with a solution at #ecs12s, but feel free to work on this!

[1] http://forge.typo3.org/issues/27248

#4
Updated by Felix Oertel 10 months ago

  • Assignee changed from Felix Oertel to Anja Leichsenring

#5
Updated by Gerrit Code Review 10 months ago

  • Status changed from Accepted to Under Review

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

#6
Updated by Anja Leichsenring 10 months ago

I created a little extension to track down the problem. It contains the test data as well. Feel free to use it to test the patch.
To see effects, manipulate the sorting options in TCA/Object1.

#7
Updated by Gerrit Code Review 10 months ago

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

#8
Updated by Markus Günther 10 months ago

  • Has patch changed from No to Yes

#9
Updated by Gerrit Code Review 10 months ago

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

#10
Updated by Alexander Schnitzler 6 months ago

  • Target version changed from Extbase 6.0 to Extbase 6.1

#11
Updated by Anja Leichsenring 6 months ago

  • Status changed from Under Review to Accepted
  • Assignee deleted (Anja Leichsenring)
  • Priority changed from Must have to Should have

#12
Updated by Alexander Schnitzler about 1 month ago

  • Status changed from Accepted to Needs Feedback
  • Assignee set to Alexander Schnitzler

#13
Updated by Alexander Schnitzler about 1 month ago

  • Target version changed from Extbase 6.1 to Extbase 6.2

Also available in: Atom PDF