Thanks for the quick answer, Pascal!

Am 24.02.2008 um 21:16 schrieb Pascal:
> +1 but...

cool

> ...this could be a major source of conflicts between engines and
> should probably be used only when nothing else works.  For example
> using a subclass of ApplicationController that all your engine
> Controllers then subclass would avoid many such conflicts.  See what I
> mean?

In general I totally agree.

In my usecase I can safely assume that all engines are designed to fit  
together though and readability-wise it really makes most sense to use  
ApplicationController here.

> The patch looks harmless enough to me (with the above caveat).

Great :)


--
sven fuchs                      [EMAIL PROTECTED]
artweb design           http://www.artweb-design.de
grünberger 65           + 49 (0) 30 - 47 98 69 96 (phone)
d-10245 berlin          + 49 (0) 171 - 35 20 38 4 (mobile)



_______________________________________________
Engine-Developers mailing list
[email protected]
http://lists.rails-engines.org/listinfo.cgi/engine-developers-rails-engines.org

Reply via email to