Bug 521 - /html/controller.html broken for some button modes
: /html/controller.html broken for some button modes
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 5.x or older
: PC Windows (legacy)
: P2 normal (vote)
: Future
Assigned To: Blackketter Dean
http://127.0.0.1:9000/html/remote.html
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-08-29 11:32 UTC by Pbox
Modified: 2008-12-18 11:55 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
log of remote.html reponse (2.02 KB, text/plain)
2004-08-30 11:11 UTC, KDF
Details
use parselines (568 bytes, patch)
2005-03-22 16:43 UTC, KDF
Details | Diff
use ir commands, fill timestamp in processURL (8.78 KB, patch)
2005-03-24 17:01 UTC, KDF
Details | Diff
replacement file, uses javascript to send ir with timecode (3.12 KB, text/plain)
2006-05-15 14:58 UTC, KDF
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Pbox 2004-08-29 11:32:46 UTC
When I go to /html/remote.html, which is the frameset I get some 
download notification. Apparently my browser does not display 
status.txt, rather wants me to decide what to do with it. I have Firefox 
0.9.3 on Windows 2K. I think the mime-type is set wrong?

Is it realted to the graphics update?
Comment 1 KDF 2004-08-30 11:10:58 UTC
This is not related to the upgrade.  Firefox shows the status.txt as being of
mime type application/octet-stream which it would expect to download instead of
display. it shoudl be set to text/html, and it appears to be so from the log,
which I'll attach shortly.
Comment 2 KDF 2004-08-30 11:11:46 UTC
Created attachment 114 [details]
log of remote.html reponse
Comment 3 Blackketter Dean 2005-03-11 14:44:55 UTC
And now it also gives "HASH(0x12345)".  broken.
Comment 4 KDF 2005-03-11 16:30:42 UTC
status.txt request causes a call to Display::Display::curLines, which now
returns a hash of parsed lines for some, text for others.  seems to highlight an
interesting issue. I'll have to look at $client->renderOverlay and see if we've
somehow handled different line results in different ways.
Comment 5 KDF 2005-03-22 16:43:30 UTC
Created attachment 362 [details]
use parselines

use parselines to grabe the text for line1 and line2.  Seems to be working with
graphics players at least.  should be global.
Comment 6 KDF 2005-03-23 00:25:22 UTC
test display fixed in svn r2684.  controller.html still broken for some modes.
Comment 7 KDF 2005-03-24 17:01:26 UTC
Created attachment 374 [details]
use ir commands, fill timestamp in processURL

just for the heck of it, controller.html can now use ir commands since they are
more generic.  Small patch to http.pm adds the needed timestamp if processing
status.txt with ir commands in the query.
Comment 8 KDF 2005-08-08 12:03:40 UTC
does anyone actually USE this?  If it was just for demo purposes, I would think
softsqueeze more than covers the need.  Given that even with patching the
irtime, this page cannot deal with repeat or hold button mappings.  Perhaps this
is best removed as a web page, leaving the status.txt processing behind as part
of the CLI.
Comment 9 KDF 2006-05-15 14:58:27 UTC
Created attachment 1242 [details]
replacement file, uses javascript to send ir with timecode

Just for fun, I went back to look at this.  The solution is fairly simple using javascript.  This replacement file sends the ircode along with the current time so that the server will be able to interpret the IR into a button for each mode.
Comment 10 KDF 2006-05-17 20:17:07 UTC
committed to trunk at change 7457.  calling this one done.
Comment 11 James Richardson 2008-12-15 13:05:53 UTC
This bug appears to have been fixed in the latest release!

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

Make sure to include the version number of the software you are seeing the error with.
Comment 12 Chris Owens 2008-12-18 11:55:07 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.