This project is archived and is in readonly mode.
[PATCH] Add :touch option to has_one associations
-
Diego CarrionPatch is attached.
-
Diego Carrion
- Tag set to activerecord, has_many, rails, touch
-
Diego Carrion
- Title changed from Add :touch option to has_many associations to [PATCH] Add :touch option to has_many associations
-
Diego Carrion
- Assigned user set to Ryan Bigg
-
-
Diego Carrion
- Tag changed from activerecord, has_many, rails, touch to activerecord, has_many, patch, rails, touch
-
Diego Carrion
Some people on the list gave me some feedback and because of that I added also the touch support for has_one associations.
A new patch is attached.
-
Diego Carrion
- Tag changed from activerecord, has_many, patch, rails, touch to activerecord, has_many, has_one, patch, rails, touch
Some people on the list gave me some feedback and because of that I added also the touch support for has_one associations.
A new patch is attached.
-
Diego Carrion
- Title changed from [PATCH] Add :touch option to has_many associations to [PATCH] Add :touch option to has_many and has_one associations
-
Diego Carrion
- Title changed from [PATCH] Add :touch option to has_many and has_one associations to [PATCH] Add :touch option to has_many associations
Sorry, I forgot the RDoc documenation in the previous patch, added a new one.
-
Michael KoziarskiAs discussed on the mailing list, I don't believe that the feature is justified for the sake of consistency alone. It would hide severe performance issues inside a simple declaration.
People needing this functionality can write a tiny after_save themselves.
-
Michael Koziarski
- State changed from new to wontfix
As discussed on the mailing list, I don't believe that the feature is justified for the sake of consistency alone. It would hide severe performance issues inside a simple declaration.
People needing this functionality can write a tiny after_save themselves.
-
Diego Carrion
- Title changed from [PATCH] Add :touch option to has_many associations to [PATCH] Add :touch option to has_one associations
As discussed with Michael via the mailing list, the touch option for the has_one association does make sense so Im sending a patch.
I added also a test covering the scenario where the associated object doesn't exists, given that the validation was in the code but after deleting it, nothing broke.
