diff options
author | Sean Griffin <sean@thoughtbot.com> | 2015-02-17 14:07:45 -0700 |
---|---|---|
committer | Sean Griffin <sean@thoughtbot.com> | 2015-02-17 14:07:45 -0700 |
commit | ad127d8836165bba70290a9429eee5b16033e20c (patch) | |
tree | 8a2a07271e287683c7ee179d0fc91d3aa8805729 /activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb | |
parent | 9ca6948f72bef56445030a60e346376a821dbc72 (diff) | |
download | rails-ad127d8836165bba70290a9429eee5b16033e20c.tar.gz rails-ad127d8836165bba70290a9429eee5b16033e20c.tar.bz2 rails-ad127d8836165bba70290a9429eee5b16033e20c.zip |
Rm `Type#type_cast`
This helper no longer makes sense as a separate method. Instead I'll
just have `deserialize` call `cast` by default. This led to a random
infinite loop in the `JSON` pg type, when it called `super` from
`deserialize`. Not really a great way to fix that other than not calling
super, or continuing to have the separate method, which makes the public
API differ from what we say it is.
Diffstat (limited to 'activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb')
-rw-r--r-- | activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb b/activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb index 44b10c94bf..bf565bcf47 100644 --- a/activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb +++ b/activerecord/lib/active_record/connection_adapters/postgresql/oid/point.rb @@ -9,13 +9,13 @@ module ActiveRecord :point end - def type_cast(value) + def cast(value) case value when ::String if value[0] == '(' && value[-1] == ')' value = value[1...-1] end - type_cast(value.split(',')) + cast(value.split(',')) when ::Array value.map { |v| Float(v) } else |