Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIXED] Wrong parsing result #11

Closed
GoogleCodeExporter opened this issue Mar 9, 2016 · 4 comments
Closed

[FIXED] Wrong parsing result #11

GoogleCodeExporter opened this issue Mar 9, 2016 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Type "december 9 at 9am"

What is the expected output? What do you see instead?
You will see: Saturday, December 01, 2007 9:00:00 AM
Should be: Sunday, December 09, 2007 9:00:00 AM

What version of the product are you using? On what operating system?
Alpha 1. Firefox

Please provide any additional information below.
Otherwise -- great script! Thanks

Original issue reported on code.google.com by niva...@gmail.com on 28 Nov 2007 at 2:19

@GoogleCodeExporter
Copy link
Author

Thanks for pointing out the bug.

The parser is getting confused with the 'at'. We're going to add support for 
ignore
words ("is at the on of") into the parser which should fix the problem.

The following works now...

Date.parse("december 9 9am");
Date.parse("dec 9 9am");

Original comment by geoff%co...@gtempaccount.com on 28 Nov 2007 at 3:07

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Work has started on a fix for this problem.

Original comment by geoff%co...@gtempaccount.com on 28 Nov 2007 at 11:41

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

Date.parse("december 9 at 9am") appears to be working now. I'm not entirely 
sure what
changed to make this work, but it does pass now. 

The online "Mad Skillz" widget has been updated with the latest code if you 
want to
confirm this online. 

The latest code has been committed to SVN.

Original comment by geoff%co...@gtempaccount.com on 18 Dec 2007 at 10:55

  • Changed title: [FIXED] Wrong parsing result

@GoogleCodeExporter
Copy link
Author

forgot to change status.

Original comment by geoff%co...@gtempaccount.com on 18 Dec 2007 at 10:57

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant