[#7575] - Correct "Unlock" Button ACL permissions
- Closed
- 25 Oct 2015
- Medium
- Build: staging
- Easy Test
- # 7575
- Diff
- pagchen:patch-1
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Unlock Button not showing if user just have core.manage rights (and not core.admin rights).
So I change the test on core.admin to only influence the Options button and not the Unlock Button.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
I extracted the button from the if statement.
In the process I did put an extra set of tabs, I removed them.
Can you please change the title of this PR to something representing what you propose? "Update view.html.php" is a bit to generic 
I would suggest a PR tile like "correct Unlock Button management rights"
| Title |
|
||||||
Changed it myself.
Yes it is a better title. Thanks
| Category | ⇒ | ACL Administration UI/UX |
| Easy | No | ⇒ | Yes |
I have tested this item 
successfully on b0d9b7f
successfully tested.
Here are some test instructions:
Add User (i.e. Manager rights)
Change User Rights for Manager Group "Global Checkin" to "Access Administration Interface" --> Allowed
Login in Backend
Check Menu "System" - "Global Checkin" --> Button "Checkin" (top left) is missing
Apply patch
Button is showing
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7575.
I have tested this item successfully on b0d9b7f
tested successfully according description of seagul30
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7575.
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
RTC. Thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7575.
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
| Milestone |
Removed: |
||
I think it is a change in the behauviour of the cms so 3.5
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-10-25 16:16:52 |
| Closed_By | ⇒ | roland-d |
| Labels |
Removed:
?
|
||
@pagchen White space change from 2 to 3 tabs for no apparent reason?