- πΊπΈUnited States joshf
The patch in #5 solved the problem for me; thanks!
It does need the test to be completely rewritten, as the current test only checks to see whether an item can be rendered more than 20 times with the same context and fails if it can (which is just completely wrong). The correct test won't be trivial to write because I think you'll need to provide some mock classes β to set up the correct behavior. The patch also needs code quality fixes.
#11 is a very limited workaround, not a fix, and should not be considered for merging.
#15 is simply incomplete; the way it's written no behavior is changed. It might work if there was a call made to
::setRenderPath()
, but there isn't. I'm hiding it because it's just not a solution and will only confuse things.@wylbur this issue is specifically for the entity_embed module; the Drupal core patch you referenced will not fix this issue. Further, I don't think patching this module will fix any problems you're having with paragraphs.