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

ASSET_NAME Folder Name validation function removing Colins and Hypens #2477

Open
dextrous0z opened this issue Jan 24, 2025 · 2 comments
Open
Assignees
Labels
bug Bug is with Plex Meta Manager status:not-yet-viewed I haven't reviewed the Feature or Bug yet

Comments

@dextrous0z
Copy link

Version Number

2.1.0 (Docker: master)

What branch are you on?

master

Describe the Bug

I've had an issue with the ASSET_NAME match within Kometa I've got folder's setup that matches the exact name within Plex however the sanatization function in util.py is causing characters such as colin's and hyphens to to be removed from my Folder name so it can't find the assets folder.

I added in debug to verify this was the issue the debug printing is shown below.
I'm happy to make a pull request however unsure how this should be rectified.

I

Image

Relevant Collection/Overlay/Playlist Definition

Logs

No response

@dextrous0z dextrous0z added bug Bug is with Plex Meta Manager status:not-yet-viewed I haven't reviewed the Feature or Bug yet labels Jan 24, 2025
@dextrous0z dextrous0z changed the title ASSET_NAME Folder Name sanatising removing Colins and Hypens ASSET_NAME Folder Name validation function removing Colins and Hypens Jan 24, 2025
@chazlarson
Copy link
Contributor

This is by design. Characters which are potentially illegal in file paths are removed to determine the target asset name.

@dextrous0z
Copy link
Author

dextrous0z commented Jan 24, 2025

I'm not trying to get filename validation removed just fixing valid paths within different operating systems.
I've taken a look at the documentation for the function is_valid_filepath we can pass in a parameter of platform: "auto" to resolve the issue ?

I've tested it and it works on my end and it fixes the issue. You should be able to see on patch-1 or something on my fork.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug is with Plex Meta Manager status:not-yet-viewed I haven't reviewed the Feature or Bug yet
Projects
None yet
Development

No branches or pull requests

3 participants