Issue #4000Opened December 7, 2021by zachsnoek0 reactions

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?

  1. In the official demo, navigate to the layer manager and copy any layer with cmd+c
  2. Click the root "Body" layer and paste with cmd+p
  3. 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)

artfDecember 9, 20210 reactions

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 👍

zachsnoekDecember 9, 20210 reactions

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:

GuiMoraesDevDecember 16, 20210 reactions

Guys, I'm facing a similar problem To me, the error happens on FileUploader

Screenshot from 2021-12-16 17-10-32

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

image

Any Idea that can help to solve this problem?

Related Questions and Answers

Continue research with similar issue discussions.

Paid Plugins That Match This Issue

Curated by issue keywords and label relevance to help you ship faster.

View all plugins

Loading paid plugin recommendations...

Browse Plugin Categories

Jump directly to plugin category pages on the marketplace.