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.

Allow per-request controller_path.

#1141

Don't let the description fool you -- it's a trivial, two-line patch.

This is about how ActionController::Base#controller_path is used.

From the commit summary:

The instance method calls the class method by default. With this change, controllers can override the instance method to set a dynamic controller_path for a given request.

Rationale:

It is possible to override the whole view_path (with prepend_view_path and append_view_path) for all requests, or for an individual request. It seems that the controller_path is a smaller change.

In fact, I believe that any argument against doing this would apply even moreso to modifying the view_path.

Reported by David Masover · September 30th, 2008 @ 04:32 AM

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

Activity

  1. Michael Koziarski
    Michael Koziarski
    • Assigned user set to josh

    Josh is the guy who's done the most work here

    September 30th, 2008 @ 11:13 AM

  2. Repository
    Repository
    • State changed from new to resolved

    (from [e69b506abd40222c87be28398c5191db160740cb]) Call controller_path instance method so it can be easily overridden [#1141 Allow per-request controller_path. state:resolved]

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

    September 30th, 2008 @ 06:08 PM