- Issue created by @shyam-sawhney
- 🇫🇷France mably
Hi @shyam-sawhney, thanks for your patch.
Markup::create
doesn't seem to be needed,renderInIsolation
already returns aMarkupInterfance
.Could you open an merge request for this issue?
- Merge request !46Issue #3522680 by shyam-sawhney, mably: LogicException: Render context is empty → (Merged) created by shyam-sawhney
- 🇫🇷France mably
Thanks for the MR.
renderInIsolation
requires Drupal 10.3+, so we will have to do something to handle it properly.Either increase our module version requirement to 10.3+ or switch back to
renderPlain
on lower D10 versions. - 🇦🇹Austria shyam-sawhney
Updated the minimum requirement for Drupal core version
- 🇫🇷France mably
Hi @shyam-sawhney, I made a few tweaks to the MR.
Could you verify everything is still working fine on your side before I merge it?
- 🇦🇹Austria shyam-sawhney
Hello @mably,
I have tested it on D 10.3.14. It's not generating any LogicException more
-
mably →
committed a9476bb8 on 3.0.x authored by
shyam-sawhney →
Issue #3522680 by shyam-sawhney, mably: LogicException: Render context...
-
mably →
committed a9476bb8 on 3.0.x authored by
shyam-sawhney →