Release Artist Slideshow addon (with skin and addon integration)
Bug 
Hi,

at first: thank for this beautiful plugin. Really good work! I think I've found a bug while playing music via Airplay.
The first artist art is loading but then artist slideshow keeps artist art of the first title even the next songs are played.
A deeper look into the code showed something that I changed. In line 706 I changed:

Code:
playing_file = xbmc.Player().getPlayingFile()
to
Code:
playing_id = xbmc.Player().getMusicInfoTag().getTitle() + xbmc.Player().getPlayingFile()
and of course in line 713ff
Code:
if playing_id != self.LASTPLAYINGFILE:    
  # if the same file is playing, use cached JSON response instead of doing a new query
                response = xbmc.executeJSONRPC ( '{"jsonrpc":"2.0", "method":"Player.GetItem", "params":{"playerid":0, "properties":["artist", "musicbrainzartistid"]},"id":1}' )
                self.LASTPLAYINGFILE = playing_id
                self.LASTJSONRESPONSE = response


xbmc.Player().getPlayingFile() always return something like stream://pipe1 and did not change while the airplay stream
was continous.


I am not a Python developer and have no deep knowledge of the xbmc-api so you have to verify my change. For music from the database and internet radio my code also did the trick.

Also there is a smaller bug if you have turned on htbackdrops for artist retrieval. Since you removed the lastFM support the variables for image width and height caused an error because of missing variables(line 877 in default.py). I put in some hardcoded width and height in the url for htbackdrops and solved this for me.
Perhaps you could find a better solution here.

It would be nice if you could add a fix, so updates don't let me lose airplay-support.

Keep up your nice work!

Best regards
Ralf
Reply


Messages In This Thread
New Test Version Available - by pkscout - 2012-03-08, 16:26
[No subject] - by mardup - 2012-03-10, 12:26
[No subject] - by pkscout - 2012-03-11, 01:35
RE: - by mardup - 2012-03-12, 17:50
RE: - by pkscout - 2012-03-13, 03:22
New Beta Version for Testing - by pkscout - 2012-07-13, 17:54
New Beta Version for Testing - by pkscout - 2012-07-14, 22:50
Pull Request for v1.4 Submitted - by pkscout - 2012-07-20, 23:01
New Beta Version for Testing - by pkscout - 2012-08-10, 03:18
New Frodo BETA Support Files - by pkscout - 2012-11-21, 06:20
Change artist while playing via Airplay - by felle42 - 2013-12-01, 22:04
New 2.0.0 Beta Coming Soon - by pkscout - 2016-05-12, 02:54
Writing tips - by Parkerbup - 2017-02-03, 14:56
Add-on Artist Slideshow - by jo26 - 2014-08-08, 22:58
Logout Mark Read Team Forum Stats Members Help
Artist Slideshow addon (with skin and addon integration)5