mirror of
https://github.com/discourse/discourse.git
synced 2026-08-06 13:08:40 +08:00
aec18e9f70Every endpoint that lists tags for browsing hand-rolled its own filter chain, and they had drifted apart. The tags page filtered synonyms out when tags were listed flat but not when they were listed by group, so a synonym of a tag belonging to no tag group was shown next to its target — a duplicate entry whose page only redirects back to the tag it duplicates. Synonyms of grouped tags were hidden by accident rather than by design: creating a synonym copies the target's tag group memberships onto it, which happens to satisfy the "belongs to no tag group" condition the ungrouped list is built from. Non-admins were shielded by a second accident, since a synonym normally ends up with a zero topic count — so any synonym that kept its topics, whether from an import, a plugin, or assigning `target_tag_id` directly, was listed to everyone. Tag visibility has two independent mechanisms: tag group permissions, and the categories a tag is attached to. Only the second one can apply to a tag that belongs to no tag group, and neither the grouped tags page nor the tag group search endpoint applied it, so the names of tags restricted to a category the viewer cannot read were listed to them. `TagGroup.visible` gates which groups are returned, never the tags inside them. The navigation menu tag picker filtered out neither synonyms nor tags only used in personal messages, and gated on staff where the tags page gates on admin, so moderators were served the entire tag table. A synonym picked there was saved as a sidebar link and rendered from then on. Top tags and tag search had the same gap. Paths that resolve a tag by name keep matching synonyms on purpose: the composer offers them so that typing a retired name finds the tag that replaced it, and hashtags in posts cooked before a rename have to keep working. Serializers that echo configuration back are left alone, because filtering a value the client posts straight back would silently delete it. |
||
|---|---|---|
| .. | ||
| db | ||
| fabricators | ||
| fixtures | ||
| generator | ||
| helpers | ||
| import_export | ||
| initializers | ||
| integration | ||
| integrity | ||
| jobs | ||
| lib | ||
| mailers | ||
| migrations | ||
| models | ||
| multisite | ||
| queries/reports | ||
| requests | ||
| script | ||
| serializers | ||
| services | ||
| support | ||
| system | ||
| tasks | ||
| views | ||
| rails_helper.rb | ||
| regenerate_swagger_docs | ||
| swagger_helper.rb | ||
