I tidied up some association preload stuff where it wasn''t quoting table names if anyone wants to take a look: http://rails.lighthouseapp.com/projects/8994/tickets/45-association-preload-should-quote-table-names#ticket-45-1 Fred --~--~---------~--~----~------------~-------~--~----~ 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 rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
Any progress on the patch to fix the integer key assumption in the same code? -Chris On Apr 25, 3:31 am, Frederick Cheung <frederick.che...@gmail.com> wrote:> I tidied up some association preload stuff where it wasn''t quoting > table names if anyone wants to take a look:http://rails.lighthouseapp.com/projects/8994/tickets/45-association-p... > > Fred--~--~---------~--~----~------------~-------~--~----~ 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 rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
On 25 Apr 2008, at 13:23, Chris Cruft wrote:> > Any progress on the patch to fix the integer key assumption in the > same code? >I was planning to look at that after this (already written some failing tests) Fred> -Chris > > On Apr 25, 3:31 am, Frederick Cheung <frederick.che...@gmail.com> > wrote: >> I tidied up some association preload stuff where it wasn''t quoting >> table names if anyone wants to take a look:http://rails.lighthouseapp.com/projects/8994/tickets/45-association-p >> ... >> >> Fred > >--~--~---------~--~----~------------~-------~--~----~ 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 rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
On 25 Apr 2008, at 13:23, Chris Cruft wrote:> > Any progress on the patch to fix the integer key assumption in the > same code? >I''ve had a stab at it at http://rails.lighthouseapp.com/projects/8994/tickets/33-associationspreload-casts-foreign-keys-to-integers Does that fix your problems? Fred> -Chris > > On Apr 25, 3:31 am, Frederick Cheung <frederick.che...@gmail.com> > wrote: >> I tidied up some association preload stuff where it wasn''t quoting >> table names if anyone wants to take a look:http://rails.lighthouseapp.com/projects/8994/tickets/45-association-p >> ... >> >> Fred > --~--~---------~--~----~------------~-------~--~----~ > 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 rubyonrails-core@googlegroups.com > To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com > For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en > -~----------~----~----~----~------~----~------~--~--- >