Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

AssociationCollection#include? ignoring build associated objects (not working in Rails 3)

#5336

Hi,

I've been working on ticket #3742 Remove remaining references to script/plugin sources for Rails 2.3.x, but I've noticed this is happening on Rails 3.x version too.

So, I've ported the same fix (that applies cleanly on master) here, let me know if you got any suggestion.

Reported by Marcelo Giorgi · August 8th, 2010 @ 07:46 PM

State: committed
Milestone: 3.0.2
Assigned to: Aaron Patterson Aaron Patterson
Importance: Low

Activity

  1. Marcelo Giorgi
    Marcelo Giorgi

    Here I attach a minor semantic change on the patch (I was using #map, where I should use #each)

    September 4th, 2010 @ 04:01 PM

  2. Santiago Pastorino
    Santiago Pastorino
    • State changed from new to open
    • Milestone cleared.
    • Assigned user set to Aaron Patterson
    • Importance changed from to Low

    September 28th, 2010 @ 07:20 PM

  3. Aaron Patterson
    Aaron Patterson

    @Marcelo do you mind rebasing this patch against 3-0-stable? Then I'll apply it.

    Thanks!

    September 30th, 2010 @ 06:32 PM

  4. Marcelo Giorgi
    Marcelo Giorgi

    Here it is, I've just tested and applies cleanly on master.

    Thank you very much!

    September 30th, 2010 @ 07:36 PM

  5. Jeremy Kemper
  6. Aaron Patterson
    Aaron Patterson

    Can you rebase this against master? Then I'll apply. Thanks!

    October 21st, 2010 @ 01:27 AM

  7. Marcelo Giorgi
    Marcelo Giorgi

    Hi Aaron,

    Seems like you already applied it: http://github.com/rails/rails/commit/ef6df93a8ddb675f1298973bb34782...

    I guess, that the reason for this ticket not to be closed, is because I put the wrong ticket number on the commit's comment :P. So, perhaps we can close this one manually.

    BTW: Thanks for applying this!

    October 22nd, 2010 @ 12:32 PM

  8. Santiago Pastorino
    Santiago Pastorino
    • State changed from open to committed

    October 22nd, 2010 @ 01:19 PM