Bug 4695 - Touch/Nokia770 skins improperly refreshing artists
: Touch/Nokia770 skins improperly refreshing artists
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 6.5.1
: PC Windows XP
: P2 minor (vote)
: ---
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-21 07:52 UTC by Jeff Roesner
Modified: 2007-01-22 08:34 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeff Roesner 2007-01-21 07:52:48 UTC
In both the Touch and Nokia770 skins I have found that on the first page of the current playlist the browser will refresh after about 10 seconds and the artist of the first track will be assigned to all other tracks on that page.  This occurs in both straight and random play.  Other pages in the playlist don't exhibit this behavior.  

This happens on Firefox 2.0.0.1 and IE6 on Windows XP as well as whatever version of Opera is installed on the Nokia N800.
Comment 1 KDF 2007-01-21 23:48:11 UTC
ben, any ideas?
Comment 2 Ben Klaas 2007-01-22 08:17:30 UTC
well, the good news is I reproduced this easily. I was concerned this was going to be one of those nasty "windows only" bugs, but I see it too. Assigning this to myself and will look for a fix.
Comment 3 Ben Klaas 2007-01-22 08:34:29 UTC
fixed in Trunk and 6.5 branches with checkins 11259 and 11260.

bklaas@mediumspicy /usr/local/slim.trunk/HTML/Nokia770 $ svn diff playlist.js
Index: playlist.js
===================================================================
--- playlist.js (revision 11258)
+++ playlist.js (working copy)
@@ -87,7 +87,8 @@
                        $(thisId).innerHTML = innerText;
                }
        }
-       refreshElement('artist_' + r.toString(),parsedData['artist_0']);
+       var artist_elem = 'artist_' + r.toString();
+       refreshElement('artist_' + r.toString(),parsedData[artist_elem]);
        refreshItemClass(parsedData, r);
 }