images/babel-plugin-jsx
2
0
Fork 0
mirror of https://github.com/vuejs/babel-plugin-jsx.git synced 2025-01-11 00:39:10 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: <A {...{ a: 1 }} /> throw error when override props

Browse Source
This commit is contained in:
Amour1688 2020-10-17 10:29:08 +08:00
parent e9aa278b20
commit f50b9e37fd
3 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/babel-plugin-jsx/src/buildProps.ts
View File

@ -56,8 +56,10 @@ const transformJSXSpreadAttribute = (
walksScope(nodePath, (argument.node as t.Identifier).name, SlotFlags.DYNAMIC); walksScope(nodePath, (argument.node as t.Identifier).name, SlotFlags.DYNAMIC);
} }
args.push(mergeProps ? argument.node : t.spreadElement(argument.node)); args.push(mergeProps ? argument.node : t.spreadElement(argument.node));
} else { } else if (mergeProps) {
args.push(t.objectExpression(properties)); args.push(t.objectExpression(properties));
} else {
args.push(...(properties as t.ObjectProperty[]));
} }
}; };

4
packages/babel-plugin-jsx/test/__snapshots__/snapshot.test.ts.snap
View File

@ -84,6 +84,10 @@ exports[`override props multiple: multiple 1`] = `
createVNode(resolveComponent(\\"A\\"), { createVNode(resolveComponent(\\"A\\"), {
\\"loading\\": true, \\"loading\\": true,
...a, ...a,
b: 1,
c: {
d: 2
},
\\"class\\": \\"x\\", \\"class\\": \\"x\\",
\\"style\\": x \\"style\\": x
}, null);" }, null);"

4
packages/babel-plugin-jsx/test/snapshot.test.ts
View File

@ -149,10 +149,10 @@ tests.forEach((
const overridePropsTests = [{ const overridePropsTests = [{
name: 'single', name: 'single',
from: '<div {...a}></div>', from: '<div {...a} />',
}, { }, {
name: 'multiple', name: 'multiple',
from: '<A loading {...a} class="x" style={x}></A>', from: '<A loading {...a} {...{ b: 1, c: { d: 2 } }} class="x" style={x} />',
}]; }];
overridePropsTests.forEach(( overridePropsTests.forEach((