Tracks without track number get 0

also applies to disc number.

Corrects the bug, needs a review.

This fixes the bug for me.

I haven't really looked at the patch thoroughly, but

- if track == -1:
+ if not track:
             cell.set_property('text', '')

Should this be "if track is None"?

@Johannes Sasongko, What is the difference between "if not track" and "if track is None"?

I haven't coded so much in Python and I actually don't reallyy no the good practices.

Thanks for the quick review ;)

"if not track" could catch any case where track evaluates to False, so None, 0, False, any empty container, etc. would all match this case. "if track is not None" explicitly checks it against None, so 0, False, etc. won't match. While not a big deal for this situation, it's a good habit to use as it makes your code more robust. Additionally, it also hints to people reading your code that you _expect_ track to be None sometimes, which is useful information.

Thanks for your clear and neat explanations ;)

patch committed trunk/2891, thanks!