only percent encode #

Author: guybedford

lib/system.js

@@ -200,14 +200,12 @@
       if (wildcard)
         outPath = outPath.replace('*', wildcard);
 
-      // percent encode each path part
-      if (isBrowser) {
-        var outParts = outPath.split('/');
-        for (var i = 0, l = outParts.length; i < l; i++) {
-          outParts[i] = encodeURIComponent(outParts[i]);
-        }
-        outPath = outParts.join('/');
-      }
+      // percent encode just '#' in module names
+      // according to https://github.com/jorendorff/js-loaders/blob/master/browser-loader.js#L238
+      // we should encode everything, but it breaks for servers that don't expect it 
+      // like in (https://github.com/systemjs/systemjs/issues/168)
+      if (isBrowser)
+        outPath = outPath.replace(/#/g, '%40');
 
       return toAbsoluteURL(this.baseURL, outPath);
     },

test/test.js

@@ -144,10 +144,7 @@ function runTests() {
 
   System.baseURL = 'http://example.org/a/';
 
-  if (typeof window != 'undefined')
-    test('Locate', System.locate({ name: '@abc/def' }), 'http://example.org/a/%40abc/def.js');
-  else
-    test('Locate', System.locate({ name: '@abc/def' }), 'http://example.org/a/@abc/def.js');
+  test('Locate', System.locate({ name: '@abc/def' }), 'http://example.org/a/@abc/def.js');
   test('Locate', System.locate({ name: 'abc/def' }), 'http://example.org/a/abc/def.js');
 
   // paths