This project is archived and is in readonly mode.

#24 ✓resolved
Tom Ward

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

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

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/...

Comments and changes to this ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

Referenced by

Pages