Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Cameron Tod’s picture

Title: Convert variable drupal_stale_file_threshold to state system » Convert variable drupal_stale_file_threshold to CMI system
Cameron Tod’s picture

Status: Active » Needs review
Issue tags: +State system
FileSize
1.47 KB

Candidate patch attached. Couple of things:

- New state name is `system.stale_file_threshold`, but I'm not sure this is the best name. It might be better to put it in the file namespace, maybe?
- I've added a general purpose update hook to convert variables to the state system. I'm not sure if this is within the scope of the patch, but at the least it will convert any saved `drupal_stale_file_threshold` values to state.

Cameron Tod’s picture

Woops, had a bad name on the update hook.

Berdir’s picture

Status: Needs review » Needs work

Should use the new helper function for the upgrade now.

Cameron Tod’s picture

Status: Needs work » Needs review
FileSize
1.36 KB

New patch with variable as config instead of state.

Cameron Tod’s picture

Duh, wrong config key.

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

RTBC if it passes the tests.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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