DateTime#to_time converts to Time unless out of range. Date#to_datetime and Date#to_s(:rfc822). Closes #8512.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@6902 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

3 files changed, 22 insertions, 9 deletions

diff --git a/activesupport/lib/active_support/core_ext/date/conversions.rb b/activesupport/lib/active_support/core_ext/date/conversions.rb
index 01dd8461e4..25745b0f8b 100644
--- a/activesupport/lib/active_support/core_ext/date/conversions.rb
+++ b/activesupport/lib/active_support/core_ext/date/conversions.rb
@@ -7,7 +7,8 @@ module ActiveSupport #:nodoc:
           :short        => "%e %b",
           :long         => "%B %e, %Y",
           :db           => "%Y-%m-%d",
-          :long_ordinal => lambda { |date| date.strftime("%B #{date.day.ordinalize}, %Y") } # => "April 25th, 2007"
+          :long_ordinal => lambda { |date| date.strftime("%B #{date.day.ordinalize}, %Y") }, # => "April 25th, 2007"
+          :rfc822       => "%e %b %Y"
         }
 
         def self.included(klass) #:nodoc:
@@ -27,17 +28,20 @@ module ActiveSupport #:nodoc:
           end
         end
 
-        # To be able to keep Dates and Times interchangeable on conversions
+        # To be able to keep Times, Dates and DateTimes interchangeable on conversions
         def to_date
           self
         end
 
+        # Converts self to a Ruby Time object; time is set to beginning of day
+        # Timezone can either be :local or :utc  (default :local)
         def to_time(form = :local)
-          if respond_to?(:hour)
-            ::Time.send(form, year, month, day, hour, min, sec)
-          else
-            ::Time.send(form, year, month, day)
-          end
+          ::Time.send("#{form}_time", year, month, day)
+        end
+        
+        # Converts self to a Ruby DateTime object; time is set to beginning of day
+        def to_datetime
+          ::DateTime.civil(year, month, day, 0, 0, 0, 0, 0)
         end
 
         def xmlschema
diff --git a/activesupport/lib/active_support/core_ext/date_time/conversions.rb b/activesupport/lib/active_support/core_ext/date_time/conversions.rb
index 0b66593499..1b78a72eea 100644
--- a/activesupport/lib/active_support/core_ext/date_time/conversions.rb
+++ b/activesupport/lib/active_support/core_ext/date_time/conversions.rb
@@ -20,11 +20,19 @@ module ActiveSupport #:nodoc:
           end
         end
 
+        # Converts self to a Ruby Date object; time portion is discarded
         def to_date
           ::Date.new(year, month, day)
         end
 
-        # To be able to keep Times and DateTimes interchangeable on conversions
+        # Attempts to convert self to a Ruby Time object; returns self if out of range of Ruby Time class
+        # If self.offset is 0, then will attempt to cast as a utc time; otherwise will attempt to cast in local time zone
+        def to_time
+          method = if self.offset == 0 then 'utc' else 'local' end
+          ::Time.send(method, year, month, day, hour, min, sec) rescue self
+        end        
+
+        # To be able to keep Times, Dates and DateTimes interchangeable on conversions
         def to_datetime
           self
         end
diff --git a/activesupport/lib/active_support/core_ext/time/conversions.rb b/activesupport/lib/active_support/core_ext/time/conversions.rb
index 3945fe8f8c..7829c2e581 100644
--- a/activesupport/lib/active_support/core_ext/time/conversions.rb
+++ b/activesupport/lib/active_support/core_ext/time/conversions.rb
@@ -29,11 +29,12 @@ module ActiveSupport #:nodoc:
           end
         end
 
+        # Converts self to a Ruby Date object; time portion is discarded
         def to_date
           ::Date.new(year, month, day)
         end
 
-        # To be able to keep Dates and Times interchangeable on conversions
+        # To be able to keep Times, Dates and DateTimes interchangeable on conversions
         def to_time
           self
         end