Bug 11923 - Webpage get lost on refresh of detail screen music scan
: Webpage get lost on refresh of detail screen music scan
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 7.3.2
: PC Other
: P5 normal (vote)
: 7.3.3
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-04-28 23:37 UTC by slimdevices
Modified: 2009-06-17 09:37 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
fix meta refresh (1.28 KB, patch)
2009-04-29 13:15 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description slimdevices 2009-04-28 23:37:51 UTC
After using a iPaq PDA (Pocket PC - Win CE 4.2 + MS Compact Framework 3.5)as remote controller for a while (and with great succes) I have stumbled on a single problem in the Handheld skin. This bug description is originating from a support call #090415-001673 The messages are translated back from Dutch to English, so in real life they may be somewhat different..

What is the problem? 

When I start a full music scan, a message on the bottom of the main menu appears "SqueezeCenter is scanning your music etc." When I click on the "details Music Scan" link, all the releavnt info is displayed...
untill the first page refresh (and all after that) and the following message screen is displayed:

http://192.168.1.100:9000/import 

404 not found

importer&player= 00:04:20:07:6A:8f×temp:200904145207

This error is easy to reproduce. From this errorscreen backward, the correct info is displayed correct again, untill the next screen refresh (etc)

Please consider this:

The OS of the handheld is not supported and the problem doesn't appear on a Win-XP desktop. But the Handheld UI is written in a way that it should run on any handheld. That means the code framework is as simple as possible. Saying this: ALL the other functions of the webinterface are working fine on my PDA. Including the page (status) refresh when playing albums or tracks, which is much more complex than displaying some textlines derived from the scanning proces.

It seems to me that the code used for refreshing the scan proces status screen is somewhat different from the other screen refreshes. Is it possible to check this by the developer (Michael Herger)and make an update?

Peter Gelder
Comment 1 James Richardson 2009-04-29 08:08:55 UTC
Ben: are you the right person to assign the Hand Held skin issues to?
Comment 2 Ben Klaas 2009-04-29 08:27:23 UTC
Michael is the handheld skin author
Comment 3 KDF 2009-04-29 11:05:14 UTC
The code for the scan progress updates is basically the same as the status updates (Now Playing page) of the HandHeld skin.  Does the "status" page refresh correctly?

This is also strange:
importer&player= 00:04:20:07:6A:8f×temp:200904145207

It looks like a partial of the correct url, but it should be more like this format:
importer&player=00:04:20:07:6A:8f&timestamp=200904145207

I'm hoping it's a copy and paste error.  Otherwise, it is interesting that an extra space is added before the player ID, an & is changed to ×, timestamp turns into "temp" and the last '=' turns into -:'
Comment 4 Michael Herger 2009-04-29 11:42:34 UTC
No assignment without a target ;-)
Comment 5 slimdevices 2009-04-29 12:32:17 UTC
All the other page refreshes are going well. Even the more complex album/track status screens. As I wrote before: this is the only error I stumbled on.

The complete error code is:

404 Not Found:
importer&player=00:04:20:07:6a:8f&timestamp=200904292816

The URL above is: 
http://192.168.1.100:9000/importer%26player=00%3A04%#A20%3A07%3A6a%3A8f%26timestamp=200904292816

When you hit the backward arrow, the correct (and already updated) info is displayed.... until the next page refresh.

00:04:20:07:6a:8f is the Mac-adress of the Squeezebox (Ethernet).
Comment 6 slimdevices 2009-04-29 12:35:13 UTC
I Made a typing error... The URL should be this:

http://192.168.1.100:9000/importer%26player=00%3A04%3A20%3A07%3A6a%3A8f%26timestamp=200904292816
Comment 7 slimdevices 2009-04-29 12:45:39 UTC
I Made a typing error... The URL should be this:

http://192.168.1.100:9000/importer%26player=00%3A04%3A20%3A07%3A6a%3A8f%26timestamp=200904292816
Comment 8 KDF 2009-04-29 13:15:58 UTC
Created attachment 5169 [details]
fix meta refresh

add the missing "url=" from the meta refresh. not all browsers are happy without it.
Comment 9 slimdevices 2009-04-29 13:24:46 UTC
Dear KDF,

I have no clue how the information is retrieved,but is it logical that the status screen is referring to the Mac address of the Squeezebox instead of the 
Slimserver / Squeezecenter ?
Comment 10 KDF 2009-04-29 13:31:31 UTC
The mac address is correct.  This is how the currently selected player is tracked.  Users with multiple squeezeboxes can use any one they choose from the same web interface.
Comment 11 slimdevices 2009-04-29 14:26:01 UTC
Dear KDF,

That goes for the status screens when playing an album or track, but does it also when scanning the folders and files within the music collection and (re)indexing the MySQL database ?

Peter
Comment 12 KDF 2009-04-29 14:40:24 UTC
All web pages track the current player.  The scan status is merely a web representation of what is going on with scanning.  It is not the scan itself.

The only thing wrong is the refresh tag in the web page, which is corrected by the patch attached above.  If you feel comfortable editing a file, you can make the change manually to see if it works.  Failing that, I'm relying on Michael to review it when he gets a chance.
Comment 13 Michael Herger 2009-04-29 23:05:28 UTC
change 26303 - thanks kdf!

Peter - this should be fixed now. But as I don't have any PocketIE to test with, please re-open this bug if it wouldn't work for you. Thanks!
Comment 14 slimdevices 2009-04-30 07:50:02 UTC
Michael - I have tried the fix from the attachment and it didn't bring a solution for the problem. I renamed the original pageheader.html and copied the fixed one in the handheld directory. I checked if it was really the correct one
(with the "url=" fix) and started a scan. As before, the first screen is ok, but after the refresh it couldn't find the page. 

The first screen is the progress.html screen and when I hit the IE refresh button (even when the 404 screen is in sight), the same page (progress.html) with updated info is displayed.

Peter
Comment 15 slimdevices 2009-04-30 07:50:57 UTC
Reopened while fix doesn't work.
Comment 16 Michael Herger 2009-04-30 09:14:14 UTC
> Michael - I have tried the fix from the attachment and it didn't bring a
> solution for the problem.

If you're on Windows, then please test today's 7.3.3 nightly build (or  
tomorrow's on other systems). My fix is slightly different from Kevin's.
Comment 17 slimdevices 2009-04-30 13:17:25 UTC
Michael - I have downloaded version 7.3.3.26303 (april 30 03:11)
and I'm very pleased to say that it works fine now !!!
Thanks to you and Kevin to help me out with this.

best regards,

Peter Gelder
Comment 18 Michael Herger 2009-04-30 13:22:13 UTC
thanks for the feedback!
Comment 19 James Richardson 2009-05-04 09:57:13 UTC
Marking bug verified based on customer feed back, Comment #17
Comment 20 James Richardson 2009-06-17 09:37:26 UTC
This bug has been fixed in the 7.3.3 release version of SqueezeCenter!

If you haven't already. please download the new version from http://www.logitechsqueezebox.com/support/download-squeezecenter.html 

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.