? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
18 Sep 2020

Pull Request for Issue Reported on JSE https://joomla.stackexchange.com/questions/28479/what-permission-do-you-need-to-check-in-others-article-at-any-time.

Summary of Changes

In view layouts and in com_checkin we use core.manage permission check to determine whether users can perform a check in. But in the model we use core.admin check. This way the check in button appears enabled but check in fails. However, user with core.manage permissions in com_checkin can still check in items through com_checkin.

Testing Instructions

Create a user in Manager group.
Give Access Administrator Interface permissions in com_checkin to managers.
While still logged in as super user checkout an article.
Login to backend as manager.
Try to check in the article from article list and from com_checkin.

Actual result BEFORE applying this Pull Request

Checking in in article list fails but works in com_checkin.

Expected result AFTER applying this Pull Request

Checking in works in both places.

Documentation Changes Required

IDK.

avatar SharkyKZ SharkyKZ - open - 18 Sep 2020
avatar SharkyKZ SharkyKZ - change - 18 Sep 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Sep 2020
Category Libraries
avatar SharkyKZ SharkyKZ - change - 18 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 18 Sep 2020
avatar SharkyKZ SharkyKZ - change - 18 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 18 Sep 2020
avatar SharkyKZ SharkyKZ - change - 18 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 18 Sep 2020
avatar richard67 richard67 - test_item - 19 Sep 2020 - Tested successfully
avatar richard67
richard67 - comment - 19 Sep 2020

I have tested this item successfully on 113aa62

First I couldn't reproduce the issue on a newly installed current staging.

The reason is that both the Access Administrator Interface and Configure ACL & Options permissions in com_checkin were already granted for the administrator group.

After having revoked the Configure ACL & Options permissions in com_checkin for administrators, everything worked as described.

@SharkyKZ Maybe you should pimp the testing instructions to handle that.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30676.

avatar SharkyKZ SharkyKZ - change - 19 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 19 Sep 2020
avatar SharkyKZ
SharkyKZ - comment - 19 Sep 2020

Better test with Manager group. Instructions updated.

avatar Formatio-hippocampi Formatio-hippocampi - test_item - 19 Sep 2020 - Tested successfully
avatar Formatio-hippocampi
Formatio-hippocampi - comment - 19 Sep 2020

I have tested this item successfully on 113aa62


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30676.

avatar richard67 richard67 - change - 19 Sep 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 19 Sep 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30676.

avatar richard67 richard67 - change - 19 Sep 2020
Labels Added: ? ?
avatar infograf768 infograf768 - change - 21 Sep 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-21 10:02:13
Closed_By infograf768
Labels
avatar infograf768 infograf768 - close - 21 Sep 2020
avatar infograf768 infograf768 - merge - 21 Sep 2020
avatar infograf768
infograf768 - comment - 21 Sep 2020

Tks

Add a Comment

Login with GitHub to post a comment