lowcode-engine icon indicating copy to clipboard operation
lowcode-engine copied to clipboard

node-children 中触发 'node.add' 事件的参数错误导致使用document.onMountNode报错

Open leopen-hu opened this issue 3 years ago • 4 comments

Describe the bug (required) / 详细描述 bug(必填)

node-children.ts 中触发 node.add 事件的参数错误导致使用 document.onMountNode() 报错


To Reproduce (required) / 如何复现 bug?(必填,非常重要)

编写插件的 init() 函数中调用接口,输出结果多一层node,代码如下

project.currentDocument.onMountNode((node) => {
  console.log('onMountNode', node, node?.node, node.node.parent);
});

Screenshots (optional) / bug 截图(可选)

调用时多了一层node

定义是Node


leopen-hu avatar Sep 07 '22 08:09 leopen-hu

你建议怎么改呀?

LeoYuan avatar Sep 07 '22 09:09 LeoYuan

onMountNode(({ node }) => { console.log('我是 node', node); });

目前来看,在不 breaking 的情况下,只能改改 onMountNode 的入参数据类型了~

LeoYuan avatar Sep 07 '22 09:09 LeoYuan

我的理解为了保持和其他接口一致性,应该是改emit那个部分不要{},但是需要看看其他涉及的地方是否也需要同步修改

leopen-hu avatar Sep 08 '22 06:09 leopen-hu

应该不行,不管内部如何重构,onMountNode 是对外的 API,这个方法签名不能动~

只能先用着,等 next-x 版本再纠正了~

LeoYuan avatar Sep 08 '22 08:09 LeoYuan