Convert path_alias_whitelist to state system

The variable needs removing during Drupal 7 to 8 upgrade. See http://drupal.org/node/1787318 for how to do this.

Looks good - bumped update to system_update_8028()

The patch attached to #7 was bogus... not sure what happened there.

Anyhow... patch in #6 still looks good - just bumped update to system_update_8028()

8029* ;) Been a busy night!

Committed and pushed that (with the renumbering, and also a > 80 chars comment fix).

I think that there was a problem with this patch. There is no second parameter in calls to state()->get.

If I understand correctly, variable_get('my_data', 'bar') should become:

$data = state()->get('my_data') ?: 'bar';

As a result, I am setting this back to needs work for the correction(s).

And the upgrade is unnecessary :)

btw... no need to provide a default as NULL will be return by the state system if it's not set... which is the same as the default.

This shouldn't really be in the state system, it's a cache, it just happens to be a cache that's /extremely/ expensive to rebuild so it was stuffed into variables.

I'd rather do #1209226: Avoid slow query for path alias whitelists (which was getting a lot closer last time I looked).

This shouldn't really be in the state system, it's a cache, it just happens to be a cache that's /extremely/ expensive to rebuild so it was stuffed into variables.

I'd rather do #1209226: Avoid slow query for path alias whitelists (which was getting a lot closer last time I looked).

Postponing, hope the other issue will fix it. The other first.
#1209226: Avoid slow query for path alias whitelists

Not sure what the proper component for this is, but configuration system isn't. :)

Also, I think this was already done as part of the alias/path refactoring.