-
-
Notifications
You must be signed in to change notification settings - Fork 217
delete tags before modifying #1441
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
Conversation
|
What is the URL for this feed? I don't use the tagging extension, but wouldn't removing tags delete those not set by gpodder, and also those which are empty in gpodder? |
The feed in the example is The Daily from The New York Times: https://feeds.simplecast.com/54nAGcIl It also happens in some episodes of This American Life: https://www.thisamericanlife.org/podcast/rss.xml Yes, with this change when "modify_tags" is enabled all tags are stripped except ones set by gpodder. |
|
Mutagen 1.45.1 and 1.46.0 delete all existing tags and don't set gpodder tags with or without this patch, even though the modify_tags code path is executed and the tags are all set to non-empty strings. |
I am on mutagen 1.45.1 and that is not the case on gpodder 3.10.21 or 3.11.0. It works as expected with the exception of the excess metadata bug I am trying to fix. These are my extension settings: |
adds an option to remove existing tags before writing basic tags
|
new commit makes this an option that is defaulted off. If "remove_before_modify" is enabled existing tags are deleted before writing basic tags. |
|
LGTM, I'll squash and merge in a week if no one has a problem with the change. Are you able to upgrade mutagen to 1.45.1 or newer? I'm curious to know if it is only broken for me on Gentoo, or if it is broken for everyone with a newer version, and no one uses the extension or realizes it is broken. |
|
Ignore that, I thought you said 1.41.1 which wouldn't run for me on Python 3.10. |
FWIW I'm running xthursdayx/gpodder-docker. You can see what is installed in the dockerfile. I reinstalled the build-deps and did a git pull on gpodder to update to latest. Tagging works great with this one exception, which could have gone unnoticed for a long time since some players don't display the extra comment data. |
|
Thank you for the improvement. |
When "modify_tags" is enabled some episodes are getting carryover unrelated metadata prepended to the comments tag.
This displays different in different players. In iTunes the bug isn't visible and it displays "People trying to coax each other [...]". However in other clients, such as Emby, it displays "preroll_1;postroll_1" and the show notes are not displayed at all.
This patch clears existing tags from the mp3 before writing the new ones, which fixes this bug.