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

Hyphen in the Episode/ID Regexp (Reopening #2865) #3967

Open
tunaflsh opened this issue Apr 5, 2024 · 0 comments
Open

Hyphen in the Episode/ID Regexp (Reopening #2865) #3967

tunaflsh opened this issue Apr 5, 2024 · 0 comments

Comments

@tunaflsh
Copy link

tunaflsh commented Apr 5, 2024

Expected behaviour:

As an example from the documentation for regexps ep_regexp: (\d\d)-(\d\d\d) should correctly identify season number and episode number.

Actual behaviour:

It does not identify the season/episode and reports but cannot find a(n) `ep` style identifier.

Proposed solution:

This has already been discussed in #2865 but the conversation quickly derailed to adjacent topics. Although there are workarounds, I think this issue is still valid.

Either fix the documentation or replace the regex [\W_]+ with (?:[^\w-]|_)+ in:

data_parts = re.split(r'[\W_]+', data_stripped)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant