[stokedfish]

Hmm... what happened to the recently added listing for music? Is it broken?



the album titles do not show up anymore...

newest dist-upgraded ubuntu release, xbmc skin from the add-on repo.

the listing still works fine for movies, weird.

[Hitcher]

I thought I'd better check this out for myself seeing as we (me and ppic) made some changes so you can actually play the whole album now and also fixed some incorrect database fields. Glad to report though that it's not the script's fault as it all work fine for me.

[ronie]

i can confirm this issue.
v2.0.3 of the script works fine (this is the one with the 'play full album fix')
v2.0.4 does not fetch the album title.

i run the script with the 'albums=True' option.

if i run it without, then the script does pull the album title from the db.

if i look at the script code, this line seems to be removed from the script:
self.WINDOW.setProperty( "LatestSong.%d.Album" % ( count + 1, ), fields[ 1 ] )

i'll ask ppic if he can have a look at it.

[ppic]

"LatestSong.%d.Title" is containing album title when using album=true

[ronie]

confirmed!

thanx, i'll change the skin code accordingly.

[Hitcher]

Sorry ronie I assumed you'd seen this -

I sent ppic an update for that yesterday.

EDIT: Here's the latest one I'm using if you want to check I've done correctly -

http://pastebin.com/13YTRtWY

Also note -

PHP Code:

"RandomSong.%d.Title" <!-- Returns the Song name when albums=False or Returns the Album name when albums=True -->
"RandomSong.%d.Album" <!-- Not used when albums=True --> 


Also added was a reset for the property LatestSong.%d.Album so you can use IsEmpty(Window.Property(LatestSong.%d.Album)) as a check for which mode you're in (Albums or Songs).

Thanks.

[ronie]

Sorry ronie I assumed you'd seen this

did see it and made the changes to the random items script back then.

i just missed the fact it was applied to recently added as well.


no problem, already fixed it in T! svn.