Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

This project is archived and is in readonly mode.

Refactor protected attribute assignment warnings

#804

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

Reported by Trevor Turk · August 12th, 2008 @ 03:50 AM

State: resolved
Milestone: 2.x
Assigned to: Michael Koziarski Michael Koziarski
Importance: none

Activity

  1. Trevor Turk
    Trevor Turk

    Here's an example monkey patch that raises errors during tests, instead of just logging them.

    August 12th, 2008 @ 03:51 AM

  2. Repository
    Repository

    (from [81c12d1f6359eb5e52b376f1f3552097a144cc8b]) move logging of protected attribute removal into log_protected_attribute_removal method

    Signed-off-by: Michael Koziarski michael@koziarski.com [#804 Refactor protected attribute assignment warnings status:committed] http://github.com/rails/rails/co...

    August 12th, 2008 @ 05:16 PM

  3. Trevor Turk
    Trevor Turk

    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'

    :)

    August 12th, 2008 @ 05:33 PM

  4. Pratik
    Pratik
    • State changed from new to resolved

    August 21st, 2008 @ 07:30 PM