[TASK] Clean up ['ctrl'] sortby and default_sortby 08/52108/2
authorChristian Kuhn <lolli@schwarzbu.ch>
Mon, 20 Mar 2017 19:43:33 +0000 (20:43 +0100)
committerFrank Nägler <frank.naegler@typo3.org>
Wed, 22 Mar 2017 10:45:41 +0000 (11:45 +0100)
sortby: An integer field with an explicit sorting value

default_sortby: A query->orderBy statement to sort records
if no explicit sortby field is given.

default_sortby is ignored if sortby is given. The patch removes
a couple of TCA ['ctrl']['default_sortby'] declarations where both
were defined.

default_sortby needs no "ORDER BY" prefix. The patch removes it
from those TCA tables where this prefix was still defined.

Additionally, the TCA reference docs have been improved in this
area to hint about both fields and their correlation. It is especially
evil if that is confused and a field like 'title' is set as sortby:
The DataHandler would happily write integers into that field on save.

Change-Id: Ib5c38a538cd3f60d3e05033488363a144f756505
Resolves: #80361
Releases: master
Reviewed-on: https://review.typo3.org/52108
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
13 files changed:
typo3/sysext/core/Configuration/TCA/be_groups.php
typo3/sysext/core/Configuration/TCA/be_users.php
typo3/sysext/core/Configuration/TCA/sys_collection.php
typo3/sysext/core/Configuration/TCA/sys_file.php
typo3/sysext/core/Configuration/TCA/sys_file_collection.php
typo3/sysext/core/Configuration/TCA/sys_file_metadata.php
typo3/sysext/core/Configuration/TCA/sys_file_storage.php
typo3/sysext/core/Configuration/TCA/sys_language.php
typo3/sysext/frontend/Configuration/TCA/fe_users.php
typo3/sysext/indexed_search/Configuration/TCA/index_config.php
typo3/sysext/rtehtmlarea/Configuration/TCA/tx_rtehtmlarea_acronym.php
typo3/sysext/sys_action/Configuration/TCA/sys_action.php
typo3/sysext/sys_note/Configuration/TCA/sys_note.php

