#38 Alternate keys does not allow to use integer value (it is forcing to use a string)

- Improving the validation regex to accept string and integer as alternate key parameter
- New test case for the integer validation

Author: mfradcourt

dist/dynamics-web-api-callbacks.js

@@ -295,7 +295,7 @@ var ErrorHelper = {
             if (alternateKeys.length) {
                 for (var i = 0; i < alternateKeys.length; i++){
                     alternateKeys[i] = alternateKeys[i].trim().replace('"', "'");
-                    /^[\w\d\_]+\='[^\'\r\n]+'$/i.exec(alternateKeys[i])[0];
+                    /^[\w\d\_]+\=('[^\'\r\n]+'|\d+)$/i.exec(alternateKeys[i])[0];
                 }
             }
 

dist/dynamics-web-api-callbacks.min.js

@@ -295,7 +295,7 @@ var ErrorHelper = {
             if (alternateKeys.length) {
                 for (var i = 0; i < alternateKeys.length; i++){
                     alternateKeys[i] = alternateKeys[i].trim().replace('"', "'");
-                    /^[\w\d\_]+\='[^\'\r\n]+'$/i.exec(alternateKeys[i])[0];
+                    /^[\w\d\_]+\=('[^\'\r\n]+'|\d+)$/i.exec(alternateKeys[i])[0];
                 }
             }
 

dist/dynamics-web-api.js

@@ -143,7 +143,7 @@ var ErrorHelper = {
             if (alternateKeys.length) {
                 for (var i = 0; i < alternateKeys.length; i++){
                     alternateKeys[i] = alternateKeys[i].trim().replace('"', "'");
-                    /^[\w\d\_]+\='[^\'\r\n]+'$/i.exec(alternateKeys[i])[0];
+                    /^[\w\d\_]+\=('[^\'\r\n]+'|\d+)$/i.exec(alternateKeys[i])[0];
                 }
             }
 

dist/dynamics-web-api.min.js

@@ -1397,6 +1397,12 @@ describe("ErrorHelper.keyParameterCheck", function () {
         expect(result).to.eq("altKey='value',anotherKey='value2'");
     });
 
+    it("checks correct alternate keys string and integer", function () {
+        var alternateKey = "altKey=123456,anotherKey='value2'";
+        var result = ErrorHelper.keyParameterCheck(alternateKey);
+        expect(result).to.eq("altKey=123456,anotherKey='value2'");
+    });
+
     it("throws an error when alternate key is incorrect", function () {
         expect(function () {
             var alternateKey = "altKey='value, anotherKey='value2'";