mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
b63d532f1e
So this bug is kinda funky. The code path is basically "if we weren't passed an
instance of the class we compose to, and we have a converter, call that".
Ignoring the hash case for a moment, everything after that was roughly intended
to be the "else" clause, meaning that we are expected to have an instance of
the class we compose to. Really, we should be blowing up in that case, as we
can give a much better error message than what they user will likely get (e.g.
`NameError: No method first for String` or something). Still, Ruby is duck
typed, so if the object you're assigning responds to the same methods as the
type you compose to, knock yourself out.
The hash case was added in 36e9be8 to remove a bunch of special cased code from
multiparameter assignment. I wrongly assumed that the only time we'd get a hash
there is in that case. Multiparameter assignment will construct a very specific
hash though, where the keys are integers, and we will have a set of keys
covering `1..part.size` exactly. I'm pretty sure this could actually be passed
around as an array, but that's a different story. Really I should convert this
to something like `class MultiParameterAssignment < Hash; end`, which I might
do soon. However for a change that I'm willing to backport to 4-2-stable, this
is what I want to go with for the time being.
Fixes #25978
83 lines
2.2 KiB
Ruby
83 lines
2.2 KiB
Ruby
class Customer < ActiveRecord::Base
|
|
cattr_accessor :gps_conversion_was_run
|
|
|
|
composed_of :address, :mapping => [ %w(address_street street), %w(address_city city), %w(address_country country) ], :allow_nil => true
|
|
composed_of :balance, :class_name => "Money", :mapping => %w(balance amount), :converter => Proc.new(&:to_money)
|
|
composed_of :gps_location, :allow_nil => true
|
|
composed_of :non_blank_gps_location, :class_name => "GpsLocation", :allow_nil => true, :mapping => %w(gps_location gps_location),
|
|
:converter => lambda { |gps| self.gps_conversion_was_run = true; gps.blank? ? nil : GpsLocation.new(gps)}
|
|
composed_of :fullname, :mapping => %w(name to_s), :constructor => Proc.new { |name| Fullname.parse(name) }, :converter => :parse
|
|
composed_of :fullname_no_converter, :mapping => %w(name to_s), class_name: "Fullname"
|
|
end
|
|
|
|
class Address
|
|
attr_reader :street, :city, :country
|
|
|
|
def initialize(street, city, country)
|
|
@street, @city, @country = street, city, country
|
|
end
|
|
|
|
def close_to?(other_address)
|
|
city == other_address.city && country == other_address.country
|
|
end
|
|
|
|
def ==(other)
|
|
other.is_a?(self.class) && other.street == street && other.city == city && other.country == country
|
|
end
|
|
end
|
|
|
|
class Money
|
|
attr_reader :amount, :currency
|
|
|
|
EXCHANGE_RATES = { "USD_TO_DKK" => 6, "DKK_TO_USD" => 0.6 }
|
|
|
|
def initialize(amount, currency = "USD")
|
|
@amount, @currency = amount, currency
|
|
end
|
|
|
|
def exchange_to(other_currency)
|
|
Money.new((amount * EXCHANGE_RATES["#{currency}_TO_#{other_currency}"]).floor, other_currency)
|
|
end
|
|
end
|
|
|
|
class GpsLocation
|
|
attr_reader :gps_location
|
|
|
|
def initialize(gps_location)
|
|
@gps_location = gps_location
|
|
end
|
|
|
|
def latitude
|
|
gps_location.split("x").first
|
|
end
|
|
|
|
def longitude
|
|
gps_location.split("x").last
|
|
end
|
|
|
|
def ==(other)
|
|
self.latitude == other.latitude && self.longitude == other.longitude
|
|
end
|
|
end
|
|
|
|
class Fullname
|
|
attr_reader :first, :last
|
|
|
|
def self.parse(str)
|
|
return nil unless str
|
|
|
|
if str.is_a?(Hash)
|
|
new(str[:first], str[:last])
|
|
else
|
|
new(*str.to_s.split)
|
|
end
|
|
end
|
|
|
|
def initialize(first, last = nil)
|
|
@first, @last = first, last
|
|
end
|
|
|
|
def to_s
|
|
"#{first} #{last.upcase}"
|
|
end
|
|
end
|