#804 Refactor protected attribute assignment warnings - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#804 ✓resolved

Refactor protected attribute assignment warnings

Reported by Trevor Turk | August 12th, 2008 @ 03:50 AM | in 2.x

As discussed here:

http://groups.google.com/group/r...

I've moved the logging of protected attribute assignment errors into a method to allow for monkey patching at least. Please see the attached and let me know if there's anything else I can do to help.

Thanks! - Trevor

log_protected_attribute_removal.diff 1.4 KB

Comments and changes to this ticket

You flagged this item as spam.
Trevor Turk August 12th, 2008 @ 03:51 AM
Here's an example monkey patch that raises errors during tests, instead of just logging them.
- log_protected_attribute_removal.rb 213 Bytes
Repository August 12th, 2008 @ 05:16 PM
(from [81c12d1f6359eb5e52b376f1f3552097a144cc8b]) move logging of protected attribute removal into log_protected_attribute_removal method

Signed-off-by: Michael Koziarski michael@koziarski.com [#804 status:committed] http://github.com/rails/rails/co...
You flagged this item as spam.
Trevor Turk August 12th, 2008 @ 05:33 PM
Thanks for committing this, Koz! Just in case anyone tries to use the attached monkey patch, note that the first line should have read:

if RAILS_ENV == 'test'

:)
Pratik August 21st, 2008 @ 07:30 PM
- State changed from “new” to “resolved”