[#3420] - [#33578] Remove ACL checks for specific menu item (J 3.x)
- Closed
- 11 Jul 2014
- Medium
- Build: staging
- # 3420
- Diff
- sanderpotjer:edit-state-3
- Foreign ID 33578
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Current ACL checks are causing unexpected results as ACL is only available for com_menus in general. So no need to check for ACL settings for com_menus.item.XX, only com_menus available.
Example of wrong behaviour right now: someone with edit state allowed for com_menus is not able to change the state by clicking on the publish/unpublish icon. This patch will fix this.
JoomlaCode: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33578
Test instructions:
- create a new user group
- allow "Admin Login" for this group in the global configuration permissions
- Go to the Menu Manager, click on the options button and allow "Access Administration Interface" and "Edit State" action for the group
- Add the new group to Access Level "special"
- create test user, assign to created group
- login with test user
- try to publish/unpublish a menu item. Result: "Edit state not permitted".
- apply patch
- try to publish/unpublish a menu item. Result: "1 menu item successfully unpublished/published"
| Title |
|
||||||
Looks like a duplicate report has been committed meanwhile: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33769 & #3636
Will close this one.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-07-11 11:54:02 |
@test - successful
Other Operations edit/delete/create .. works fine as well.
You may blame the J!Tracker Application for transmitting this comment.