On Tuesday, July 4, 2017 at 8:02:05 AM UTC-7, Aryk Grosz wrote:
>
> Hey Jeremy, not sure if this was intended or not:
>
> opts[:decomposer] = proc do
> if (o = compositions[name]).nil?
> setter_meths.each{|sm| get_column_value(sm, nil)}
> else
> setters.each{|sm, cm| get_column_value(sm, o.send(cm))}
> end
>
>
>
> I think you would want "set_column_value" here, right? They both call
> "send" by default which is why it works now, but if somebody overwrote
> those methods, this would be a problem.
>
Yep, that looks like a bug, Can you send a PR to fix it?
Thanks,
Jeremy
--
You received this message because you are subscribed to the Google Groups
"sequel-talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sequel-talk.
For more options, visit https://groups.google.com/d/optout.
