Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
21 Apr 2017

Pull Request for Issue #8569

Summary of Changes

Respect the access levels

Testing Instructions

Create a TAG in the Tags component, set the access to Super Users
Login in as a manager or another user not in Super User group
Create new article and select Tags

Expected result

The tag with view access Super Users is not visible (for non superusers)

Actual result

All tags visible, regardless of access set in Tag component (for non superusers)

Documentation Changes Required

None

avatar zero-24 zero-24 - open - 21 Apr 2017
avatar zero-24 zero-24 - change - 21 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Apr 2017
Category Libraries
avatar mbabker
mbabker - comment - 21 Apr 2017

This needs to be tested to ensure there is no data loss during the save processes if a user with a higher level access sets some tags, then a user with lower level access edits the item later.

avatar zero-24
zero-24 - comment - 21 Apr 2017

hmm correct this is a issue how do you suggest to fix this? Doing some expensive save the current value compare that with the new value and ignore changes which the current user has no permissions to change?

avatar zero-24
zero-24 - comment - 21 Apr 2017

As such a handling needs to be implemented in any extension that has tags support implemented

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 22 Apr 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Apr 2017

I have tested this item successfully on dd56c17

Tag havin Access "Super Users" isn't shown in Article-View if logged-in as "Manager" (not Member of User-Group "Super Users").


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Apr 2017
Easy No Yes
avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Apr 2017
Category Libraries ACL com_tags Libraries
avatar rjcf18
rjcf18 - comment - 25 Apr 2017

I have tested this item successfully on dd56c17

Created a new tag with access to Super Users, created a manager user and logged in as that user, created a new article, checked the tags and the tag with view access Super Users was not visible as expected.


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

avatar rjcf18 rjcf18 - test_item - 25 Apr 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 25 Apr 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 25 Apr 2017

RTC after two successful tests.

avatar zero-24
zero-24 - comment - 25 Apr 2017

@franz-wohlkoenig @rjcf18 did you also check the issue that @mbabker noted?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 May 2017

@zero-24 need to test Issue @mbabker noted: do you mean "there is no data loss during the save processes" tags or i.E. Fulltext, Publishes Date, ...?

avatar zero-24
zero-24 - comment - 1 May 2017

@franz-wohlkoenig

I mean:

This needs to be tested to ensure there is no data loss during the save processes if a user with a higher level access sets some tags, then a user with lower level access edits the item later.

So this means:

  • Super User adds a article with super user tags (tags who can only be added by a super user)
  • admin (that can't access that superuser tags) open and save the article.
  • The Super User checks if the "superuser tags" are gone or not.
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 May 2017

@zero-24 thanks for Explination, will test Tomorrow.

avatar rjcf18
rjcf18 - comment - 1 May 2017

Ah ok I see. Then Ill test that as well as soon as I can.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 May 2017

I have tested this item 🔴 unsuccessfully on dd56c17

Super-User-Tag is gone after a Non-Super-User saved an Article having a Super-User-Tag.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 May 2017

I have tested this item 🔴 unsuccessfully on dd56c17

Super-User-Tag is gone after a Non-Super-User saved an Article having a Super-User-Tag.


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 2 May 2017 - Tested unsuccessfully
avatar rjcf18
rjcf18 - comment - 2 May 2017

I have tested this item 🔴 unsuccessfully on dd56c17

@zero-24 I've verified and after opening and saving an article with a super user tag as a non super user the tag was indeed gone from the article when it was reopened later.
deepinscreenshot20170502143933


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15467.
avatar rjcf18 rjcf18 - test_item - 2 May 2017 - Tested unsuccessfully
avatar rdeutz rdeutz - change - 4 May 2017
Status Ready to Commit Pending
avatar zero-24
zero-24 - comment - 21 May 2017

Closing here based on the unsuccessfull tests above.

avatar zero-24 zero-24 - change - 21 May 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-05-21 18:35:03
Closed_By zero-24
avatar zero-24 zero-24 - close - 21 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 21 May 2017
Category Libraries ACL com_tags Libraries ACL

Add a Comment

Login with GitHub to post a comment