[TASK] Improve sorting of usergroups in adminpanel 04/35404/3
authorGeorg Ringer <georg.ringer@gmail.com>
Sat, 13 Dec 2014 23:32:58 +0000 (00:32 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sun, 14 Dec 2014 19:36:56 +0000 (20:36 +0100)
To improve the usability, the usergroups are ordered by title and
the uid is added at the end to allow working with the keyboard.

Resolves: #17290
Releases: master, 6.2
Change-Id: Ie855a73e81a8b66265d4b94567db088f0991fd5d
Reviewed-on: http://review.typo3.org/35404
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/frontend/Classes/View/AdminPanelView.php

index 2f0ff0f..5687ea0 100644 (file)
@@ -360,9 +360,15 @@ class AdminPanelView {
 
                        $options = '';
 
-                       $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('fe_groups.uid, fe_groups.title', 'fe_groups,pages', 'pages.uid=fe_groups.pid AND pages.deleted=0 ' . BackendUtility::deleteClause('fe_groups') . ' AND ' . $GLOBALS['BE_USER']->getPagePermsClause(1));
+                       $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery(
+                               'fe_groups.uid, fe_groups.title',
+                               'fe_groups,pages',
+                               'pages.uid=fe_groups.pid AND pages.deleted=0 ' . BackendUtility::deleteClause('fe_groups') . ' AND ' . $GLOBALS['BE_USER']->getPagePermsClause(1),
+                               '',
+                               'fe_groups.title ASC'
+                       );
                        while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
-                               $options .= '<option value="' . $row['uid'] . '"' . ($GLOBALS['BE_USER']->uc['TSFE_adminConfig']['preview_simulateUserGroup'] == $row['uid'] ? ' selected="selected"' : '') . '>' . htmlspecialchars(('[' . $row['uid'] . '] ' . $row['title'])) . '</option>';
+                               $options .= '<option value="' . $row['uid'] . '"' . ($GLOBALS['BE_USER']->uc['TSFE_adminConfig']['preview_simulateUserGroup'] == $row['uid'] ? ' selected="selected"' : '') . '>' . htmlspecialchars(($row['title'] . ' [' . $row['uid'] . ']')) . '</option>';
                        }
                        $GLOBALS['TYPO3_DB']->sql_free_result($res);
                        if ($options) {