They need to return a unique value, not necessarily an array. This shouldn't affect them.
Vipul A.M. +91-8149-204995 On Tue, Jan 20, 2015 at 2:14 AM, Matt Jones <[email protected]> wrote: > > On Jan 17, 2015, at 6:10 AM, Vipul A M <[email protected]> wrote: > > `to_key` was first introduced in > https://github.com/rails/rails/commit/9acd686753c43612984aaa4002e80113fda2b255 > > As far as I can tell, we have never actually needed to wrap the primary > key in an array and don't have its use anywhere. > > Thoughts on changing this behaviour and just returning single primary > field instead? > > > The composite_primary_key user community probably have an opinion on this. > They'd be stuck making workarounds to replace this: > > > https://github.com/composite-primary-keys/composite_primary_keys/blob/7a519429600715f7140f3dcda5efe18c73976996/lib/composite_primary_keys/base.rb#L126 > > --Matt Jones > > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" 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 http://groups.google.com/group/rubyonrails-core. > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" 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 http://groups.google.com/group/rubyonrails-core. For more options, visit https://groups.google.com/d/optout.
