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 <al2o...@gmail.com> wrote: > > On Jan 17, 2015, at 6:10 AM, Vipul A M <vipulnsw...@gmail.com> 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 rubyonrails-core+unsubscr...@googlegroups.com. > To post to this group, send email to rubyonrails-core@googlegroups.com. > 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 rubyonrails-core+unsubscr...@googlegroups.com. To post to this group, send email to rubyonrails-core@googlegroups.com. Visit this group at http://groups.google.com/group/rubyonrails-core. For more options, visit https://groups.google.com/d/optout.
