diff options
| author | Sean Griffin <sean@thoughtbot.com> | 2014-05-19 10:59:57 -0700 |
|---|---|---|
| committer | Sean Griffin <sean@thoughtbot.com> | 2014-05-19 11:32:13 -0700 |
| commit | d0f8c46e1962b28d77209f367f12c2d2c77f4b12 (patch) | |
| tree | a20b809cd5b17c706709a51594c0ab1480503c6c /activerecord/test/cases/connection_adapters/type | |
| parent | 09cc922ed31bf699b26fafe4822fc7048b821825 (diff) | |
| download | rails-d0f8c46e1962b28d77209f367f12c2d2c77f4b12.tar.gz rails-d0f8c46e1962b28d77209f367f12c2d2c77f4b12.tar.bz2 rails-d0f8c46e1962b28d77209f367f12c2d2c77f4b12.zip | |
Remove :timestamp column type
The `:timestamp` type for columns is unused. All database adapters treat
them as the same database type. All code in `ActiveRecord` which changes
its behavior based on the column's type acts the same in both cases.
However, when the type is passed to code that checks for the `:datetime`
type, but not `:timestamp` (such as XML serialization), the result is
unexpected behavior.
Existing schema definitions will continue to work, and the `timestamp`
type is transparently aliased to `datetime`.
Diffstat (limited to 'activerecord/test/cases/connection_adapters/type')
| -rw-r--r-- | activerecord/test/cases/connection_adapters/type/type_map_test.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/test/cases/connection_adapters/type/type_map_test.rb b/activerecord/test/cases/connection_adapters/type/type_map_test.rb index 565f44eef8..300c2ed225 100644 --- a/activerecord/test/cases/connection_adapters/type/type_map_test.rb +++ b/activerecord/test/cases/connection_adapters/type/type_map_test.rb @@ -21,7 +21,7 @@ module ActiveRecord def test_overriding_registered_types time = Time.new - timestamp = Timestamp.new + timestamp = DateTime.new mapping = TypeMap.new mapping.register_type(/time/i, time) @@ -51,7 +51,7 @@ module ActiveRecord def test_changing_type_changes_aliases time = Time.new - timestamp = Timestamp.new + timestamp = DateTime.new mapping = TypeMap.new mapping.register_type(/timestamp/i, time) |