cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #3326 from eetulatja/master

Browse Source 
Fix spread to work with super method calls
This commit is contained in:
Logan Smyth 2016-02-10 22:49:02 -07:00
commit ff2bf2642b
5 changed files with 24 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/babel-plugin-transform-es2015-spread/src/index.js
View File

@ -101,6 +101,10 @@ export default function ({ types: t }) {
node.callee = t.memberExpression(node.callee, t.identifier("apply"));
}
if (t.isSuper(contextLiteral)) {
contextLiteral = t.thisExpression();
}
node.arguments.unshift(contextLiteral);
},

5
packages/babel-plugin-transform-es2015-spread/test/fixtures/spread/contexted-method-call-super-multiple-args/actual.js vendored Normal file
View File

@ -0,0 +1,5 @@
class Foo {
bar() {
super.bar(arg1, arg2, ...args);
}
}

5
packages/babel-plugin-transform-es2015-spread/test/fixtures/spread/contexted-method-call-super-multiple-args/expected.js vendored Normal file
View File

@ -0,0 +1,5 @@
class Foo {
bar() {
super.bar.apply(this, [arg1, arg2].concat(babelHelpers.toConsumableArray(args)));
}
}

5
packages/babel-plugin-transform-es2015-spread/test/fixtures/spread/contexted-method-call-super-single-arg/actual.js vendored Normal file
View File

@ -0,0 +1,5 @@
class Foo {
bar() {
super.bar(...args);
}
}

5
packages/babel-plugin-transform-es2015-spread/test/fixtures/spread/contexted-method-call-super-single-arg/expected.js vendored Normal file
View File

@ -0,0 +1,5 @@
class Foo {
bar() {
super.bar.apply(this, babelHelpers.toConsumableArray(args));
}
}