Prefer tap to ensure in this case, as we don't want assignment to hap…

…pen when error is raised
ianwhite · Feb 9, 2009 · 62ab2b5 · 62ab2b5
1 parent ad0976d
commit 62ab2b5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/inherit_views.rb b/lib/inherit_views.rb
@@ -130,9 +130,9 @@ def self.included(base)
       base.class_eval do
         # use InheritViews::PathSet, and give it my controller's inherit_view_paths
         def view_paths=(value)
-          @view_paths = InheritViews::PathSet.new(value)
-        ensure
-          @view_paths.inherit_view_paths = controller.inherit_view_paths if (controller.inherit_views? rescue false)
+          @view_paths = InheritViews::PathSet.new(value).tap do |paths|
+            paths.inherit_view_paths = controller.inherit_view_paths if (controller.inherit_views? rescue false)
+          end
         end
 
         alias_method_chain :render, :parent