<div dir="ltr"><div>Success!  </div><div>Two of two retries now return the Artist, Album and tracks correctly.</div><div>Thanks Alan and Steve for the diagnosis and the fix.   Brilliant.</div><div>-Ed</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 20, 2016 at 6:10 PM, Steve McIntyre <span dir="ltr"><<a href="mailto:steve@einval.com" target="_blank">steve@einval.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Sat, Feb 20, 2016 at 01:46:56PM -0700, Alan W. Kerr wrote:<br>
><br>
>in a statement in the "HERE" block.  So, lines 98-101 are:<br>
><br>
>       my @events = @{$release->release_event_list()->events()};<br>
>       my $rel_year =  substr($events[0]->date(),0,4);<br>
><br>
>        $releasenum++;<br>
><br>
>and the problem seems to be the "date()" command.  Which might also explain<br>
>why this happens infrequently...  you probably have few CD that put you in<br>
>the "HERE" block.<br>
><br>
>SO!  Finally...  to the programmers/maintainers of this:<br>
><br>
>1) is there an error here?  can someone please fix it?<br>
>2) if there is NOT an error (the problem is in the musicbrainz database),<br>
>then can someone add some perl logic to gracefully skip this error and<br>
>continue?<br>
<br>
</span>Thanks Alan, this looks like a good catch. That code was added in the<br>
last update to abcde-musicbrainz-tool back in October last year:<br>
<br>
commit 64a362f4ad828821eb27c7cb14ceeaf14312eab1<br>
Author: Steve McIntyre <<a href="mailto:steve@einval.com">steve@einval.com</a>><br>
Date:   Sat Oct 3 12:13:29 2015 +0100<br>
<br>
    Grab year information too when using musicbrainz<br>
<br>
    Thanks to Marco Hoppstaedter for the patch. Closes issue 10:<br>
    <a href="http://abcde.einval.com/bugzilla/show_bug.cgi?id=10" target="_blank" rel="noreferrer">http://abcde.einval.com/bugzilla/show_bug.cgi?id=10</a><br>
<br>
It looks to me like maybe there are some releases without event<br>
information, and that causes confusion. I've not seen this myself on<br>
any CDs here that I'm aware of, but that could just be dumb luck of<br>
course.<br>
<br>
If you change the code<br>
<span><br>
        my $rel_year =  substr($events[0]->date(),0,4);<br>
<br>
</span>to<br>
<br>
        my $rel_year = "";<br>
        if ($release->release_event_list()->count() > 0) {<br>
<span>            $rel_year =  substr($events[0]->date(),0,4);<br>
        }<br>
<br>
</span>and try again with one of your problem CDs, does it work better?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Steve McIntyre, Cambridge, UK.                                <a href="mailto:steve@einval.com">steve@einval.com</a><br>
Support the Campaign for Audiovisual Free Expression: <a href="http://www.eff.org/cafe/" target="_blank" rel="noreferrer">http://www.eff.org/cafe/</a><br>
<br>
</font></span></blockquote></div><br></div>