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.

This project is archived and is in readonly mode.

Fix for ActionMailer::Base.method_missing so that it raises NoMethodError when no method is found

#1330

This was originally reported in #1318 Fix for matches_dynamic_method? on ActionMailer

ActionMailer::Base's method_messing method raises an exception

undefined method []' for nil:NilClass"

when it does match any method.

This bug was introduced in commit 3cf773b187e803e16b8237e5923fa4c1139cde8a.

The attached patch resolves this.

Reported by Joel Chippindale · November 5th, 2008 @ 09:57 AM

State: resolved
Milestone: 2.x
Assigned to: nobody
Importance: none

Activity

  1. Tom Lea
    Tom Lea

    +1 Confusing error message made clear.

    November 5th, 2008 @ 03:11 PM

  2. James Mead
    James Mead

    +1 Thanks for fixing my original commit and sorry for introducing the bug. Cheers, James.

    November 11th, 2008 @ 01:41 PM

  3. Repository
    Repository
    • State changed from new to resolved

    (from [a62e9e90d8f47a643c9355f782ab4891fa8fefaa]) Fix for ActionMailer::Base.method_missing so that it raises NoMethodError when no method is found [#1330 Fix for ActionMailer::Base.method_missing so that it raises NoMethodError whe... state:resolved]

    Signed-off-by: Joshua Peek josh@joshpeek.com http://github.com/rails/rails/co...

    November 11th, 2008 @ 03:47 PM