- Issue created by @gauravvvv
- Merge request !6057check for the existence of node and drupalSettings.bigPipePlaceholderIds before access → (Open) created by Unnamed author
- Status changed to Needs review
11 months ago 4:58am 8 January 2024 - 🇮🇳India prashant.c Dharamshala
Thanks for raising and the fix @Gauravvvv. Yes, I am also able to reproduce this issue, however this JS error appears on every content type not specifically on "Basic page" only.
Put some comments in the MR.
Thank you.
- 🇮🇳India prashant.c Dharamshala
@cilefen Yes, at-least I got this error on clean 11.x install.
- Status changed to Needs work
11 months ago 2:41pm 8 January 2024 - 🇺🇸United States smustgrave
Issue summary is incomplete
Will need steps to reproduce + tests.
- 🇳🇱Netherlands casey
Static patch file of MR up until #4 for use with composer-patches
- 🇩🇪Germany jox
I'm getting this error also when editing a node e.g. at
/node/16/edit. The patch fixes it there as well. - 🇯🇴Jordan Rajab Natshah Jordan
Facing the same issue.
Patch #9 fixed the issue - last update
11 months ago 25,968 pass, 1,846 fail - 🇿🇦South Africa mauzilla
Getting the same error in Drupal 10.2 - Seems to affect all admin pages
- 🇺🇸United States jennypanighetti
Patch #9 seems to have fixed it for my admin pages as well - 10.2.1 and 10.2.2
- First commit to issue fork.
- last update
10 months ago 25,795 pass, 1,808 fail - Status changed to Needs review
10 months ago 7:17am 19 January 2024 - Status changed to Needs work
10 months ago 7:19am 19 January 2024 - 🇮🇳India Hardik_Patel_12 India
Returning to "needs work" as a test is required.
- First commit to issue fork.
- Status changed to Needs review
10 months ago 2:24pm 23 January 2024 - 🇮🇳India vakulrai
Re-rolled patch as its reproducible for 11.x branch and for the tests as the errors are more due to the JS mutation observer is missing a check and the error in the console cant be tracked in the Tests so can anyone suggest a way to have a test coverage here Else we can move this ticket forward.
Thanks!
- 🇪🇸Spain albeorte
I have tested the added changes and they work correctly! RTBC
I add the new changes in file.patch format
- Status changed to RTBC
10 months ago 3:51pm 23 January 2024 - Status changed to Needs work
10 months ago 9:06pm 1 February 2024 - 🇦🇺Australia acbramley
Marked 🐛 big_pipe JS checkMutation can throw type errors if parentNode is NULL Closed: duplicate as a duplicate. This still needs tests. Please keep work in the MR, patches are not required here.
- 🇧🇪Belgium herved
I'm seeing this on Drupal 10.2.3 as well, issue 🐛 big_pipe sometimes fails to load blocks Active got included in 10.2.1 so I guess it makes sense.
The error is almost on all pages for me. It seems one of my browser extensions (Scraper 1.7, on brave, which I now disabled as I'm not using it) is causing DOM mutations on the body element, which triggers this.big_pipe needs to be more resilient.
The MR fixes it. I wonder though if the NULL check should happen above, incheckMutationAndProcess?
I'll add a static diff patch of it (latest commit 6580c866), for composer, in case anyone else needs it. - Status changed to Closed: duplicate
9 months ago 9:08pm 20 February 2024 - 🇦🇺Australia acbramley
This has been fixed in 🐛 big_pipe.js' checkMutation() does not check if node exists before using it Fixed
