Tags in tag groups not visible to assigned user group

Contribute Bug

AI-generated summary

A bug was discovered where tags in tag groups are not visible to assigned user groups. nolo reported the issue, providing screenshots to demonstrate the problem. JammyDodger provided repro steps to verify the issue, which included enabling tags_listed_by_group, creating a tag group with restricted visibility, and testing with a trust level 0 user. The expected behavior was for the trust level 0 user to see the tag group, but they were unable to do so. The bug was later fixed via a commit as reported by techAPJ.

1

Came across this on a client instance and could replicate it on a testing instance as well:

I restrict the visibility of tags using a tag group:

Screenshot from 2022-06-15 10-20-24
Screenshot from 2022-06-15 10-20-24536×482 11.1 KB

So for admins the group shows as:

Screenshot from 2022-06-15 10-23-04
Screenshot from 2022-06-15 10-23-04568×132 1.7 KB

But for trust_level_0 the the tag and tag group don’t show on the tags page. We also use tag groups to bundle tags on a component and they don’t show there either.

They do show as a single tag on topics.

3 Likes
2

Let’s try for a repro. :slight_smile:

  • Enable tags_listed_by_group in admin settings
  • Create tag ‘newsletter’ on a public topic
  • Create tag group called ‘Members Only’
  • Add ‘newsletter’ tag to tag group, and set tag group visibility to trust_level_zero

  • Set test user as TL0
  • Look at /tags page


    Expected: TL0 test user would be able to see TL0+ tag group
    Turns out: TL0 test user cannot see the TL0+ tag group

I also tried it with the test user as TL1-4 as well, but the TL0+ tag group only shows up when I crank them up to Moderator (or Admin).

I also tried it with a non-trust-level group too, but that also didn’t show the restricted tag group when the test user was added to that group.

5 Likes
6

Fixed via:

https://github.com/discourse/discourse/commit/59f0815c9d42a8d53671dcfffaf88c89b634cddd

Thanks for reporting the issue @manuel and for the repro steps @JammyDodger :+1:

3 Likes
7

This topic was automatically closed after 8 hours. New replies are no longer allowed.