Subtask of #1775842: [meta] Convert all variables to state and/or config systems. Removed the variable_get as as far as I can see the hook just needs to return a string so just showing that should be clear enough I think with the comment above.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ACF’s picture

Status: Active » Needs review
FileSize
555 bytes

Patch.

catch’s picture

Status: Needs review » Reviewed & tested by the community
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

David_Rothstein’s picture

Status: Fixed » Needs work

It's impossible to create a shortcut set with a machine-readable name that looks like that (at least through the UI), so I don't think that code really works.

It's also a little odd to return a hardcoded string there (would only really ever make sense in custom code, rather than a contrib module) but I guess we can let that slide.

David_Rothstein’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
521 bytes

This switches to using a more likely (and legal) machine-readable name for the example shortcut set.

I think I can RTBC my own patch here...

webchick’s picture

Component: configuration system » documentation
Status: Reviewed & tested by the community » Fixed

D'oh! Thanks, David.

Committed and pushed to 8.x. Again. :)

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