Fixed some inconsistencies

diff --git a/sima/client.py b/sima/client.py
index 9eae523bd574e209d5e0c7ff8459e532b90d9370..7c0e5a7c365758d1cced6d0b412b88486a6abbc8 100644 (file)
--- a/sima/client.py
+++ b/sima/client.py
@@ -217,7 +217,7 @@ class PlayerClient(Player):
         # Check against the actual string in artist list
         if artist.orig in self.artists:
             self.log.debug('found exact match for "%s"' % artist)
-            return [artist]
+            return [artist.orig]
         # Then proceed with fuzzy matching if got nothing
         match = get_close_matches(artist.orig, self.artists, 50, 0.73)
         if not match:

diff --git a/sima/lib/track.py b/sima/lib/track.py
index 55fb3675403d81507b12fd1b87419e46c1e51a4c..5b25bd33527bad5e94ef2eb043990a1eb4dc755b 100644 (file)
--- a/sima/lib/track.py
+++ b/sima/lib/track.py
@@ -128,8 +128,10 @@ class Track:
 
     def get_artist(self):
        """Get artist object from track"""
-       return Artist(name=self.albumartist or self.artist,
-               mbid=self.musicbrainz_artistid)
+       name = self.artist
+       if self.albumartist and self.albumartist != 'Various Artists':
+           name = self.albumartist
+       return Artist(name=name, mbid=self.musicbrainz_artistid)
 
 # VIM MODLINE
 # vim: ai ts=4 sw=4 sts=4 expandtab

diff --git a/sima/lib/webserv.py b/sima/lib/webserv.py
index f1afd55a41b7e2f279fed0d3ec0d02bd28ec6594..a4fb056eca87cb820768779b0d697f6b0b9eae4a 100644 (file)
--- a/sima/lib/webserv.py
+++ b/sima/lib/webserv.py
@@ -143,8 +143,7 @@ class WebService(Plugin):
         # TODO: move to utils as a decorator
         duration = self.daemon_conf.getint('sima', 'history_duration')
         art_in_hist = list()
-        for trk in self.sdb.get_history(duration=duration,
-                                        artists=alist):
+        for trk in self.sdb.get_history(duration=duration, artists=alist):
             if trk[0] not in art_in_hist:
                 art_in_hist.append(trk[0])
         art_in_hist.reverse()
@@ -179,8 +178,7 @@ class WebService(Plugin):
         # initialize artists deque list to construct from DB
         as_art = deque()
         as_artists = self.ws().get_similar(artist=artist)
-        self.log.debug('Requesting {1} for "{0}"'.format(artist,
-                                                         self.ws.name))
+        self.log.debug('Requesting {} for {!r}'.format(self.ws.name,artist))
         try:
             # TODO: let's propagate Artist type
             [as_art.append(str(art)) for art in as_artists]
@@ -251,6 +249,9 @@ class WebService(Plugin):
         if ret & queued_artists:
             self.log.debug('Removing already queued artist: {0}'.format(ret & queued_artists))
             ret = list(ret - queued_artists)
+        if self.player.current.artist in ret:
+            self.log.debug('Removing current artist: {0}'.format(self.player.current.artist))
+            ret = list(ret - {self.player.current.artist})
         # Move around similars items to get in unplayed|not recently played
         # artist first.
         return self._get_artists_list_reorg(ret)