This project is archived and is in readonly mode.
Saving objects with associated has_one :throughs loaded broken
Reported by Will Bryant | March 11th, 2009 @ 01:16 PM
5cda000, which changed the autosave logic between rc1 & rc2, broke the save of any objects that have has_one :through associations :included.
The attached patch fixes this using the same bit of code used higher up in autosave_associations.rb, and adds spec coverage for has_one :through, has_one, and belongs_to to make sure they don't have the same problem.
Comments and changes to this ticket
-
Eloy Duran March 11th, 2009 @ 02:26 PM
- Assigned user set to “Eloy Duran”
- State changed from “new” to “open”
- Milestone cleared.
Patch applies and fixes mentioned problem. So +1.
I just wonder if the tests shouldt be added to the "default" tests in autosave_association_test.rb.
-
-
Eloy Duran March 12th, 2009 @ 10:21 AM
- State changed from “open” to “verified”
- Assigned user changed from “Eloy Duran” to “Pratik”
-
Repository March 12th, 2009 @ 02:36 PM
- State changed from “verified” to “resolved”
(from [92dadf6d7927fde1482ba1d96d0916093bfb83ca]) Fixed autosave checks on objects with hm:t in :include [#2213 state:resolved]
Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Eloy Duran
José Valim
Pratik
Attachments
Referenced by
-
2213
Saving objects with associated has_one :throughs loaded broken
(from [92dadf6d7927fde1482ba1d96d0916093bfb83ca]) Fixed a...