Closed (fixed)
Project:
Entity Embed
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Nov 2014 at 11:59 UTC
Updated:
28 Nov 2014 at 15:34 UTC
Jump to comment: Most recent
Comments
Comment #1
cs_shadow commentedPR: https://github.com/drupal-media/entity_embed/pull/105
Comment #2
slashrsm commentedTests failing.
Comment #3
yched commentedNo clue whether switching to drupal_render_root() is the correct fix or not, but if it is, might be worth a comment about why this requires a drupal_render_root(), as IIRC drupal_render_root() is supposed to be the exception.
Might also be good to have @Wim Leers chime on this issue ?
Comment #4
yched commentedAlso, the bug is porobably not that much that comment forms are not displayed anymore (that in itself is not a really important feature ?), but more generally that cache placeholders are not replaced (which could be much more annoying for other stuff besides comment forms) ?
Comment #5
wim leersI found this at https://github.com/drupal-media/entity_embed/pull/105, I commented there, confirming that you want to use
drupal_render_root().Comment #6
wim leersActually, it's not quite that simple anymore because we introduced safeguards to protect users from shooting themselves into the foot, which also prevents the suggested solution from working (and hence also the Travis testbot fails). We're working on adding an API to support this over at #2382667: #post_render_callback's that result from other #post_render_calback are not processed .
Comment #7
wim leers#2382667: #post_render_callback's that result from other #post_render_calback are not processed has now been committed; this change should no longer be necessary, it should all be working fine with the current
entity_embedcode against 8.x HEAD! :)