Merge remote-tracking branch 'grunskis/develop' into develop

This commit is contained in:
Stein Magnus Jodal 2011-04-05 21:37:21 +02:00
commit dec1643e7f
3 changed files with 11 additions and 3 deletions

View File

@ -100,8 +100,11 @@ def _convert_mpd_data(data, tracks, music_dir):
albumartist_kwargs = {}
if 'track' in data:
album_kwargs['num_tracks'] = int(data['track'].split('/')[1])
track_kwargs['track_no'] = int(data['track'].split('/')[0])
if '/' in data['track']:
album_kwargs['num_tracks'] = int(data['track'].split('/')[1])
track_kwargs['track_no'] = int(data['track'].split('/')[0])
else:
track_kwargs['track_no'] = int(data['track'])
if 'artist' in data:
artist_kwargs['name'] = data['artist']

View File

@ -85,7 +85,7 @@ def artists_to_mpd_format(artists):
:rtype: string
"""
artists.sort(key=lambda a: a.name)
return u', '.join([a.name for a in artists])
return u', '.join([a.name for a in artists if a.name])
def tracks_to_mpd_format(tracks, start=0, end=None, cpids=None):
"""

View File

@ -95,6 +95,11 @@ class TrackMpdFormatTest(unittest.TestCase):
translated = translator.artists_to_mpd_format(artists)
self.assertEqual(translated, u'ABBA, Beatles')
def test_artists_to_mpd_format_artist_with_no_name(self):
artists = [Artist(name=None)]
translated = translator.artists_to_mpd_format(artists)
self.assertEqual(translated, u'')
class PlaylistMpdFormatTest(unittest.TestCase):
def test_mpd_format(self):