Unable to see PM lists for non-ASCII tag

Contribute Bug

AI-generated summary

A user, alexaw, reported an issue where non-ASCII tags in personal messages (PMs) do not return a list of PMs when clicked under /u/username/messages/tags or /tags. The issue is due to the parameterization of tag names, which strips special characters from strings. techAPJ identified the problem and created a fix by removing the parameterization (PR: https://github.com/discourse/discourse/pull/9839). The fix was tested rigorously and merged, resolving the issue.

1

When a personal message is tagged with non-ASCII character tag, the non-ASCII tag can be displayed in the PM Inbox, Sent, Archive and PM Tags list.

However, when clicking the tag under /u/username/messages/tags or /tags, only ASCII tag will return a list of PMs. Clicking on a non-ASCII tag will show “No topics found”.

I am using non-ASCII (Japanese and Chinese) tags sitewide, and haven’t had any problem except in PM.

Screenshots

image
image1000×265 26.3 KB

image
image600×241 20.1 KB

image
image1000×364 33.4 KB

image
image600×295 14.9 KB

Repro steps:

  1. Add non-ASCII tag and ASCII tag to the same PM
  2. Navigate to /u/username/messages/tags or /tags
  3. Click on the non-ASCII tag and ASCII tag respectively
  4. An empty result set for non-ASCII PM tag is returned, while a normal result for ASCII tag is returned
5 Likes
2

Probably a bug we should assign your Monday @sam?

3 Likes
4

The issue here is that we are parameterizing the tag name here which strips special characters from strings. This also means that tags with emoji in name will show same behaviour.

The fix is pretty simple (PR):

https://github.com/discourse/discourse/pull/9839

But I am not sure of the side effects of removing parameterizing. The interesting thing is it was I who added parameterization two years ago in:

https://github.com/discourse/discourse/commit/18f50ca01abfa5497b3dc423470c3b5e84e48771

But not sure what issue it was fixing.

I have tested the PR pretty rigorously locally and I don’t see any issue with removing parameterizing of tag names.

EDIT: PR is merged.

8 Likes