6.2.4

CHANGELOG.md

@@ -1,3 +1,6 @@
+SystemJS 6.2.4
+* Fix problem where System.delete did not remove named register modules. (https://github.com/systemjs/systemjs/pull/2125 @k-j-kim)
+
 SystemJS 6.2.3
 * Fix another race condition with named registers. (https://github.com/systemjs/systemjs/pull/2121)
 

dist/extras/named-register.js

@@ -44,15 +44,22 @@
       // Prefer import map (or other existing) resolution over the registerRegistry
       return resolve.call(this, id, parentURL);
     } catch (err) {
-      if (id in this.registerRegistry)
+      if (id in this.registerRegistry) {
         return id;
+      }
       throw err;
     }
   };
 
   const instantiate = systemJSPrototype.instantiate;
   systemJSPrototype.instantiate = function (url, firstParentUrl) {
-    return this.registerRegistry[url] || instantiate.call(this, url, firstParentUrl);
+    const result = this.registerRegistry[url];
+    if (result) {
+      this.registerRegistry[url] = null;
+      return result;
+    } else {
+      return instantiate.call(this, url, firstParentUrl);
+    }
   };
 
   const getRegister = systemJSPrototype.getRegister;

dist/s.js

@@ -1,5 +1,5 @@
 /*
-* SJS 6.2.3
+* SJS 6.2.4
 * Minimal SystemJS Build
 */
 (function () {

dist/system.js

@@ -1,5 +1,5 @@
 /*
-* SystemJS 6.2.3
+* SystemJS 6.2.4
 */
 (function () {
   const hasSelf = typeof self !== 'undefined';

package.json

@@ -1,6 +1,6 @@
 {
   "name": "systemjs",
-  "version": "6.2.3",
+  "version": "6.2.4",
   "description": "Dynamic ES module loader",
   "repository": {
     "type": "git",

test/browser/named-register.js

@@ -153,4 +153,18 @@ suite('Named System.register', function() {
       assert.equal(System.resolve('b'), 'b');
     });
   });
+
+  test('Ensure import still works after registerRegistry cleanup', function () {
+    return System.import('./fixtures/browser/named-bundle.js').then(function (m) {
+      assert.equal(m.a, 'b');
+      return System.import('b');
+    })
+    .then(function (m) {
+      assert.equal(m.b, 'b');
+      return System.import('b');
+    })
+    .then(function (m) {
+      assert.equal(m.b, 'b');
+    });
+  });
 });