momodalo / rubyripper

Automatically exported from code.google.com/p/rubyripper
3 stars 2 forks source link

musicbrainz: wrong debug output #550

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
When I run rubyripper with debug and verbose enabled and musicbrainz, console 
output shows me something alike this:

DEBUG: CGI query: http://musicbrainz.org/ws/2//ws/2/discid/[...]

Now, there's obviously something wrong: ws/2 is there twice. However, it seems 
that this is only a bug in the debug output, the lookup goes to the correct URL.

Attached patch should do the trick (but please review, as I'm not 100% sure if 
this works in all situations)

Original issue reported on code.google.com by hanno@hboeck.de on 27 Dec 2012 at 8:16

Attachments:

GoogleCodeExporter commented 8 years ago
The same code is shared with freedb lookup. The problem is clear indeed. Did 
enough for tonight, I want a beer ;).

Original comment by boukewou...@gmail.com on 27 Dec 2012 at 9:10

GoogleCodeExporter commented 8 years ago
Fixed with commit (thanks for the patch):
http://code.google.com/p/rubyripper/source/detail?r=638fb5720f64a1ce5d9224905ae2
56de94c992cc

Original comment by boukewou...@gmail.com on 2 Jan 2013 at 6:23