index e381a07..b51239b 100644 (file)
@@ -7,7 +7,7 @@ return [
         'crdate' => 'crdate',
         'cruser_id' => 'cruser_id',
         'delete' => 'deleted',
-        'default_sortby' => 'ORDER BY title',
+        'default_sortby' => 'title',
         'prependAtCopy' => 'LLL:EXT:lang/Resources/Private/Language/locallang_general.xlf:LGL.prependAtCopy',
         'adminOnly' => true,
         'rootLevel' => 1,
index 85fdf43..768c4cd 100644 (file)
@@ -10,7 +10,7 @@ return [
         'delete' => 'deleted',
         'adminOnly' => true,
         'rootLevel' => 1,
-        'default_sortby' => 'ORDER BY admin, username',
+        'default_sortby' => 'admin, username',
         'enablecolumns' => [
             'disabled' => 'disable',
             'starttime' => 'starttime',
index 9675e1c..c23ba5c 100644 (file)
@@ -12,7 +12,7 @@ return [
         'languageField' => 'sys_language_uid',
         'transOrigPointerField' => 'l10n_parent',
         'transOrigDiffSourceField' => 'l10n_diffsource',
-        'default_sortby' => 'ORDER BY crdate',
+        'default_sortby' => 'crdate',
         'delete' => 'deleted',
         'type' => 'type',
         'rootLevel' => -1,
index b92b097..298b74b 100644 (file)
@@ -7,7 +7,7 @@ return [
         'type' => 'type',
         'hideTable' => true,
         'rootLevel' => true,
-        'default_sortby' => 'ORDER BY name ASC',
+        'default_sortby' => 'name ASC',
         'typeicon_column' => 'type',
         'typeicon_classes' => [
             '1' => 'mimetypes-text-text',
index 833de93..e4d8649 100644 (file)
@@ -11,7 +11,7 @@ return [
         'languageField' => 'sys_language_uid',
         'transOrigPointerField' => 'l10n_parent',
         'transOrigDiffSourceField' => 'l10n_diffsource',
-        'default_sortby' => 'ORDER BY crdate',
+        'default_sortby' => 'crdate',
         'delete' => 'deleted',
         'type' => 'type',
         'typeicon_column' => 'type',
index c98aa0b..e213e60 100644 (file)
@@ -14,7 +14,7 @@ return [
         'transOrigDiffSourceField' => 'l10n_diffsource',
         'versioningWS' => true,
         'origUid' => 't3_origuid',
-        'default_sortby' => 'ORDER BY crdate DESC',
+        'default_sortby' => 'crdate DESC',
         'typeicon_classes' => [
             'default' => 'mimetypes-other-other'
         ],
index bb3088c..a632806 100644 (file)
@@ -6,7 +6,7 @@ return [
         'tstamp' => 'tstamp',
         'crdate' => 'crdate',
         'cruser_id' => 'cruser_id',
-        'default_sortby' => 'ORDER BY name',
+        'default_sortby' => 'name',
         'delete' => 'deleted',
         'descriptionColumn' => 'description',
         'rootLevel' => true,
index 18bcefe..0235571 100644 (file)
@@ -3,7 +3,6 @@ return [
     'ctrl' => [
         'label' => 'title',
         'tstamp' => 'tstamp',
-        'default_sortby' => 'ORDER BY sorting',
         'sortby' => 'sorting',
         'title' => 'LLL:EXT:lang/Resources/Private/Language/locallang_tca.xlf:sys_language',
         'adminOnly' => true,
index 9f55b47..583e423 100644 (file)
@@ -3,7 +3,7 @@ return [
     'ctrl' => [
         'label' => 'username',
         'descriptionColumn' => 'description',
-        'default_sortby' => 'ORDER BY username',
+        'default_sortby' => 'username',
         'tstamp' => 'tstamp',
         'crdate' => 'crdate',
         'cruser_id' => 'cruser_id',
index a209f57..ab556c0 100644 (file)
@@ -8,7 +8,7 @@ return [
         'crdate' => 'crdate',
         'cruser_id' => 'cruser_id',
         'type' => 'type',
-        'default_sortby' => 'ORDER BY crdate',
+        'default_sortby' => 'crdate',
         'enablecolumns' => [
             'disabled' => 'hidden',
             'starttime' => 'starttime'
index f62011f..13968f6 100644 (file)
@@ -3,7 +3,6 @@ return [
     'ctrl' => [
         'title' => 'LLL:EXT:rtehtmlarea/Resources/Private/Language/locallang_db.xlf:tx_rtehtmlarea_acronym',
         'label' => 'term',
-        'default_sortby' => 'ORDER BY term',
         'sortby' => 'sorting',
         'delete' => 'deleted',
         'enablecolumns' => [
index 3679fe0..50ffd71 100644 (file)
@@ -4,7 +4,6 @@ return [
         'label' => 'title',
         'descriptionColumn' => 'description',
         'tstamp' => 'tstamp',
-        'default_sortby' => 'ORDER BY title',
         'sortby' => 'sorting',
         'prependAtCopy' => 'LLL:EXT:lang/Resources/Private/Language/locallang_general.xlf:LGL.prependAtCopy',
         'title' => 'LLL:EXT:sys_action/Resources/Private/Language/locallang_tca.xlf:sys_action',
index 76ec5ea..31e39ca 100644 (file)
@@ -2,7 +2,6 @@
 return [
     'ctrl' => [
         'label' => 'subject',
-        'default_sortby' => 'ORDER BY crdate',
         'tstamp' => 'tstamp',
         'crdate' => 'crdate',
         'cruser_id' => 'cruser',