This project is archived and is in readonly mode.
add has_many test for #5579 case
Reported by Neeraj Singh | September 26th, 2010 @ 05:57 AM | in 3.0.2
Fix for #5579 involved the code change for both has_one
and has_many
relationships. The patch included test only for has_one. This
patch
adds test for has_many relationship.
However because of a bug in the way nested_attribute works. This patch should be applied only after the patch for #5705 has been applied.
Comments and changes to this ticket
-
Neeraj Singh September 26th, 2010 @ 05:59 AM
- Assigned user set to “José Valim”
- Tag set to “patched”
-
Repository September 27th, 2010 @ 10:12 PM
- State changed from “new” to “resolved”
(from [4966b915fe96db73933d33176c1ea5cc53e58a22]) Fix for #5579 involved the code change for both has_one and has_many relationships. The path included test only for has_one. This patch adds test for has_many relationship.
[#5706 state:resolved]
Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/4966b915fe96db73933d33176c1ea5... -
José Valim September 27th, 2010 @ 10:23 PM
- State changed from “resolved” to “open”
- Milestone cleared.
Neeraj, could you please provide a patch for 3-0-stable? Thanks! :D
-
-
-
Repository September 28th, 2010 @ 09:36 AM
- State changed from “open” to “resolved”
(from [2525bfc5dd255a865ae3fae77b5daff833396758]) backport of #5706 and #5579 to 3-0-stable
[#5706 state:resolved]
Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/2525bfc5dd255a865ae3fae77b5daf... -
Andrea Campi October 11th, 2010 @ 07:22 AM
- Tag changed from “patched” to “activerecord, patch”
bulk tags cleanup
-
-
-
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
Jeremy Kemper
José Valim
Neeraj Singh
Ryan Bigg
Attachments
Referenced by
-
5706
add has_many test for #5579 case
[#5706 state:resolved]
-
5706
add has_many test for #5579 case
(from [2525bfc5dd255a865ae3fae77b5daff833396758])
backpor...
-
5706
add has_many test for #5579 case
[#5706 state:resolved]