Add ability to parse "%m-%d-Y"

[seerahulsingh]

Hi,

Right now it only parse dates in the strict format of "%m/%d/%Y". Can we use "%m-%d-Y" format also.

With this "%m-%d-Y" format, when I tried to parse it, it raised an exception

ArgumentError: invalid date

[jeremyevans]

By design, this library emulates ruby 1.8.7 date parsing, which doesn't parse %m-%d-%Y dates, so I don't plan on adding support for this. Please fork the library if you would like that date format handled.

[seerahulsingh]

ok, thanks -:)

[djberg96]

In 1.1 and earlier our app was doing this:

require 'american_date'
class << Date
  remove_const(:AMERICAN_DATE_RE)
  AMERICAN_DATE_RE = %r_\A\s*(\d{1,2})[-/](\d{1,2})[-/](\d{4}|\d{2})_.freeze
end

However, this doesn't work in 1.2.x because of the switch to Date.instance_eval.

[jeremyevans]

In 1.1 and earlier our app was doing this:

require 'american_date'
class << Date
  remove_const(:AMERICAN_DATE_RE)
  AMERICAN_DATE_RE = %r_\A\s*(\d{1,2})[-/](\d{1,2})[-/](\d{4}|\d{2})_.freeze
end

However, this doesn't work in 1.2.x because of the switch to Date.instance_eval.

Looks like that is a bug in 1.2, because AMERICAN_DATE_RE gets defined at top-level. I'll fix it.

[djberg96]

Thanks! Any idea when the fix will be pushed out?

[jeremyevans]

Hopefully I can release a new version later this week.