PICARD-187: Support for manually removing cover art

PICARD-187: Support for manually removing cover art

finishing PICARD-187

Author: ShubhamBhut

picard/formats/apev2.py

@@ -263,6 +263,11 @@ def _remove_deleted_tags(self, metadata, tags):
                 if real_name in tags:
                     del tags[real_name]
 
+        if self.metadata.images._deleted:
+            for tag in tags:
+                if tag.lower().startswith('cover art'):
+                    del tags[tag]
+
     def _get_tag_name(self, name):
         if name in self.__casemap:
             return self.__casemap[name]

picard/formats/asf.py

@@ -304,6 +304,8 @@ def _remove_deleted_tags(self, metadata, tags):
             real_name = self._get_tag_name(tag)
             if real_name and real_name in tags:
                 del tags[real_name]
+        if self.metadata.images._deleted:
+            del tags['WM/Picture']
 
     @classmethod
     def supports_tag(cls, name):

picard/formats/id3.py

@@ -659,6 +659,11 @@ def _remove_deleted_tags(self, metadata, tags):
             except KeyError:
                 pass
 
+        if self.metadata.images._deleted:
+            for key, frame in list(tags.items()):
+                if frame.FrameID == 'APIC':
+                    del tags[key]
+
     @classmethod
     def supports_tag(cls, name):
         return ((name and not name.startswith('~') and name not in UNSUPPORTED_TAGS)

picard/formats/mp4.py

@@ -340,6 +340,9 @@ def _remove_deleted_tags(self, metadata, tags):
                 if tag not in {'totaltracks', 'totaldiscs'}:
                     del tags[real_name]
 
+        if self.metadata.images._deleted:
+            del tags['covr']
+
     @classmethod
     def supports_tag(cls, name):
         return (name

picard/formats/vorbis.py

@@ -321,7 +321,7 @@ def _save(self, filename, metadata):
                     base64.b64encode(picture.write()).decode('ascii'))
 
         file.tags.update(tags)
-
+        self._clear_cover_art(file)
         self._remove_deleted_tags(metadata, file.tags)
 
         kwargs = {}
@@ -357,6 +357,17 @@ def _remove_deleted_tags(self, metadata, tags):
                         # both tag and real_name are to be deleted in this case
                         del tags[tag]
                     del tags[real_name]
+    
+    def _clear_cover_art(self, file):
+        is_flac = self._File == mutagen.flac.FLAC
+        
+        if self.metadata.images._deleted:
+            if is_flac:
+                file.clear_pictures()
+            else:
+                for tag in file:
+                    if tag.lower().startswith('metadata_block_picture'):
+                        del file.tags[tag]
 
     def _get_tag_name(self, name):
         if name == '~rating':

picard/ui/coverartbox.py

@@ -347,6 +347,9 @@ def open_release_page(self):
         lookup = self.tagger.get_file_lookup()
         lookup.album_lookup(self.release)
 
+def image_delete(obj, image_index):
+    obj.metadata.images.strip_selected_image(image_index)
+    obj.metadata_images_changed.emit()
 
 def set_image_replace(obj, coverartimage):
     obj.metadata.images.strip_front_images()
@@ -545,6 +548,28 @@ def load_remote_image(self, url, data):
             log.warning("Can't load image: %s", e)
             return
 
+    def delete_cover_art(self):
+        if not self.item or not self.item.can_show_coverart:
+            return
+
+        metadata = self.item.metadata
+        if not metadata or not metadata.images:
+            return
+
+        cover_art_list = [image.source or _("Unnamed Cover Art") for image in metadata.images]
+
+        selected_item, ok_pressed = QtWidgets.QInputDialog.getItem(self, _("Delete Cover Art"),
+            _("Select the cover art image to delete:"), cover_art_list, 0, False)
+
+        if ok_pressed:
+            selected_index = cover_art_list.index(selected_item)
+            if selected_index < len(metadata.images):
+                image_delete(self.item, selected_index)
+                self.update_display(force=True)
+            else:
+                QtWidgets.QMessageBox.warning(self, _("Invalid Selection"),
+                    _("Please select a valid cover art image."))
+
     def _try_load_remote_image(self, url, data):
         coverartimage = CoverArtImage(
             url=url.toString(),
@@ -634,6 +659,12 @@ def contextMenuEvent(self, event):
             show_more_details_action.triggered.connect(self.show_cover_art_info)
             menu.addAction(show_more_details_action)
 
+        if self.item and self.item.can_show_coverart:
+            name = _("Delete cover art")
+            delete_cover_art_action = QtGui.QAction(name, self.parent)
+            delete_cover_art_action.triggered.connect(self.delete_cover_art)
+            menu.addAction(delete_cover_art_action)
+
         if self.orig_cover_art.isVisible():
             name = _("Keep original cover art")
             use_orig_value_action = QtGui.QAction(name, self.parent)

picard/util/imagelist.py

@@ -33,6 +33,7 @@ def __init__(self, iterable=()):
         self._images = list(iterable)
         self._hash_dict = {}
         self._changed = True
+        self._deleted = False
 
     def __len__(self):
         return len(self._images)
@@ -50,6 +51,7 @@ def __delitem__(self, index):
 
     def insert(self, index, value):
         self._changed = True
+        self._deleted = False
         return self._images.insert(index, value)
 
     def __repr__(self):
@@ -95,6 +97,14 @@ def strip_front_images(self):
         self._images = [image for image in self._images if not image.is_front_image()]
         self._changed = True
 
+    def strip_selected_image(self, image_index):
+        if len(self._images) > image_index:
+            del self._images[image_index]
+            if len(self._images) == 0:
+                self._deleted = True
+        else:
+            raise IndexError("Invalid image index")
+
     def hash_dict(self):
         if self._changed:
             self._hash_dict = {img.datahash.hash(): img for img in self._images}