Files: 
CommentFileSizeAuthor
#1 drupal-session_write_interval_settings-1969846-1.patch2.8 KBParisLiakos
PASSED: [[SimpleTest]]: [MySQL] 54,286 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new2.8 KB
PASSED: [[SimpleTest]]: [MySQL] 54,286 pass(es).
[ View ]

Issue tags:+WSCCI

Tagging, because I didn't even see this...

Status:Needs review» Reviewed & tested by the community

And it looks like a nice and straightforward conversion.

Status:Reviewed & tested by the community» Fixed

Committed bb1817e and pushed to 8.x. Thanks!

Any reason why this was not converted to config?
I mean we definitely should not introduce a UI for this, but I don't get why this is environment-specific. If for some reason you want the value to be lower I guess you would want to deploy that, no?

I had assumed that this was used before the config system is available

it seems to me that this value would change per server..eg in my dev server i want to have it to 0...on my live at 180 or more
i am fine though reverting this and turning to config, i just want to get rid of the variable_get call there
http://drupal.org/node/1775842#comment-7297152

Automatically closed -- issue fixed for 2 weeks with no activity.