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

You must be a member of this account.

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