From e50bc12a889d4b98930fce6cfbc610b520a180db Mon Sep 17 00:00:00 2001
From: Laurent Monin <github@norz.org>
Date: Wed, 28 May 2014 15:28:06 +0200
Subject: [PATCH] Remove unneeded wrapper methods

---
 picard/coverart.py                   | 8 ++------
 picard/coverartproviders/__init__.py | 7 -------
 picard/coverartproviders/caa.py      | 5 +++--
 3 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/picard/coverart.py b/picard/coverart.py
index ebd0867a9..898d56e97 100644
--- a/picard/coverart.py
+++ b/picard/coverart.py
@@ -158,7 +158,7 @@ class CoverArt:
             echo=None
         )
         log.debug("Downloading %r" % coverartimage)
-        self.xmlws_download(
+        self.album.tagger.xmlws.download(
             coverartimage.host,
             coverartimage.port,
             coverartimage.path,
@@ -166,6 +166,7 @@ class CoverArt:
             priority=True,
             important=False
         )
+        self.album._requests += 1
 
     def queue_put(self, coverartimage):
         "Add an image to queue"
@@ -188,11 +189,6 @@ class CoverArt:
         """Display message to status bar"""
         QObject.tagger.window.set_statusbar_message(*args, **kwargs)
 
-    def xmlws_download(self, *args, **kwargs):
-        """xmlws.download wrapper"""
-        self.album._requests += 1
-        self.album.tagger.xmlws.download(*args, **kwargs)
-
 
 def coverart(album, metadata, release):
     """Gets all cover art URLs from the metadata and then attempts to
diff --git a/picard/coverartproviders/__init__.py b/picard/coverartproviders/__init__.py
index 4ad6931cd..e5087220c 100644
--- a/picard/coverartproviders/__init__.py
+++ b/picard/coverartproviders/__init__.py
@@ -49,7 +49,6 @@ class CoverArtProvider:
         self.release = coverart.release
         self.metadata = coverart.metadata
         self.album = coverart.album
-        self.xmlws_download = coverart.xmlws_download
 
     def enabled(self):
         return True
@@ -57,12 +56,6 @@ class CoverArtProvider:
     def queue_downloads(self):
         raise NotImplementedError
 
-    def requests_count_increment(self):
-        self.coverart.album._requests += 1
-
-    def requests_count_decrement(self):
-        self.coverart.album._requests -= 1
-
     def error(self, msg):
         self.coverart.album.error_append(msg)
 
diff --git a/picard/coverartproviders/caa.py b/picard/coverartproviders/caa.py
index 57ce74e91..f258326ea 100644
--- a/picard/coverartproviders/caa.py
+++ b/picard/coverartproviders/caa.py
@@ -105,7 +105,7 @@ class CoverArtProviderCaa(CoverArtProvider):
         return caa_has_suitable_artwork
 
     def queue_downloads(self):
-        self.xmlws_download(
+        self.album.tagger.xmlws.download(
             CAA_HOST,
             CAA_PORT,
             "/release/%s/" % self.metadata["musicbrainz_albumid"],
@@ -113,12 +113,13 @@ class CoverArtProviderCaa(CoverArtProvider):
             priority=True,
             important=False
         )
+        self.album._requests += 1
         # we will call next_in_queue() after json parsing
         return CoverArtProvider.WAIT
 
     def _caa_json_downloaded(self, data, http, error):
         """Parse CAA JSON file and queue CAA cover art images for download"""
-        self.requests_count_decrement()
+        self.album._requests -= 1
         if error:
             self.error(u'CAA JSON error: %s' % (unicode(http.errorString())))
         else: