Permalink
Browse files

Add check for 'No Cover' when applying coverart to videos.

This considers a 'No Cover' value properly as non-existent, when
importing data from a VideoGrabber search result, to allow a new
cover to be downloaded in it's place, rather than refusing to
overwrite an existing image.
(cherry picked from commit 2076d12)
  • Loading branch information...
1 parent 099533d commit bd0f9eba93b37471bb6c3d0867b7470f6a116248 @wagnerrp wagnerrp committed Sep 20, 2012
Showing with 2 additions and 1 deletion.
  1. +2 −1 mythtv/bindings/python/MythTV/dataheap.py
@@ -993,7 +993,8 @@ def _allow_change(self, tag, overwrite):
for image in metadata.images:
if not hasattr(self, image.type):
continue
- if getattr(self, image.type) and not overwrite:
+ current = getattr(self, image.type)
+ if current and (current != 'No Cover') and not overwrite:
continue
setattr(self, image.type, image.filename)
getattr(self, image.type).downloadFrom(image.url)

0 comments on commit bd0f9eb

Please sign in to comment.