Hi ! Another patch that I feel needs more attention: http://dev.rubyonrails.org/ticket/3024 This patch ensures aggregations are reloaded when the parent record is. This bit me during unit testing. I was going bonkers thinking there was something wrong with my code. Anyway, this patch applies the same strategy as associations reloading. Worth taking a look and applying ? It includes unit tests too. I believe this is fit for trunk and stable. Bye ! -- François Beausoleil http://blog.teksol.info/ _______________________________________________ Rails-core mailing list Rails-core@lists.rubyonrails.org http://lists.rubyonrails.org/mailman/listinfo/rails-core
David Heinemeier Hansson
2005-Dec-08 04:56 UTC
Re: #3024 - Reloading aggregation instances
> Worth taking a look and applying ? It includes unit tests too. I > believe this is fit for trunk and stable.Great job. Applied. -- David Heinemeier Hansson http://www.loudthinking.com -- Broadcasting Brain http://www.basecamphq.com -- Online project management http://www.backpackit.com -- Personal information manager http://www.rubyonrails.com -- Web-application framework
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On Dec 5, 2005, at 4:14 PM, Francois Beausoleil wrote:> Another patch that I feel needs more attention: > http://dev.rubyonrails.org/ticket/3024 > > This patch ensures aggregations are reloaded when the parent record > is. This bit me during unit testing. I was going bonkers thinking > there was something wrong with my code. > > Anyway, this patch applies the same strategy as associations > reloading. > > Worth taking a look and applying ? It includes unit tests too. I > believe this is fit for trunk and stable.Applied. Look for it in 1.0rc5. Thanks for the patch (and tests!) Best, jeremy -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (Darwin) iD8DBQFDl70nAQHALep9HFYRAspiAJ0f1Jr/+D0TE8DUy1BHkkuKMH0CHQCcDl1s Nmd5Wumy2T82U0S+4O454QE=6V/U -----END PGP SIGNATURE-----