unable to set a node context on Layout Override of path `something/%`

[jenlampton]

Description of the bug

I'm unable to set a node context on my layout. Though the UI will allow me to select "Node" to map to the placeholder in my path, saving the Layout takes me away from the configure form, and does not save my context. Returning to that form results in the Context being reset to String pass-through.

I do have hook_menu entries for this site that use %node in position 1, so the problem isn't that this path doesn't exist, or that it's not properly defined. These layouts should be "Overrides" for me, and not Layout pages.

Steps To Reproduce

To reproduce the behavior:

1. Create a new layout at the path test/%
2. Select Node as the context
3. Save the layout - you will be taken to the manage blocks screen.
4. Return to the conigure screen
5. Confirm your context was not saved.

Actual behavior

Node context can be assigned.

Expected behavior

Node context can be assigned

Additional information

Add any other information that could help, such as:

• Backdrop CMS version:
• Web server and its version:
• PHP version:
• Database sever (MySQL or MariaDB?) and its version:
• Operating System and its version:
• Browser(s) and their versions:

[olafgrabienski]

@jenlampton I've tried the steps and was not able to reproduce the behavior on one of my test sites. When I returned to the Configure Layout page, the node context was still there.

[quicksketch]

In the report there's this:

I do have hook_menu entries for this site that use %node in position 1, so the problem isn't that this path doesn't exist, or that it's not properly defined. These layouts should be "Overrides" for me, and not Layout pages.

Maybe this problem occurs when using a Layout configuration that overrides an existing node path, other than node/%?

[argiepiano]

Maybe this problem occurs when using a Layout configuration that overrides an existing node path, other than node/%?

I think that's it.

1. Implement hook_node() in a custom module. Create an item with path dummy/%node and a dummy callback
2. Head to Structure > Layouts and create an override of dummy/%. The layout is shown as an override
3. Choose Node in Context for Position 2
4. Save
5. Visit the configuration page for the layout. The context type has been changed to "String passthrough". The Node context doesn't stick.