-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Debug permissions in com_users #14408
Conversation
… users and groups view. The defaullt is to on
If not set then it defaults to ON Change the IF statement use to turn on the display of the debug link to use the param instead of the JDEBUG
If not set then it defaults to ON
…to use the param instead of the JDEBUG
Add missing Options button from toolbar
Simplify parameter checks
I have tested this item ✅ successfully on 1fff9dd This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
I have tested this item ✅ successfully on 1fff9dd This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
@Bakual can you fix conflicts, please? |
Back to Pending due to conflicting files. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
@jeckodevelopment Conflicts are solved now. |
I have tested this item ✅ successfully on fab4401 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
I have tested this item ✅ successfully on fab4401 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
Just some smal CS fixes for 14408
I have tested this item ✅ successfully on 2555c79 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
1 similar comment
I have tested this item ✅ successfully on 2555c79 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14408. |
* Add a fieldset in the component config to support disabling the debug users and groups view. The defaullt is to on * Add code to read the params from the config.xml If not set then it defaults to ON Change the IF statement use to turn on the display of the debug link to use the param instead of the JDEBUG * Add code to read the params from the config.xml If not set then it defaults to ON * Change the IF statement use to turn on the display of the debug link to use the param instead of the JDEBUG * fix the params IF statement * Fix the params fallback * Add language strings * Add missing Cancel button from toolbar (replaces PR14384) Add missing Options button from toolbar * Simplify param check * Simplify param check #2 * change the strings from debug to advanced * fix toolbar permissions check * Update config.xml * Update default.php * Update default.php
We have a very nice and helpful view for "debugging" issues with user permissions. But that view is only available when "Debug System" is enabled in the global configuration. If it's disabled (like in normal production) the link to it is hidden.
Summary of Changes
This PR adds two new options in the user manager which will allow to enable/disable that
"Debug""Advanced" view for the user and usergroup manager. Making the view more known.The new options are enabled by default since there is no drawback to have them enabled. It's just a link shown. Also it will be B/C for those who had Debug enabled for whatever reason.
Testing Instructions
Expected result
Have a link to those views not depending on Debug status.
Actual result
Links only shown when Debug is active.
Documentation Changes Required
Maybe tutorials need to be adjusted.
Disclaimer
Work is done by @brianteeman