Project

General

Profile

Actions

Bug #27659

closed

sorting by $TCA foreign_sortby vs. foreign_default_sortby

Added by Peter Niederlag almost 13 years ago. Updated over 3 years ago.

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

0%

Estimated time:
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).


Files


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #64197: Add support for foreign_default_sortby on TCA inline fieldsClosed2015-01-08

Actions
Actions #1

Updated by Thomas Mammitzsch almost 13 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).

Actions #2

Updated by Gerrit Code Review over 12 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

Actions #3

Updated by Felix Oertel almost 12 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

Actions #4

Updated by Felix Oertel almost 12 years ago

  • Assignee changed from Felix Oertel to Anja Leichsenring
Actions #5

Updated by Gerrit Code Review almost 12 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

Actions #6

Updated by Anja Leichsenring almost 12 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.

Actions #7

Updated by Gerrit Code Review almost 12 years ago

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

Actions #9

Updated by Gerrit Code Review almost 12 years ago

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

Actions #10

Updated by Alexander Schnitzler over 11 years ago

  • Target version changed from Extbase 6.0 to Extbase 6.1
Actions #11

Updated by Anja Leichsenring over 11 years ago

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

Updated by Alexander Schnitzler about 11 years ago

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

Updated by Alexander Schnitzler about 11 years ago

  • Target version changed from Extbase 6.1 to Extbase 6.2
Actions #14

Updated by Anja Leichsenring about 11 years ago

  • Target version changed from Extbase 6.2 to Extbase 6.3
Actions #15

Updated by Gerrit Code Review almost 11 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

Actions #16

Updated by Gerrit Code Review over 10 years ago

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

Actions #17

Updated by Gerrit Code Review over 10 years ago

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

Actions #18

Updated by Witali Rott over 10 years ago

!!Please remove this Comment!!

Actions #19

Updated by Alexander Schnitzler over 10 years ago

  • Assignee deleted (Alexander Schnitzler)
Actions #20

Updated by Gerrit Code Review about 10 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

Actions #21

Updated by Gerrit Code Review about 10 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

Actions #22

Updated by Gerrit Code Review about 10 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

Actions #23

Updated by Alexander Opitz over 9 years ago

  • Project changed from 534 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
Actions #24

Updated by Mathias Schreiber over 9 years ago

  • Target version changed from 7.0 to 7.1 (Cleanup)
Actions #25

Updated by Benni Mack almost 9 years ago

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

Updated by Anja Leichsenring almost 9 years ago

  • Status changed from Under Review to New
Actions #27

Updated by Susanne Moog almost 9 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #28

Updated by Benni Mack over 8 years ago

  • Target version deleted (7.5)
Actions #29

Updated by Pascal Rinker almost 5 years ago

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

Updated by Mathias Brodala about 4 years 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.

Actions #31

Updated by Rainer Becker over 3 years 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.

Actions

Also available in: Atom PDF