From 46aeb3bfcc913c0ff01f31dbd0d9171129f2d296 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Sat, 19 Oct 2013 20:15:38 +0200 Subject: [PATCH] core.tracklist: Move logging into mark_unplayable() --- mopidy/core/playback.py | 1 - mopidy/core/tracklist.py | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/mopidy/core/playback.py b/mopidy/core/playback.py index b0e0f2e1..d127fbbe 100644 --- a/mopidy/core/playback.py +++ b/mopidy/core/playback.py @@ -237,7 +237,6 @@ class PlaybackController(object): self.core.tracklist.mark_playing(tl_track) self._trigger_track_playback_started() else: - logger.warning('Track is not playable: %s', tl_track.track.uri) self.core.tracklist.mark_unplayable(tl_track) if on_error_step == 1: # TODO: can cause an endless loop for single track repeat. diff --git a/mopidy/core/tracklist.py b/mopidy/core/tracklist.py index 125843b2..940eb0ec 100644 --- a/mopidy/core/tracklist.py +++ b/mopidy/core/tracklist.py @@ -438,6 +438,7 @@ class TracklistController(object): def mark_unplayable(self, tl_track): """Private method used by :class:`mopidy.core.PlaybackController`.""" + logger.warning('Track is not playable: %s', tl_track.track.uri) if self.random and self._shuffled: self._shuffled.remove(tl_track)