IndexError: list index out of range

[crotalus]

When I click on an artist, I get 98% of the time:

This artist failed 0
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/AmpacheTools/AmpacheSession.py", line 360, in get_albums_by_artist
    album_disk = int(child.getElementsByTagName('disk')[0].childNodes[0].data)
IndexError: list index out of range

I am using ampache 3.5.4, but the same happened to ampache 3.6 alpha1.

I few artists succeed, and I can play whatever song are shown on the dialog area.

Of course, my ampache installation is working fine. This is on a Fedora 14.

Thanks.

[Dave Eddy (daveeddydotcom)]

Which version of viridian are you using/how did you install it?

[crotalus (promac)]

Viridian 1.1.1

I created an rpm for it, but also running it without installing,
the result is the same.

I was hoping to make Viridian available for the Fedora community.

Thanks.

[Dave Eddy (daveeddydotcom)]

Well i'm glad to see Viridian being made available to the fedora community!

This issue seems to be something that I have dealt with with Ampache before, and it is an issue with the XML being returned from Ampache and the XML parser I'm using for Viridian breaking. I will commit a fix for this issue after the holidays, but for now if you would like to fix this issue it can most likely be fixed by finding the line in AmpacheSession.py that looks like this...

album_disk = int(child.getElementsByTagName('disk')[0].childNodes[0].data)

and replacing it with this

try:
        album_disk = int(child.getElementsByTagName('disk')[0].childNodes[0].data)
except:
        album_disk = 0

[crotalus (promac)]

That is it! It is working just fine now.

I have the patched source rpm available here:

http://people.atrpms.net/~pcavalcanti/srpms/viridian-1.1.1-1.fc14.src.rpm

and the spec file, here:

http://people.atrpms.net/~pcavalcanti/specs/viridian.spec

Thanks.

[crotalus (promac)]

Viridian can be downloaded for Fedora/Rhel, from ATrpms:

http://packages.atrpms.net/name/viridian/

Thanks.

[Dave Eddy (daveeddydotcom)]

That's awesome thank you!

[Dave Eddy (daveeddydotcom)]

I have added this patch to the newest release, version 1.2