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

Ensure that the found context is active before pushing it.

Browse Source
This commit is contained in:
Logan Smyth 2016-03-01 08:14:14 -08:00
parent b1477a3e8c
commit f2fea017c6
4 changed files with 48 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/babel-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/T7165/actual.js vendored Normal file
View File

@ -0,0 +1,6 @@
import foo from 'foo';
import { something } from 'bar';
const anything = {};
export * from 'bar';

31
packages/babel-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/T7165/expected.js vendored Normal file
View File

@ -0,0 +1,31 @@
'use strict';
Object.defineProperty(exports, "__esModule", {
value: true
});
var _bar = require('bar');
var _loop = function (_key2) {
if (_key2 === "default") return 'continue';
Object.defineProperty(exports, _key2, {
enumerable: true,
get: function () {
return _bar[_key2];
}
});
};
for (var _key2 in _bar) {
var _ret = _loop(_key2);
if (_ret === 'continue') continue;
}
var _foo = require('foo');
var _foo2 = _interopRequireDefault(_foo);
function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
var anything = {};

6
packages/babel-plugin-transform-es2015-modules-commonjs/test/fixtures/regression/T7165/options.json vendored Normal file
View File

@ -0,0 +1,6 @@
{
"plugins": [
"transform-es2015-block-scoping",
"transform-es2015-modules-commonjs"
]
}

6
packages/babel-traverse/src/path/modification.js
View File

@ -47,7 +47,11 @@ export function _containerInsert(from, nodes) {
if (this.context) {
let path = this.context.create(this.parent, this.container, to, this.listKey);
path.pushContext(this.context);
// While this path may have a context, there is currently no guarantee that the context
// will be the active context, because `popContext` may leave a final context in place.
// We should remove this `if` and always push once T7171 has been resolved.
if (this.context.queue) path.pushContext(this.context);
paths.push(path);
} else {
paths.push(NodePath.get({