Feeling Lucky
?
?
[#33638] - [4.0] Ignore access level check while parsing SEF router
- Fixed in Code Base
- 9 May 2021
- Medium
- Build: 4.0-dev
- # 33638
- Diff
- joomdonation:fix_parse_sef_router
User tests: Successful: Unsuccessful:
Pull Request for Issue #33500.
Summary of Changes
Ignore menu items access level check while parsing sef router so that in case users access to a menu item which requires login, they will be redirected to login page instead of seeing 404 error (same behavior with Joomla! 3)
Testing Instructions
- Create a menu item to link to an article, set Access to Registered. Set alias of that menu item to test or something you want.
- Access to frontend of your site, remember to log-out if you are logged in
- Try to access to the the menu item which you created above. The link will have this format https://localhost/joomla4/index.php/test or https://localhost/joomla4/test (depends on your site setting)
- Replace https://localhost/joomla4/ with URL of your site
- Replace test with Alias of the menu item which you created in step 1
- Before patch: You get 404 error
- After patch: You are being redirected to login page which allows you to login to access to that link
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
alikon
- comment
- 9 May 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33638.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 9 May 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33638.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-09 11:17:15 |
| Closed_By | ⇒ | richard67 | |
| Labels |
Added:
?
?
|
||
richard67
- comment
- 9 May 2021
Thanks!
I have tested this item✅ successfully on a96401d
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33638.