On Tue, Feb 10, 2009 at 3:29 PM, Will Bryant <[email protected]> wrote: > > Hi all, > > Good to see Array.wrap get added. However it doesn't DWIM with associations: > >>> Array.wrap(Icp.find(16).registers) > => [[#<Register ...>]] > >>> Array.wrap(DisputedRead.unresolved) > => [[#<DisputedRead ...>, #<DisputedRead ...>]] > > ie. array-like collections and scopes aren't getting treated as arrays > and are being wrapped again. > > The short-term solution the coder who hit this suggested was to add a > case to the #wrap: > > when ActiveRecord::NamedScope::Scope, > ActiveRecord::Associations::AssociationCollection > object.to_a > > But this has the effect of loading ActiveRecord from ActiveSupport, > which is undesirable. Is there a better way to detect > association-collection-ness? > > I'm thinking that it might be better to check for the #to_ary method > on all objects and use that instead, without having a special case for > collections/scopes. Does that sound OK? If so I'll submit a patch.
Yeah, good call. Please do! jeremy --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
