Skip to content
Browse files

Change layout discovery to take into account the change in semantics …

…with File.join and nil arguments.

git-svn-id: 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
1 parent cffd1a9 commit 8d4d88a4f01ed7b569b62489654f585434beea3a Marcel Molina committed Jan 22, 2006
Showing with 3 additions and 1 deletion.
  1. +2 −0 actionpack/CHANGELOG
  2. +1 −1 actionpack/lib/action_controller/layout.rb
2 actionpack/CHANGELOG
@@ -1,5 +1,7 @@
+* Change layout discovery to take into account the change in semantics with File.join and nil arguments. [Marcel Molina Jr.]
* Raise a RedirectBackError if redirect_to :back is called when there's no HTTP_REFERER defined #3049 []
* Treat timestamps like datetimes for scaffolding purposes #3388 [Maik Schmidt]
2 actionpack/lib/action_controller/layout.rb
@@ -207,7 +207,7 @@ def active_layout(passed_layout = nil)
# Explicitly passed layout names with slashes are looked up relative to the template root,
# but auto-discovered layouts derived from a nested controller will contain a slash, though be relative
# to the 'layouts' directory so we have to check the file system to infer which case the layout name came from.
- nested_controller =, 'layouts', active_layout)))
+ nested_controller =, 'layouts', active_layout))) if active_layout
active_layout.include?('/') && !nested_controller ? active_layout : "layouts/#{active_layout}" if active_layout

0 comments on commit 8d4d88a

Please sign in to comment.
Something went wrong with that request. Please try again.