Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ParisLiakos’s picture

Status: Active » Needs review
FileSize
2.8 KB
ParisLiakos’s picture

Crell’s picture

Issue tags: +WSCCI

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

Crell’s picture

Status: Needs review » Reviewed & tested by the community

And it looks like a nice and straightforward conversion.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed bb1817e and pushed to 8.x. Thanks!

tstoeckler’s picture

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?

alexpott’s picture

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

ParisLiakos’s picture

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.