| Summary: | abcde-musicbrainz-tool bug in release year | ||
|---|---|---|---|
| Product: | abcde | Reporter: | tecknicaltom <einval> |
| Component: | CD lookup | Assignee: | Steve McIntyre <steve> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | abcde, andrew.david.strong, luca.defeo, travisbeale |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| Attachments: | patch for fix | ||
Applied, thanks! *** Bug 62 has been marked as a duplicate of this bug. *** *** Bug 65 has been marked as a duplicate of this bug. *** |
Created attachment 47 [details] patch for fix The fix for #30 implemented in 1f1d33c64206841195a196595ed48f78952c07e6 relies on the release-event-list node from MB containing a count attribute. However in my testing, that isn't the case, and so the release year is never populated. This change checks that the release_event_list isn't undef before trying to get the events (the cause of the exception in #30) but then doesn't rely on the count() of the release_event_list.