Patch #11057
open
Confirm admin flag change on a separate page
0%
Description
Having admin flag on the general tab in user's profile makes it
way too easy to flip the flag accidentally.
Attached patch adds a separate page to flip the flag.
Files
Updated by Jean-Philippe Lang over 12 years ago
It's a bit overkill IMO. Why not adding a simple javascript confirmation message when changing the checkbox state?
Updated by Etienne Massip over 12 years ago
Why not simply move it in its own "Administrator" fieldset?
Not sure it should remain in "Information" anyway, this is a security related concept, not a general information?
Updated by Alex Shulgin over 12 years ago
Jean-Philippe Lang wrote:
It's a bit overkill IMO. Why not adding a simple javascript confirmation message when changing the checkbox state?
I disagree: a simple Yes/No popup is too easy to overlook or accept accidentally.
Etienne Massip wrote:
Why not simply move it in its own "Administrator" fieldset?
Not sure it should remain in "Information" anyway, this is a security related concept, not a general information?
Well, maybe that's a way to go. Want me to produce an adjusted patch with a separate tab?
Updated by Alex Shulgin over 12 years ago
- File 0001-Confirm-admin-flag-change-on-a-separate-tab.patch 0001-Confirm-admin-flag-change-on-a-separate-tab.patch added
Alex Shulgin wrote:
Etienne Massip wrote:
Why not simply move it in its own "Administrator" fieldset?
Not sure it should remain in "Information" anyway, this is a security related concept, not a general information?
Well, maybe that's a way to go. Want me to produce an adjusted patch with a separate tab?
Attached is a reworked version with the admin tab instead of a separate page.