[TASK] Improve options for "Simulate backend user" 77/33277/3
authorGeorg Ringer <georg.ringer@gmail.com>
Sun, 12 Oct 2014 17:35:34 +0000 (19:35 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Sun, 12 Oct 2014 19:24:06 +0000 (21:24 +0200)
The option uses the syntax "username (real name)",
however the () are even added if no full name is available.

As a solution, add the () only if a real name is given.

Resolves: #62179
Releases: master
Change-Id: Icbf375574b0399ff37a16df42bc59272a52fb0e4
Reviewed-on: http://review.typo3.org/33277
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/setup/Classes/Controller/SetupModuleController.php

index 203bdcf..e0f16aa 100644 (file)
@@ -639,7 +639,8 @@ class SetupModuleController {
                        $opt = array();
                        foreach ($users as $rr) {
                                if ($rr['uid'] != $GLOBALS['BE_USER']->user['uid']) {
-                                       $opt[] = '<option value="' . $rr['uid'] . '"' . ($this->simUser == $rr['uid'] ? ' selected="selected"' : '') . '>' . htmlspecialchars(($rr['username'] . ' (' . $rr['realName'] . ')')) . '</option>';
+                                       $label = htmlspecialchars(($rr['username'] . ($rr['realName'] ? ' (' . $rr['realName'] . ')' : '')));
+                                       $opt[] = '<option value="' . $rr['uid'] . '"' . ($this->simUser == $rr['uid'] ? ' selected="selected"' : '') . '>' . $label . '</option>';
                                }
                        }
                        if (count($opt)) {