diff options
| author | Pratik Naik <pratiknaik@gmail.com> | 2009-01-07 17:51:11 +0000 |
|---|---|---|
| committer | Pratik Naik <pratiknaik@gmail.com> | 2009-01-07 17:51:11 +0000 |
| commit | 17da45b789e0a2581eae6e6b2b1ae8d2b98e0f5d (patch) | |
| tree | 7c98524e11bf17bcf29a838c0ad9db2d972bf3ee | |
| parent | 2f923133248eb3a11671f47695bb9c5f36ee6aef (diff) | |
| download | rails-17da45b789e0a2581eae6e6b2b1ae8d2b98e0f5d.tar.gz rails-17da45b789e0a2581eae6e6b2b1ae8d2b98e0f5d.tar.bz2 rails-17da45b789e0a2581eae6e6b2b1ae8d2b98e0f5d.zip | |
Fix JSON decoder date-converter regexp [#1662 state:resolved] [Jonathan del Strother]
| -rw-r--r-- | activesupport/lib/active_support/json/decoding.rb | 2 | ||||
| -rw-r--r-- | activesupport/test/json/decoding_test.rb | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/activesupport/lib/active_support/json/decoding.rb b/activesupport/lib/active_support/json/decoding.rb index fdb219dbf7..9da4048272 100644 --- a/activesupport/lib/active_support/json/decoding.rb +++ b/activesupport/lib/active_support/json/decoding.rb @@ -16,7 +16,7 @@ module ActiveSupport protected # matches YAML-formatted dates - DATE_REGEX = /^\d{4}-\d{2}-\d{2}|\d{4}-\d{1,2}-\d{1,2}[ \t]+\d{1,2}:\d{2}:\d{2}(\.[0-9]*)?(([ \t]*)Z|[-+]\d{2}?(:\d{2})?)?$/ + DATE_REGEX = /^(?:\d{4}-\d{2}-\d{2}|\d{4}-\d{1,2}-\d{1,2}[ \t]+\d{1,2}:\d{2}:\d{2}(\.[0-9]*)?(([ \t]*)Z|[-+]\d{2}?(:\d{2})?)?)$/ # Ensure that ":" and "," are always followed by a space def convert_json_to_yaml(json) #:nodoc: diff --git a/activesupport/test/json/decoding_test.rb b/activesupport/test/json/decoding_test.rb index 19ae3a01a8..558b03b90d 100644 --- a/activesupport/test/json/decoding_test.rb +++ b/activesupport/test/json/decoding_test.rb @@ -15,7 +15,8 @@ class TestJSONDecoding < Test::Unit::TestCase # no time zone %({a: "2007-01-01 01:12:34"}) => {'a' => "2007-01-01 01:12:34"}, # needs to be *exact* - %({a: " 2007-01-01 01:12:34 Z "}) => {'a' => " 2007-01-01 01:12:34 Z "}, + %({a: " 2007-01-01 01:12:34 Z "}) => {'a' => " 2007-01-01 01:12:34 Z "}, + %({a: "2007-01-01 : it's your birthday"}) => {'a' => "2007-01-01 : it's your birthday"}, %([]) => [], %({}) => {}, %(1) => 1, |