gryf/pentadactyl-pm
1
0
Fork 0
mirror of https://github.com/gryf/pentadactyl-pm.git synced 2026-03-02 05:25:46 +01:00
Code Issues Projects Releases Wiki Activity

Fix numeric option value validation for "".

Browse Source 
Number("") => 0, Number("077") => 77, parseInt("008") => 0 etc.

--HG--
extra : transplant_source : %5B%A2%EC%06%BB%85A%C7%D3%B5%7E%B5%91f%9B1%8F%D3%3D%98
This commit is contained in:
Doug Kearns
2009-09-25 19:36:10 +10:00
parent d24adb5f35
commit 252e1965c0
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
common/content/options.js
View File

@@ -340,10 +340,11 @@ Option.prototype = {
break; break;
case "number": case "number":
let value = Number(values); // deduce radix // TODO: support floats? Validators need updating.
if (!/^[+-]?(?:0x[0-9a-f]+|0[0-7]+|0|[1-9]\d*)$/i.test(values))
return "E521: Number required after := " + this.name + "=" + values;
if (isNaN(value) || value != parseInt(value)) let value = parseInt(values/* deduce radix */);
return "E521: Number required";
switch (operator) switch (operator)
{ {
@@ -411,7 +412,7 @@ Option.prototype = {
break; break;
default: default:
return "E685: Internal error: option type `" + option.type + "' not supported"; return "E685: Internal error: option type `" + this.type + "' not supported";
} }
if (newValue == null) if (newValue == null)