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
Embedart: remove_art_file on import #1675
Conversation
Should fix #1662. I think the fix is easy. Don't know if it's clean though. Did some tests (+/- 10 albums), all successful. This is the first time I'm using python so please let me know what I should improve:)
@@ -80,7 +80,7 @@ def embed_func(lib, opts, args): | |||
art.embed_album(self._log, album, maxwidth, False, | |||
compare_threshold, ifempty) | |||
|
|||
if remove_art_file and album.artpath is not None: | |||
if remove_art_file and album.artpath: |
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.
@Kraymer, I assume this line can be changed as well?
Awesome! Thank you for looking into this. A bit of refactoring will avoid code duplication. Then, we'll just need a changelog entry and we'll be ready to merge. Thanks again! ✨ |
Damn it, that was really hard for me:D. First time seriously using python. Please review it carefuly.
You're welcome. I was really struggling with the function. I probably made some mistakes and t.b.h. I'm a bit stuck now... |
You were just missing a : at the end of the function declaration. Fixing now and merging. |
Embedart: remove_art_file on import
Should fix #1662.
I think the fix is easy. Don't know if it's clean though. Did some tests (+/- 10 albums), all successful. This is the first time I'm using python so please let me know what I should improve:)