"at X in the morning" defaults to 8am, rather than Xam about date

[matthewmueller]

oh interesting, yah this is definitely a bug. the morning default of 8am should have been overwritten in the last example

[bulkan]

I've been trying to look into fixing this bug, but getting stuck as the rAtHour wont run for date("tomorrow at 9 in the morning") wont find any matches.

@matthewmueller any tips to where to look ?

[matthewmueller]

Does it execute: https://github.com/MatthewMueller/date/blob/master/lib/parser.js#L307 ?

[bulkan]

the function athour does execute but the rAtHour regex doesn't find any matches

the following prints null

var dateStr = "at 9 in morning"
var rAtHour = /^at\s?(\d{1,2})$/;
console.log(rAtHour.exec(dateStr));

[matthewmueller]

Ah okay, well that wouldn't execute because it has the $ on the end.

The athour function should receive the string at 9 though, which should match.

[bulkan]

Oh my example had the wrong dateStr

Now that you mention that, this.str progressively becomes the following;

test string is "tomorrow at 9 in the morning

at 9 in the morning
9 in the morning
in the morning
the morning

this explains why the rAtHour regex doesn't run. Looking at the execution of the functions in order

• tomorrow
• string
• number
• string
• string
• morning

[bulkan]

@matthewmueller i've fixed this issue by doing this bulkan@749941d.

Probably, a better way would be to peek back once from the number and see if the token is at

[bulkan]

Hey @matthewmueller should I submit a pull request for the changes in bulkan@749941d ❓

[matthewmueller]

@bulkan yep, then i can review