Merge pull request mdn#23 from lmorchard/no-trimmed-decimals-782930

fix bug 782930: Don't use parseInt() to parse numeric arguments

lib/kumascript/parser.pegjs

@@ -103,7 +103,7 @@ ArgumentList
 Argument
   = c:( Number / DoubleQuotedArgumentChars / SingleQuotedArgumentChars )
 
-Number = c:[\-.0-9]+ { return parseInt(c.join('')); }
+Number = c:[\-.0-9]+ { return c.join(''); }
 
 DoubleQuotedArgumentChars
   = '"' c:DoubleQuotedArgumentChar* '"' { return c.join(''); }

tests/fixtures/documents/document1-expected.txt

@@ -9,4 +9,4 @@ Here is { some text } with braces, but not a macro.
 
 Look, it's JSON without escaping: ["a","b","c"]
 
-Here is a multi-line macro: ["With many","parameters","including \"quoted\"","text of both 'kinds'","even \"escaped\"","like 'these'","and numbers",-10,0,0,10,100] that even works inline.
+Here is a multi-line macro: ["With many","parameters","including \"quoted\"","text of both 'kinds'","even \"escaped\"","like 'these'","and numbers","-10","0","0.1","10","100"] that even works inline.

tests/fixtures/macros-decimal-argument.txt

@@ -0,0 +1,3 @@
+* {{fx_minversion_header(3.6)}}
+---
+* ["fx_minversion_header",["3.6"]]

tests/fixtures/macros1.txt

@@ -36,6 +36,6 @@ The following items are all macros, though:
 * ["ParenTastic",["()()()"]]
 * ["Bracepocalypse",["{{}}{{}}{{}}"]]
 
-Here is a ["MultiLineMacro",["With many","parameters","including \"quoted\"","text of both 'kinds'","even \"escaped\"","like 'these'","and numbers",-10,0,0,10,100]] that even works inline.
+Here is a ["MultiLineMacro",["With many","parameters","including \"quoted\"","text of both 'kinds'","even \"escaped\"","like 'these'","and numbers","-10","0","0.1","10","100"]] that even works inline.
 
 Here is a ["MacroWithJson",[{"es":"es/JavaScript/Acerca_de_JavaScript","en":"en/JavaScript/About_JavaScript"}]] that even works inline.

tests/test-macros.js

@@ -89,6 +89,18 @@ module.exports = nodeunit.testCase({
             });
     },
 
+    "A numeric macro argument with a decimal point should not be trimmed to an integer": function (test) {
+        var mp = new ks_macros.MacroProcessor({
+            macro_timeout: 500,
+            loader_class: ks_test_utils.JSONifyLoader
+        });
+        processFixture(test, mp, 'macros-decimal-argument.txt',
+            function (errors, result) {
+                test.ok(!errors, "There should be no errors");
+                test.done();
+            });
+    },
+
     "Escaped single and double quotes should work in any quoting context": function (test) {
         var mp = new ks_macros.MacroProcessor({
             macro_timeout: 500,