User tests: Successful: Unsuccessful:
Pull Request for Issue #8569
Respect the access levels
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
The tag with view access Super Users is not visible (for non superusers)
All tags visible, regardless of access set in Tag component (for non superusers)
None
Status | New | ⇒ | Pending |
Category | ⇒ | Libraries |
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?
As such a handling needs to be implemented in any extension that has tags support implemented
I have tested this item
Tag havin Access "Super Users" isn't shown in Article-View if logged-in as "Manager" (not Member of User-Group "Super Users").
Easy | No | ⇒ | Yes |
Category | Libraries | ⇒ | ACL com_tags Libraries |
I have tested this item
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.
Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
@franz-wohlkoenig @rjcf18 did you also check the issue that @mbabker noted?
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:
Ah ok I see. Then Ill test that as well as soon as I can.
I have tested this item
Super-User-Tag is gone after a Non-Super-User saved an Article having a Super-User-Tag.
I have tested this item
Super-User-Tag is gone after a Non-Super-User saved an Article having a Super-User-Tag.
I have tested this item
@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.
Status | Ready to Commit | ⇒ | Pending |
Closing here based on the unsuccessfull tests above.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-21 18:35:03 |
Closed_By | ⇒ | zero-24 |
Category | Libraries ACL com_tags | ⇒ | Libraries ACL |
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.