-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
ftintitle: New customization option to keep feature in artist field #5356
Conversation
The new keep_in_artist option allows keeping the feat. part in the artist metadata field while still changing the title.
Add functionality tests for the new keep_in_artist option of the ftintitle plugin.
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.
This looks very straight forward. Thanks! I think it doesn't hurt if you also add the typical usecase you mention in the PR description to the docs as well:
Searching for features using both title and artist (this can be very helpful when using music players with GUI)
Either in the docs introductional text or in the option description itself.
Depending on where you put it, you might want to rephrase. Again just brainstorming here. Something like:
....To be able to search for features in artist fields as well, this option can be used.....
It's helpful in music players that let us search in artist fields in general, not only with GUI.
Update the documentation of the new `keep_in_artist` option, adding an example use case.
Update the log message when the artist is kept unchanged due to setting the keep_in_artist option to true.
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.
Looks good to me. Please let me know if you have other change requests
I have updated the documentation and log message as you have suggested. Please let me know if there are any other changes you would like to see. |
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.
Many thanks. This is good to go 🎉
This PR produces errors for me
Line here Line 122 in cd360b6
I am able to reproduce just by running |
Thank you for spotting this @chrishoage. I have created a PR to fix this (#5433) |
Description
This PR adds a new option
keep_in_artist
to theftintitle
plugin.Previous Behavior
The artist field is always changed to match the albumartist field.
New Behavior
The new option allows a customization of the old behavior. If the option is set to yes/true, the "feat. X" part is kept in the artist field.
In order to be backwards compatible and not introduce unexpected behavior, the default is set to false, i.e., matching the old behavior.
Benefits
To Do