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.

Prevent Dependencies.new_constants_in cleanup code masking real error when collecting watch_frames fails

#24

Previously, the code collecting watch_frames could fail leaving watch_frames

defined but nil. The cleanup code checked watch_frames is defined, but not that it holds a value, raising an undefined method on NilClass error rather than the original cause. This can make debugging the underlying problem a total pain.

My (very simple) patch fixes this:

http://github.com/tomafro/rails/...

Reported by Tom Ward · April 19th, 2008 @ 04:28 PM

State: resolved
Milestone: none
Assigned to: Jeremy Kemper Jeremy Kemper
Importance: none

Activity

  1. josh
    josh
    • Assigned user set to Jeremy Kemper
    • State changed from new to open

    April 19th, 2008 @ 10:24 PM

  2. Tom Ward
  3. Pratik
    Pratik
    • State changed from open to resolved

    May 20th, 2008 @ 11:50 AM

  4. Jeff Kreeftmeijer
    Jeff Kreeftmeijer
    • Tag cleared.

    Automatic cleanup of spam.

    November 8th, 2010 @ 08:22 AM