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.

Make ActiveSupport::Configurable work with Modules

#6486

Currently ActiveSupport::Configurable mixin can only be included in Classes (because it calls their superclass method). By adding an additional check (responds_to?(:superclass)), the mixin can be included in regular Modules as well. Patch with tests included.

Reported by Josep M. Bach · February 28th, 2011 @ 01:36 PM

State: committed
Milestone: none
Assigned to: Santiago Pastorino Santiago Pastorino
Importance: Low

Activity

  1. Jeff Kreeftmeijer
    Jeff Kreeftmeijer
    • State changed from new to verified
    • Assigned user set to Santiago Pastorino
    • Importance changed from to Low

    Verified. The added test fails (raising a NameError because superclass is undefined) without the patch. Thanks Josep! :)

    February 28th, 2011 @ 02:35 PM

  2. Repository
    Repository
    • State changed from verified to committed

    February 28th, 2011 @ 03:44 PM