Problem/Motivation

#2559825: Preprocess functions from base hooks not triggered for theme hooks not using the __ pattern is a bug that forces the use of double underscores in theme hooks in order for base hooks to work as expected.
#2296423: Implement layout plugin type in core works around this by forcibly prepending layout__ to each auto-generated theme hook.
This does not affect the template naming, but does affect the preprocess. This can lead to confusing effects, see #2296423-184: Implement layout plugin type in core for details.

Proposed resolution

Remove workaround from LayoutPluginManager

Remaining tasks

User interface changes

API changes

The naming of preprocess functions will change. Template names will not.

Data model changes

Comments

tim.plunkett created an issue. See original summary.

tim.plunkett’s picture

effulgentsia’s picture

Priority: Normal » Major

For me, this should be a blocker for layout_discovery reaching beta stability level, so raising priority to Major. Because per #2296423-184: Implement layout plugin type in core, I don't think non-experimental core modules should contain occurrences of a mismatch between theme hook name and template name, and per the issue summary, this is an API change in the sense of affecting preprocess function names, so best to do it before beta.

effulgentsia’s picture

Issue tags: +API change

Tagging as well, for easier finding later when the module gets close to beta.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.0-alpha1 will be released the week of January 30, 2017, which means new developments and disruptive changes should now be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

tim.plunkett’s picture