[BUGFIX] Show "switch user" button for non-admin users #68
Conversation
* Enable "switch user" button for non-admin users (if action is allowed) Related: koninklijke-collective#67
* Removed unneeded ViewHelper namespace Related: koninklijke-collective#67
* Added modified "SwitchUserController" to be able to successfully call "switchUserAction" as non-admin * Small condition update in "IndexListRow" partial Related: koninklijke-collective#68
| if (ExtensionManagementUtility::isLoaded('beuser')) { | ||
| // Set backend user listing module as starting module if installed | ||
| $currentUser->uc['startModuleOnFirstLogin'] = 'system_BeuserTxBeuser'; | ||
| } |
There was a problem hiding this comment.
What is the intention behind this? In my case this leads to troubles. When the non-admin user hits exist the user sees an "access denied". I am currently thinking what would be the best solution. Following come to mind:
- Make
$currentUser->uc['startModuleOnFirstLogin'] = 'system_BeuserTxBeuser';configurable. - Completely remove these lines.
However, for that it is helpful to understand the motivation for this condition.
There was a problem hiding this comment.
There is no special intention behind that. It's just a copy from what the core does: https://github.com/TYPO3/typo3/blob/main/typo3/sysext/backend/Classes/Controller/SwitchUserController.php#L80
So I think your solution no. 2 would be the best option. There is no need to keep these lines, imo.
There was a problem hiding this comment.
Ah, I see. Thanks for the quick response and update!
Fixed issue: > When the non-admin user hits exit the user sees an "access denied". Related: koninklijke-collective#67 (see koninklijke-collective#68 (review))
|
I have stumbled on the same issue. The problem is that the extension relies on a ViewHelper from the "beuser" extension |
|
Hi, |
Related: #67