- 🇬🇧United Kingdom scott_euser
Okay tests still failing, and I think its because maybe we still have a flaw here.
The commenting around the BC change in LibraryDiscoveryParser::setOverrideValue() is a bit unclear to me. It seems like it wants to check for the workaround like
libraries-override: contextual/drupal.contextual-links: css: component: /core/themes/stable/css/contextual/contextual.module.css: false
So to me that means it should be checking if the override like
/core/themes/stable/css/contextual/contextual.module.css
is from the base theme of the current theme (ie, direct parent theme, not grandparent), and if so, then throw the deprecation, but continue to work.Is that the correct intention? If so I think its not working like that quite yet. Happy to dig a bit but wanted to check if that's the right direction first.
Thanks!
- @scott_euser opened merge request.
- 🇬🇧United Kingdom scott_euser
- Started converting to MR
- Updated some tests but more still fail, e.g.
phpunit -c core/phpunit.xml core/tests/Drupal/KernelTests/Core/Asset/LibraryDiscoveryIntegrationTest.php --filter=testBaseThemeLibrariesOverrideInSubTheme
- Updated issue summary to target D12 for removal of BC layer
- First commit to issue fork.