Process the definition via layout plugin manager, so the new plugin get fully populated.

nuvoleweb / ui_patterns

[NOTE] Development has moved to https://drupal.org/project/ui_patterns

https://drupal.org/project/ui_patterns

GNU General Public License v2.0

85 stars 56 forks source link

Process the definition via layout plugin manager, so the new plugin get fully populated. #289

Open dlopez-akalam opened 4 years ago

dlopez-akalam commented 4 years ago

As long as the plugin manager process the definition individually there would be no performance impact. Without the patch \Drupal\Core\Layout\LayoutPluginManager::getThemeImplementations() process wrongly the layout patterns

christianwiedemann commented 4 years ago

I applied the patch and my pattern crashes in layout_discovery.module:44. There is no content variable. Did I missed something.

geek-andi commented 2 years ago

I applied the patch and my pattern crashes in layout_discovery.module:44. There is no content variable. Did I missed something.

The same happens for me.