-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
{Custom Format:Format}
Rename Token
#6585
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add some tests to verify the behaviour (and prevent breaking changes in the future) in a new fixture in https://github.com/Sonarr/Sonarr/tree/develop/src/NzbDrone.Core.Test/OrganizerTests/FileNameBuilderTests
I added Unit Tests for both Something I ran into was that Custom Formats allow for spaces in their Names. Multiple Trash Guide formats have them (example). I tweaked the Did some quick tests in an expression tester: All existing FileNameBuilder tests passed after the change as well. |
Co-authored-by: Mark McDowall <markus.mcd5@gmail.com>
Description
Implementing the
{Custom Format:Format}
rename token to allow position of Custom Formats when renaming.This re-implements #4453 for Custom Formats, and would satisfy Radarr/Radarr#6801 if it gets cherry-picked to over there.
It is a smaller change because it's really just allowing exact placement of the Format Name instead of the old Preferred Words implementation that replaced with the matched values.
Database Migration
NO
Issues Fixed or Closed by this PR