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.

Files: 
CommentFileSizeAuthor
#5 shortcuts-followup-1975150-5.patch521 bytesDavid_Rothstein
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch shortcuts-followup-1975150-5.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 1975150-shortcut_remove_variable_get-cmi-drupal-1.patch555 bytesACF
PASSED: [[SimpleTest]]: [MySQL] 54,497 pass(es).
[ View ]

Comments

ACF’s picture

Status:Active» Needs review
StatusFileSize
new555 bytes
PASSED: [[SimpleTest]]: [MySQL] 54,497 pass(es).
[ View ]

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
StatusFileSize
new521 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch shortcuts-followup-1975150-5.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

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.