diff --git a/mopidy/mpd/protocol/__init__.py b/mopidy/mpd/protocol/__init__.py index e6b88dbd..b69d5a2a 100644 --- a/mopidy/mpd/protocol/__init__.py +++ b/mopidy/mpd/protocol/__init__.py @@ -22,8 +22,8 @@ ENCODING = 'UTF-8' #: The MPD protocol uses ``\n`` as line terminator. LINE_TERMINATOR = '\n' -#: The MPD protocol version is 0.17.0. -VERSION = '0.17.0' +#: The MPD protocol version is 0.19.0. +VERSION = '0.19.0' def load_protocol_modules(): diff --git a/mopidy/mpd/translator.py b/mopidy/mpd/translator.py index 8359f86b..2cd9e3ad 100644 --- a/mopidy/mpd/translator.py +++ b/mopidy/mpd/translator.py @@ -37,7 +37,7 @@ def track_to_mpd_format(track, position=None, stream_title=None): # TODO: only show length if not none, see: # https://github.com/mopidy/mopidy/issues/923#issuecomment-79584110 ('Time', track.length and (track.length // 1000) or 0), - ('Artist', artists_to_mpd_format(track.artists)), + ('Artist', concat_multi_values(track.artists, 'name')), ('Title', track.name or ''), ('Album', track.album and track.album.name or ''), ] @@ -58,26 +58,27 @@ def track_to_mpd_format(track, position=None, stream_title=None): result.append(('Id', tlid)) if track.album is not None and track.album.musicbrainz_id is not None: result.append(('MUSICBRAINZ_ALBUMID', track.album.musicbrainz_id)) - # FIXME don't use first and best artist? - # FIXME don't duplicate following code? + if track.album is not None and track.album.artists: - artists = artists_to_mpd_format(track.album.artists) - result.append(('AlbumArtist', artists)) - artists = [ - a for a in track.album.artists if a.musicbrainz_id is not None] - if artists: - result.append( - ('MUSICBRAINZ_ALBUMARTISTID', artists[0].musicbrainz_id)) + result.append( + ('AlbumArtist', concat_multi_values(track.album.artists, 'name'))) + musicbrainz_ids = concat_multi_values( + track.album.artists, 'musicbrainz_id') + if musicbrainz_ids: + result.append(('MUSICBRAINZ_ALBUMARTISTID', musicbrainz_ids)) + if track.artists: - artists = [a for a in track.artists if a.musicbrainz_id is not None] - if artists: - result.append(('MUSICBRAINZ_ARTISTID', artists[0].musicbrainz_id)) + musicbrainz_ids = concat_multi_values(track.artists, 'musicbrainz_id') + if musicbrainz_ids: + result.append(('MUSICBRAINZ_ARTISTID', musicbrainz_ids)) if track.composers: - result.append(('Composer', artists_to_mpd_format(track.composers))) + result.append( + ('Composer', concat_multi_values(track.composers, 'name'))) if track.performers: - result.append(('Performer', artists_to_mpd_format(track.performers))) + result.append( + ('Performer', concat_multi_values(track.performers, 'name'))) if track.genre: result.append(('Genre', track.genre)) @@ -90,17 +91,24 @@ def track_to_mpd_format(track, position=None, stream_title=None): return result -def artists_to_mpd_format(artists): +def concat_multi_values(models, attribute): """ - Format track artists for output to MPD client. + Format mopidy model values for output to MPD client. - :param artists: the artists - :type track: array of :class:`mopidy.models.Artist` + :param models: the models + :type models: array of :class:`mopidy.models.Artist`, + :class:`mopidy.models.Album` or :class:`mopidy.models.Track` + :param attribute: the attribute to use + :type attribute: string :rtype: string """ - artists = list(artists) - artists.sort(key=lambda a: a.name) - return ', '.join([a.name for a in artists if a.name]) + # Don't sort the values. MPD doesn't appear to (or if it does it's not + # strict alphabetical). If we just use them in the order in which they come + # in then the musicbrainz ids have a higher chance of staying in sync + return ';'.join( + getattr(m, attribute) + for m in models if getattr(m, attribute, None) is not None + ) def tracks_to_mpd_format(tracks, start=0, end=None): diff --git a/tests/mpd/test_translator.py b/tests/mpd/test_translator.py index 646a22b2..99c87dad 100644 --- a/tests/mpd/test_translator.py +++ b/tests/mpd/test_translator.py @@ -103,14 +103,19 @@ class TrackMpdFormatTest(unittest.TestCase): result = translator.track_to_mpd_format(track) self.assertIn(('MUSICBRAINZ_ARTISTID', 'foo'), result) - def test_artists_to_mpd_format(self): + def test_concat_multi_values(self): artists = [Artist(name='ABBA'), Artist(name='Beatles')] - translated = translator.artists_to_mpd_format(artists) - self.assertEqual(translated, 'ABBA, Beatles') + translated = translator.concat_multi_values(artists, 'name') + self.assertEqual(translated, 'ABBA;Beatles') - def test_artists_to_mpd_format_artist_with_no_name(self): + def test_concat_multi_values_artist_with_no_name(self): artists = [Artist(name=None)] - translated = translator.artists_to_mpd_format(artists) + translated = translator.concat_multi_values(artists, 'name') + self.assertEqual(translated, '') + + def test_concat_multi_values_artist_with_no_musicbrainz_id(self): + artists = [Artist(name='Jah Wobble')] + translated = translator.concat_multi_values(artists, 'musicbrainz_id') self.assertEqual(translated, '')