Bug #27659

sorting by $TCA foreign_sortby vs. foreign_default_sortby

Added by Peter Niederlag over 9 years ago. Updated 16 days ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2012-07-22
Due date:
% Done:

0%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

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

DataMapFactory.php.patch View (1.92 KB) Witali Rott, 2013-10-28 14:56


Related issues

Related to TYPO3 Core - Feature #64197: Add support for foreign_default_sortby on TCA inline fields Under Review 2015-01-08

History

#1 Updated by Thomas Mammitzsch over 9 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 8 years 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 over 8 years 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 over 8 years ago

  • Assignee changed from Felix Oertel to Anja Leichsenring

#5 Updated by Gerrit Code Review over 8 years 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 over 8 years 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 over 8 years 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 about 8 years ago

  • Has patch changed from No to Yes

#9 Updated by Gerrit Code Review about 8 years 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 almost 8 years ago

  • Target version changed from Extbase 6.0 to Extbase 6.1

#11 Updated by Anja Leichsenring almost 8 years 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 over 7 years ago

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

#13 Updated by Alexander Schnitzler over 7 years ago

  • Target version changed from Extbase 6.1 to Extbase 6.2

#14 Updated by Anja Leichsenring over 7 years ago

  • Target version changed from Extbase 6.2 to Extbase 6.3

#15 Updated by Gerrit Code Review over 7 years ago

  • Status changed from Needs Feedback to Under Review

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

#16 Updated by Gerrit Code Review about 7 years ago

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

#17 Updated by Gerrit Code Review almost 7 years ago

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

#18 Updated by Witali Rott almost 7 years ago

!!Please remove this Comment!!

#19 Updated by Alexander Schnitzler almost 7 years ago

  • Assignee deleted (Alexander Schnitzler)

#20 Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/21565

#21 Updated by Gerrit Code Review over 6 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/21565

#22 Updated by Gerrit Code Review over 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/21565

#23 Updated by Alexander Opitz about 6 years ago

  • Project changed from Extbase MVC Framework to TYPO3 Core
  • Category changed from Extbase: Generic Persistence to Extbase
  • Target version changed from Extbase 6.3 to 7.0
  • TYPO3 Version set to 6.2
  • Is Regression set to No

#24 Updated by Mathias Schreiber almost 6 years ago

  • Target version changed from 7.0 to 7.1 (Cleanup)

#25 Updated by Benni Mack over 5 years ago

  • Target version changed from 7.1 (Cleanup) to 7.4 (Backend)

#26 Updated by Anja Leichsenring over 5 years ago

  • Status changed from Under Review to New

#27 Updated by Susanne Moog about 5 years ago

  • Target version changed from 7.4 (Backend) to 7.5

#28 Updated by Benni Mack about 5 years ago

  • Target version deleted (7.5)

#29 Updated by Pascal Rinker about 1 year ago

  • Related to Feature #64197: Add support for foreign_default_sortby on TCA inline fields added

#30 Updated by Mathias Brodala 5 months ago

  • Status changed from New to Closed

Given that both this and #64197 have abandoned changes and are about the same topic we should keep everything in one ticket. So I'm closing this in favor of #64197.

#31 Updated by Rainer Becker 16 days ago

Any news on this? In TYPO3 10.4.9 the field used in foreign_sortby is still hidden from IRRE editing like described in the original ticket https://forge.typo3.org/issues/27659#note-1.

Also available in: Atom PDF