Changeset 8885

Timestamp:

02/16/08 23:39:40 (8 months ago)

Author:

gbuesing

Message:

Time.=== returns true for TimeWithZone instances

Files:

• trunk/activesupport/CHANGELOG (modified) (1 diff)
• trunk/activesupport/lib/active_support/core_ext/time/calculations.rb (modified) (1 diff)
• trunk/activesupport/test/core_ext/time_ext_test.rb (modified) (1 diff)

trunk/activesupport/CHANGELOG

@@ -1 +1 @@
-*SVN*
+
+* Time.=== returns true for TimeWithZone instances [Geoff Buesing]
-
-* TimeWithZone #+ and #- behave consistently with numeric arguments regardless of whether wrapped time is a Time or DateTime; consistenty answers false to #acts_like?(:date) [Geoff Buesing]

trunk/activesupport/lib/active_support/core_ext/time/calculations.rb

@@ -25 +25 @@
-
-        module ClassMethods
+          # Overriding case equality method so that it returns true for ActiveSupport::TimeWithZone instances
+          def ===(other)
+            other.is_a?(::Time)
+          end
+          
-          # Return the number of days in the given month. 
-          # If no year is specified, it will use the current year. 

trunk/activesupport/test/core_ext/time_ext_test.rb

@@ -469 +469 @@
-  end
-
+  def test_case_equality
+    assert Time === Time.utc(2000)
+    assert Time === ActiveSupport::TimeWithZone.new(Time.utc(2000), TimeZone['UTC'])
+    assert_equal false, Time === DateTime.civil(2000)
+  end
+
-  protected
-    def with_env_tz(new_tz = 'US/Eastern')