TypeError with DiscID "Rl7E4c2SbxEwLh9gwYX57V6pR6I-" (stubs) about musicbrainz-isrcsubmit HOT 6 CLOSED

I can verify this with setting the discID to this value (with any disc in the drive):

diff --git a/isrcsubmit.py b/isrcsubmit.py
index 2b92db7..46c406d 100755
--- a/isrcsubmit.py
+++ b/isrcsubmit.py
@@ -331,7 +331,8 @@ class Disc(object):

     @property
     def id(self):
-        return self._disc.getId()
+        return "Rl7E4c2SbxEwLh9gwYX57V6pR6I-"
+        #return self._disc.getId()

     @property
     def trackCount(self):

I can also verify the problem with isrcsubmit 0.4 (setting discID to this id).

It looks like a problem in the python-musicbrainz2 library. This would be bad, because python-musicbrainz2 is deprecated and might not receive bugfixes.

The "-" at the end of the id doesn't seem to be the problem. That is quite common and i tested another disc with such an id.

from musicbrainz-isrcsubmit.

Looking it up (via Picard) shows that the release has a stub. Perhaps this is what is throwing things off? (E.g., it's finding a (stub) release but no releaseId?)

https://musicbrainz.org/cdtoc/attach?id=Rl7E4c2SbxEwLh9gwYX57V6pR6I-&tracks=20&toc=1+20+296340+150+14760+30827+46300+60862+72887+86030+104495+121460+134050+149780+169217+185267+200284+211002+228445+240005+250897+265872+279385&tport=8000

from musicbrainz-isrcsubmit.

Oh, didn't see your comment. :)

from musicbrainz-isrcsubmit.

The problem in python-musicbrainz2 is, that the musicbrainz id (!= disc id) is "None". Possibly because this is a stub.

Bare test for pymb2:

#!/usr/bin/env python2

from musicbrainz2.webservice import Query, WebService, ReleaseFilter

agent = "isrcsubmit-test"
ws = WebService(userAgent=agent)
q = Query(ws, clientId=agent)
discId_filter = ReleaseFilter(discId="Rl7E4c2SbxEwLh9gwYX57V6pR6I-")
# working id
#discId_filter = ReleaseFilter(discId="KvJdT.MxfPnKhhaa8STa2MhBoWk-")
results = q.getReleases(filter=discId_filter)
release = results[0].release
print release.getArtist().getName(),
print "-", release.getTitle(),
#print "(" + release.getTypes()[1].rpartition('#')[2] + ")"
print "(" + ", ".join(release.getTypes()) + ")"
print "id: ", release.getId()

# this is the part that will error out
results = q.getReleaseById(release.getId())

from musicbrainz-isrcsubmit.

Oh boy..
After writing the bare test for pymb2 I relize, that I can just check for release.getId() is not None in my script..

from musicbrainz-isrcsubmit.

Stubs are now ignored (since they are no "real" musicbrainz releases), but I show some information about the stub with d8f8c3c.

Thanks for the report.

from musicbrainz-isrcsubmit.