Bug 15995 - Default skin's panel width resets after 7 days
: Default skin's panel width resets after 7 days
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.4.2
: PC Windows XP
: P3 normal (vote)
: 7.5.1
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-04-04 00:47 UTC by Jim McAtee
Modified: 2010-04-12 03:43 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 Jim McAtee 2010-04-04 00:47:48 UTC
In /server/HTML/Default/html/Main.js, set ExtJS's state cookie expirations to one year:

Ext.state.Manager.setProvider(
  new Ext.state.CookieProvider({
    expires: new Date(new Date().getTime()+(86400000*365))
  })
);
Comment 1 Michael Herger 2010-04-04 06:21:28 UTC
I first of all must to update ExtJS to something recent, as we're seeing issues with IE8 support too. This problem might be resolved as a side-effect of the update.
Comment 2 Jim McAtee 2010-04-04 11:08:27 UTC
(In reply to comment #1)
> This problem might be resolved as a side-effect of the update.

Only if they change the default 7 day cookie expiration for some reason.  In any case, why rely on their default?
Comment 3 Michael Herger 2010-04-04 11:35:18 UTC
You found the code which should change the default 7 days to a year yourself. But for some reason it doesn't work. First thing I'll try is the extjs update. Only if it then still fails I will investigate any further.
Comment 4 SVN Bot 2010-04-04 12:32:49 UTC
 == Auto-comment from SVN commit #30471 to the slim repo by mherger ==
 == https://svn.slimdevices.com/slim?view=revision&revision=30471 ==

Fixed Bug: 15995
Description: Set Cookie expiry for panel width to 1yr.

Jim - only just looked at the code and realised that your code wasn't the current code, but the suggested solution. Thanks for that!
Comment 5 Jim McAtee 2010-04-04 12:50:00 UTC
Other than white space, I don't see the difference.  The calculation for milliseconds in a day is correct.  Anyway... could you set a 7.5.1 target and backport it after 7.5.0 is released?

Sorry about not posting a patch, but since I'm working in Windows and my editors all want to add CRs to endlines, the diff files end up replacing the entire file.  I need to find a different editor or start working more with Linux.
Comment 6 SVN Bot 2010-04-07 00:23:21 UTC
 == Auto-comment from SVN commit #30484 to the slim repo by mherger ==
 == https://svn.slimdevices.com/slim?view=revision&revision=30484 ==

Bug: 15995
Description: backport change 30471 from 7.6. Fix cookie expiry delay.
Comment 7 Michael Herger 2010-04-12 03:43:11 UTC
*** Bug 8705 has been marked as a duplicate of this bug. ***