This project is archived and is in readonly mode.
[PATCH] add strip_attributes support to ActiveModel and Active Record
-
CancelProfileIsBroken
- Tag changed from 3.0, activemodel, activerecord, strip_attribtutes to 3.0, activemodel, activerecord, bugmash, strip_attribtutes
-
Blue Box Jesse
BugMash: +1
Applies to master cleanly. Do not see any related errors as a result of the patch.
Good idea / useful tool.
-
sr.iniv.t
+1 verified.
Great idea - i have run into situations where we needed this kind of behavior would have been helpful. The patch applies without any problems and all tests pass.
-
CancelProfileIsBroken
- Tag changed from 3.0, activemodel, activerecord, bugmash, strip_attribtutes to 3.0, activemodel, activerecord, bugmash-review, strip_attribtutes
-
Rizwan Reza
- Tag changed from 3.0, activemodel, activerecord, bugmash-review, strip_attribtutes to 3.0, activemodel, activerecord, bugmash, strip_attribtutes
The patch doesn't apply to master anymore.
-
Prem Sichanugrist (sikachu)
I've modify the patch to be applied on
master -
Prem Sichanugrist (sikachu)
- Tag changed from 3.0, activemodel, activerecord, bugmash, strip_attribtutes to 3.0, activemodel, activerecord, bugmash, bugmash-review, review, strip_attribtutes
-
Prem Sichanugrist (sikachu)
@Josh Sharpe
After I have discuss your addition in #rails-contrib, I can't really find a good use case for this method. So, I think this feature would be nice to be made as a plugin instead. If there're enough user using it, then it can be merged into core later.
Thank you :)
-
Yehuda Katz (wycats)
- State changed from new to invalid
-
Rizwan Reza
- Tag changed from 3.0, activemodel, activerecord, bugmash, bugmash-review, review, strip_attribtutes to 3.0, activemodel, activerecord, bugmash, review, strip_attribtutes
