Skip to content

Commit 0249d60

Browse files
fixed bug related to parseInt bug, leading 0 is interpreated in a fucked upp way
1 parent ab13d0c commit 0249d60

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

lib/query.js

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -170,15 +170,16 @@ var dateParser = function(isoDate) {
170170
var match = dateMatcher.exec(date);
171171
var splitDate = date.split('-');
172172
var year = match[1];
173-
var month = parseInt(match[2])-1;
173+
var month = parseInt(match[2],10)-1;
174174
var day = match[3];
175175

176176
var splitTime = time.split(':');
177-
var hour = parseInt(splitTime[0]);
178-
var min = splitTime[1];
177+
var hour = parseInt(splitTime[0],10);
178+
var min = parseInt(splitTime[1],10);
179179
var end = splitTime[2];
180180
var seconds = /(\d{2})/.exec(end);
181181
seconds = (seconds ? seconds[1] : 0);
182+
seconds = parseInt(seconds,10);
182183
var mili = /\.(\d{1,})/.exec(end);
183184
mili = mili ? mili[1].slice(0,3) : 0;
184185
var tZone = /([Z|+\-])(\d{2})?(\d{2})?/.exec(end);
@@ -189,10 +190,10 @@ var dateParser = function(isoDate) {
189190
switch(type) {
190191
case 'Z': break;
191192
case '-':
192-
tzAdjust = -(((parseInt(tZone[2])*60)+(parseInt(tZone[3]||0))));
193+
tzAdjust = -(((parseInt(tZone[2],10)*60)+(parseInt(tZone[3]||0,10))));
193194
break;
194195
case '+':
195-
tzAdjust = (((parseInt(tZone[2])*60)+(parseInt(tZone[3]||0))));
196+
tzAdjust = (((parseInt(tZone[2],10)*60)+(parseInt(tZone[3]||0,10))));
196197
break;
197198
default:
198199
throw new Error("Unidentifed tZone part " + type);
@@ -205,6 +206,9 @@ var dateParser = function(isoDate) {
205206
return date;
206207
};
207208

209+
// To help we test dateParser
210+
Query.dateParser = dateParser;
211+
208212
var dataTypeParsers = {
209213
20: parseInt,
210214
21: parseInt,

0 commit comments

Comments
 (0)