Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.When using the inline diff, the ajax call works, but the output isn't rendered as the ajax wrapper doesn't exist.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | diff-1310422-fix-inline.patch | 1.21 KB | skwashd |
Comments
Comment #1
skwashd commentedThe attached patch fixes this problem.
Comment #2
deciphered@skwashd,
Can you please explain this patch, or at least provide some reproduction details, as the change itself seems redundant as it's simply replacing one ID with another, unless there are other places where the new ID is being used, or if the first ID is being nuked somehow.
At a glance, the patch makes no sense to me, but I'm happy to be shown the error in my ways.
Cheers,
Deciphered.
Comment #3
skwashd commentedThe current element id for the ajax wrapper never exists in the page and so the request is made, the diff content is returned it is never injected back into the page as the target isn't there. This patch uses the standard Drupal node element id as the wrapper so it can be replaced by the ajax response.
Comment #4
alexh commentedThe patch in #1 solves the problem. Thanks skwashd!
Comment #5
realityloop commentedcommited, thanks skwashd