TYPO3 Core

Is this committed?

According to the thread in Core List it should be fixed with #21798; but I can't find any of these in ChangeLog...

No, Christopher! #21798 is set to resolved as this bug entry is open as duplicate.
It's pending in core list since May 31 (sent by Reinhard), but no votes are given, yet!

@Reinhard
Could you please attach the tag "pening in core list" to BT entries, when RFC was sent? Thanks

Sorry, missed one part of the thread. Seems to be more tricky, may be you can remind Christian or you have a look at it again and provide a solution.

Hi Alexander,

yes, this issue still exists!

Here is a quick diff against the current GIT repository.
I am not sure about how to format this really long line properly, but you should at least see what should be changed.

Should I try to push the change to Gerrit?

diff --git a/typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateModuleController.php b/typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateModuleController.php
index db878dd..079102b 100644
--- a/typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateModuleController.php
+++ b/typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateModuleController.php
@@ -189,7 +189,15 @@ class TypoScriptTemplateModuleController extends \TYPO3\CMS\Backend\Module\BaseS
                 TABLE#ts-overview tr.t3-row-header td { padding: 2px 4px; font-weight:bold; color: #fff; }
             ';
             // Template pages:
-            $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('pages.uid, count(*) AS count, max(sys_template.root) AS root_max_val, min(sys_template.root) AS root_min_val', 'pages,sys_template', 'pages.uid=sys_template.pid' . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('pages') . \TYPO3\CMS\Backend\Utility\BackendUtility::versioningPlaceholderClause('pages') . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('sys_template') . \TYPO3\CMS\Backend\Utility\BackendUtility::versioningPlaceholderClause('sys_template'), 'pages.uid');
+            $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery(
+                'pages.uid, count(*) AS count, max(sys_template.root) AS root_max_val, min(sys_template.root) AS root_min_val', 
+                'pages,sys_template', 
+                'pages.uid=sys_template.pid' . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('pages') . 
+                \TYPO3\CMS\Backend\Utility\BackendUtility::versioningPlaceholderClause('pages') . 
+                \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('sys_template') . 
+                \TYPO3\CMS\Backend\Utility\BackendUtility::versioningPlaceholderClause('sys_template'), 
+                'pages.sorting'
+            );
             $templateArray = array();
             $pArray = array();
             while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {

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

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

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

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

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40490