Fix Resource template replacement.

The previous implementation did not work with template keys that are numbers,
because the regex syntax was not escaped properly.  Instead of building
a regex for each replacement, we can use a replace callback method.

Author: shunter

CHANGES.md

@@ -18,6 +18,7 @@ Change Log
 * Fixed the value for `BlendFunction.ONE_MINUS_CONSTANT_COLOR`. [#7624](https://github.com/AnalyticalGraphicsInc/cesium/pull/7624)
 * Fixed `HeadingPitchRoll.pitch` being `NaN` when using `.fromQuaternion` do to a rounding error
 for pitches close to +/- 90°. [#7654](https://github.com/AnalyticalGraphicsInc/cesium/pull/7654)
+* Fixed an error in `Resource` when used with template replacements using numeric keys. [#7668](https://github.com/AnalyticalGraphicsInc/cesium/pull/7668)
 
 ### 1.55 - 2019-03-01
 

Source/Core/Resource.js

@@ -569,15 +569,13 @@ define([
         // objectToQuery escapes the placeholders.  Undo that.
         var url = uri.toString().replace(/%7B/g, '{').replace(/%7D/g, '}');
 
-        var template = this._templateValues;
-        var keys = Object.keys(template);
-        if (keys.length > 0) {
-            for (var i = 0; i < keys.length; i++) {
-                var key = keys[i];
-                var value = template[key];
-                url = url.replace(new RegExp('{' + key + '}', 'g'), encodeURIComponent(value));
-            }
-        }
+        var templateValues = this._templateValues;
+        url = url.replace(/{(.*?)}/g, function(match, key) {
+            // use the replacement value from templateValues if there is one,
+            // otherwise leave it unchanged
+            return defaultValue(templateValues[key], match);
+        });
+
         if (proxy && defined(this.proxy)) {
             url = this.proxy.getURL(url);
         }

Specs/Core/Iau2006XysDataSpec.js

@@ -1,10 +1,12 @@
 defineSuite([
         'Core/Iau2006XysData',
+        'Core/buildModuleUrl',
         'Core/defined',
         'Core/Iau2006XysSample',
         'Specs/pollToPromise'
     ], function(
         Iau2006XysData,
+        buildModuleUrl,
         defined,
         Iau2006XysSample,
         pollToPromise) {
@@ -51,4 +53,16 @@ defineSuite([
     it('returns undefined after the last XYS table sample', function() {
         expect(xys.computeXysRadians(2442396 + 27427, 0.0)).toBeUndefined();
     });
+
+    it('allows configuring xysFileUrlTemplate', function() {
+        xys = new Iau2006XysData({
+            // this should be the same location as the default, but specifying the value
+            // takes the code through a different code path.
+            xysFileUrlTemplate: buildModuleUrl('Assets/IAU2006_XYS/IAU2006_XYS_{0}.json')
+        });
+
+        return pollToPromise(function() {
+            return defined(xys.computeXysRadians(2442398, 1234.56));
+        });
+    });
 });

Specs/Core/ResourceSpec.js

@@ -193,6 +193,13 @@ defineSuite([
 
     it('templateValues are respected', function() {
         var resource = new Resource({
+            url: 'http://test.com/tileset/{foo}/{bar}'
+        });
+
+        // if template values are not provided, they are left unchanged
+        expect(resource.url).toEqual('http://test.com/tileset/{foo}/{bar}');
+
+        resource = new Resource({
             url: 'http://test.com/tileset/{foo}/{bar}',
             templateValues: {
                 foo: 'test1',
@@ -201,6 +208,16 @@ defineSuite([
         });
 
         expect(resource.url).toEqual('http://test.com/tileset/test1/test2');
+
+        resource = new Resource({
+            url: 'http://test.com/tileset/{0}/{1}',
+            templateValues: {
+                '0': 'test1',
+                '1': 'test2'
+            }
+        });
+
+        expect(resource.url).toEqual('http://test.com/tileset/test1/test2');
     });
 
     it('getDerivedResource sets correct properties', function() {