Unified Diff: safari/ext/common.js
Issue 5171007186796544:
Issue 1517 - Always call parseInt() with a radix as second argument (Closed)
Patch Set:
Created Oct. 31, 2014, 2:52 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« include.preload.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: safari/ext/common.js |
| =================================================================== |
| --- a/safari/ext/common.js |
| +++ b/safari/ext/common.js |
| @@ -115,7 +115,7 @@ |
| var content = rawMessage.placeholders[placeholder].content; |
| if (/^\$\d+$/.test(content)) |
| - placeholders[parseInt(content.substr(1)) - 1] = placeholder; |
| + placeholders[parseInt(content.substr(1), 10) - 1] = placeholder; |
| else |
| text = replacePlaceholder(text, placeholder, content); |
| } |
« no previous file with comments
|
« include.preload.js ('k')
|
no next file »
|
no next file with comments »
