Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug Report] Non-ASCII performers fail to be tagged #2293

Open
Still34 opened this issue Feb 6, 2022 · 1 comment · Fixed by #2336
Open

[Bug Report] Non-ASCII performers fail to be tagged #2293

Still34 opened this issue Feb 6, 2022 · 1 comment · Fixed by #2336
Labels
bug Something isn't working
Milestone

Comments

@Still34
Copy link
Contributor

Still34 commented Feb 6, 2022

Describe the bug
Non-ASCII performers appear to invalid to the auto taggers and cause the tagger to ignore the paths - specifically Japanese/Chinese in my case. Other non-ASCII characters were not tested.

To Reproduce
Steps to reproduce the behavior:

  1. Create a directory containing non-ASCII character of the performer's name and move a valid media file to the directory.
  2. Create a new performer with the non-ASCII character (e.g., 伏字).
  3. Enqueue auto-tagging tasks.

Expected behavior
The media within the performer's directory should be tagged with the performer's metadata, but it is not.
e.g., /media/伏字/hello-world.mp4 should be tagged with the performer 伏字 that was just created.

Stash Version: (from Settings -> About):
v0.12.0-40-gc857e301

Desktop (please complete the following information):

  • OS: Both Windows and Linux
@Still34 Still34 added the help wanted Extra attention is needed label Feb 6, 2022
@kermieisinthehouse kermieisinthehouse added bug Something isn't working bug report Bug reports that are not yet verified and removed help wanted Extra attention is needed labels Feb 8, 2022
@kermieisinthehouse kermieisinthehouse added this to the Version 0.13.0 milestone Feb 8, 2022
@WithoutPants
Copy link
Collaborator

Reopening with a known issue that names with single unicode letters will not match due to a performance optimisation.

@WithoutPants WithoutPants reopened this Mar 7, 2022
@WithoutPants WithoutPants modified the milestones: Version 0.13.0, Backlog Mar 7, 2022
@WithoutPants WithoutPants removed the bug report Bug reports that are not yet verified label Apr 6, 2022
@github-project-automation github-project-automation bot moved this to To triage in Bug fixing Feb 13, 2024
@WithoutPants WithoutPants moved this from To triage to Backlog in Bug fixing Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

3 participants