* morituri/common/musicbrainzngs.py:

Fix case of metadata not having length.
	  Fix case of release not having a date.
This commit is contained in:
Thomas Vander Stichele
2011-10-28 16:36:39 +00:00
parent b5fc509747
commit cd6865c9d2
2 changed files with 11 additions and 3 deletions

View File

@@ -1,3 +1,9 @@
2011-10-28 Thomas Vander Stichele <thomas at apestaart dot org>
* morituri/common/musicbrainzngs.py:
Fix case of metadata not having length.
Fix case of release not having a date.
2011-10-22 Thomas Vander Stichele <thomas at apestaart dot org>
* morituri/common/Makefile.am:

View File

@@ -24,7 +24,6 @@
Handles communication with the musicbrainz server using NGS.
"""
import urlparse
import urllib2
from morituri.common import log
@@ -106,7 +105,10 @@ def _getMetadata(release, discid):
metadata.artist = artist['name']
metadata.sortName = artist['sort-name']
# FIXME: is format str ?
metadata.release = release['date']
if not release.has_key('date'):
log.warning('musicbrainzngs', 'Release %r does not have date', release)
else:
metadata.release = release['date']
metadata.mbid = release['id']
metadata.mbidArtist = artist['id']
@@ -151,7 +153,7 @@ def _getMetadata(release, discid):
track.mbid = t['recording']['id']
# FIXME: unit of duration ?
track.duration = int(t['recording']['length'])
track.duration = int(t['recording'].get('length', 0))
if not track.duration:
log.warning('getMetadata',
'track %r (%r) does not have duration' % (