BUG: Pasting a component in root body layer throws TypeError
Question
GrapesJS version
- I confirm to use the latest version of GrapesJS
What browser are you using?
Chrome v89
Reproducible demo link
https://grapesjs.com/demo.html
Describe the bug
How to reproduce the bug?
- In the official demo, navigate to the layer manager and copy any layer with
cmd+c - Click the root "Body" layer and paste with
cmd+p - Navigate to the browser console and see the error
What is the expected behavior? Pasting in the body layer probably shouldn't be allowed, but it shouldn't throw an error.
What is the current behavior?
Pasting in the root Body layer throws this exception:
Uncaught TypeError: Cannot read property 'indexOf' of undefined
at PasteComponent.js:13
at Array.forEach (<anonymous>)
at run (PasteComponent.js:10)
at r.callRun (CommandAbstract.js:90)
at Object.runCommand (index.js:388)
at Object.method (index.js:158)
at w (keymaster.js:143)
at HTMLDocument.<anonymous> (keymaster.js:330)
The error is coming from the paste commend; comp.collection is undefined.
I don't know this codebase well enough to know if .collection on this component should return children. We were able to fix this temporarily by overriding the paste command and adding the following guard:
// ...
const coll = comp.collection;
if (!coll) {
return;
}
const at = coll.indexOf(comp) + 1;
// ...
If this is a fine solution, let me know and I'll happily open a PR.
Code of Conduct
- I agree to follow this project's Code of Conduct
Answers (3)
Thanks @zachsnoek yeah, as the wrapper (body) is a root component it doesn't have the related collection. Probably we might need to update the paste logic but your current fix is a good patch for now to avoid such an error so, the PR is highly welcome 👍
Thanks @zachsnoek yeah, as the wrapper (body) is a root component it doesn't have the related collection. Probably we might need to update the paste logic but your current fix is a good patch for now to avoid such an error so, the PR is highly welcome 👍
Sounds good; I'll put up a PR by the end of the week :+1:
Guys, I'm facing a similar problem To me, the error happens on FileUploader

I've tried to enter on grapes demo to see if this behavior happens there too, and it's happening!
I realize that If I clean my LocalStorage and reload the page, then the Editor loads the standard template, and I can drop any block inside it that will work well, but if I clean the editor and try to drop a Text block on Editor, the Editor saves an image tag in HTML key Take a look bellow

Any Idea that can help to solve this problem?
Related Questions and Answers
Continue research with similar issue discussions.
Issue #5378
BUG: Uncaught TypeError: Cannot read properties of undefined (reading 'Canvas') while dragging a component in Layer Manager Panel
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v116.0.5845.97 Reproducible demo li...
Issue #4257
BUG: Background modification doesn't work for non-images
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v100 Reproducible demo link https:/...
Issue #6669
BUG: Cross-level reordering of blocks in Layer Manager causes Sorter errors and breaks further move attempts
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://grap...
Issue #6281
BUG: Layer Manager does not allow reordering on same level. only nesting is allowed on top level(right under body) older versions (e.g: 0.21.8
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? latest chrome, firefox, edge Reproducible...
Paid Plugins That Match This Issue
Curated by issue keywords and label relevance to help you ship faster.
Loading paid plugin recommendations...
Browse Plugin Categories
Jump directly to plugin category pages on the marketplace.