Bug 372731 - Error on loading jamendo catalog
Summary: Error on loading jamendo catalog
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: rhythmbox
Version: 7
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Bastien Nocera
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-11-09 14:05 UTC by Seby Carta
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version: 0.10.1-6.fc7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-11-26 18:54:47 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Seby Carta 2007-11-09 14:05:59 UTC
Description of problem:
When loading jamendo catalog i obtain the following traceback:
----------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib64/rhythmbox/plugins/jamendo/JamendoSource.py", line 162, in
finish_loadscreen
    self.__load_db ()
  File "/usr/lib64/rhythmbox/plugins/jamendo/JamendoSource.py", line 321, in
__load_db
    self.__db.set(entry, rhythmdb.PROP_TRACK_NUMBER, int(track['trackno']))
TypeError: could not convert argument from type '(null)' (0) to correct type
'gulong'
----------------------------------------------------------------

Version-Release number of selected component (if applicable):

Rhytmbox 0.10.1

How reproducible:
Always

Steps to Reproduce:
1.Open Rhytmbox
2.Click on "jamendo"
  
Actual results:
Nothing happen ... you will see the traceback

Expected results:
Load jamendo catalog

Additional info:

This problem is also in F8

Comment 1 Bastien Nocera 2007-11-12 03:45:27 UTC
Was fixed in:
http://svn.gnome.org/viewvc/rhythmbox?view=revision&revision=5426
and is already fixed in 0.11.3 (which will be in F8 and rawhide shortly).

Comment 2 Bastien Nocera 2007-11-12 04:09:27 UTC
Update built, test packages at:
http://koji.fedoraproject.org/koji/buildinfo?buildID=23973

Comment 3 Fedora Update System 2007-11-13 00:10:33 UTC
rhythmbox-0.10.1-6.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update rhythmbox'

Comment 4 Seby Carta 2007-11-13 07:00:34 UTC
The version suggested by you has the same problem!

I modified the /usr/lib64/rhythmbox/plugins/jamendo/JamendoSource.py in this way:

line 321:

self.__db.set(entry, rhythmdb.PROP_TRACK_NUMBER, int(track['trackno']))

convert in:

trackno = int(track['trackno'])
if trackno >= 0:
   self.__db.set(entry, rhythmdb.PROP_TRACK_NUMBER, trackno)


and now works.

Comment 5 Bastien Nocera 2007-11-13 12:07:36 UTC
This is already the patch I added to rhythmbox in Fedora 7:
http://cvs.fedora.redhat.com/viewcvs/rpms/rhythmbox/F-7/rhythmbox-0.10.1-jamendo-neg-tracknum.patch?view=markup

What's the output of "rpm -qa rhythmbox" on your machine?

Comment 6 Seby Carta 2007-11-13 12:36:00 UTC
okay .. i didn't look at version , i have rhythmbox-0.10.1-5.fc7 
there is no rhythmbox-0.10.1-6.fc7 in the testing yet ... i'll wait

Thank you.

Comment 7 Bastien Nocera 2007-11-13 13:23:01 UTC
(In reply to comment #6)
> okay .. i didn't look at version , i have rhythmbox-0.10.1-5.fc7 
> there is no rhythmbox-0.10.1-6.fc7 in the testing yet ... i'll wait
> 
> Thank you.

I posted the URL to download them by hand already. See comment 2

Comment 8 Fedora Update System 2007-11-26 18:54:45 UTC
rhythmbox-0.10.1-6.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.