[4.4] Access level registered for mod_user #42254
Open
+6
−4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
richard67
reviewed
Oct 31, 2023
administrator/components/com_admin/sql/updates/mysql/4.4.0-2023-10-31.sql
Outdated
Show resolved
Hide resolved
richard67
reviewed
Oct 31, 2023
administrator/components/com_admin/sql/updates/postgresql/4.4.0-2023-10-31.sql
Outdated
Show resolved
Hide resolved
…3-10-31.sql Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
…0-2023-10-31.sql Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
|
Is it intended top remove the link to the frontend of the site from the login page? |
chmst
changed the title
|
I think we can't hard code access level ids in sql statements (there is no guarantee that 2 === registered) |
HLeithner
requested changes
Nov 23, 2023
Quy
added
the
Updates Requested
|
Should not be enough for this module to appear that the user have access to the administrator? How I see it, this module do not need to proccess any access level to see if he can or not show up in that position. If you log in, you need a way to log out and you don't need an access level to define it. |
HLeithner
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request for Issue #28804 .
Summary of Changes
Remove rendering status modules from login, Set accesslevel "registered" for mod menu
Testing Instructions
Grant access to the backend for a usergroup which has not viewlevel "special", for example to usergroup registered.
The user then can login to the backend.
Actual result BEFORE applying this Pull Request
He cannot log out
Expected result AFTER applying this Pull Request
Set access-level for mod_user to "registered" or make a fresh installation for a new test.
mod_user is active, the user can log out.
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed