From d2ccb852d4e1f6f1b01e43f32213053ae3bef408 Mon Sep 17 00:00:00 2001
From: Joshua Peek <josh@joshpeek.com>
Date: Fri, 18 Jul 2008 16:00:20 -0500
Subject: Removed lagacy TemplateHandler#render API. Left in a legacy
 TemplateHandler and Compilable stub so plugins will not have to change
 anything.

---
 actionpack/lib/action_view/base.rb                 |  4 ++--
 actionpack/lib/action_view/renderable.rb           |  6 +++---
 actionpack/lib/action_view/template_handler.rb     | 23 ++++++++--------------
 actionpack/lib/action_view/template_handlers.rb    |  1 -
 .../action_view/template_handlers/compilable.rb    | 20 -------------------
 5 files changed, 13 insertions(+), 41 deletions(-)
 delete mode 100644 actionpack/lib/action_view/template_handlers/compilable.rb

(limited to 'actionpack/lib')

diff --git a/actionpack/lib/action_view/base.rb b/actionpack/lib/action_view/base.rb
index a6872b1a47..ae6b284854 100644
--- a/actionpack/lib/action_view/base.rb
+++ b/actionpack/lib/action_view/base.rb
@@ -379,8 +379,8 @@ module ActionView #:nodoc:
         @assigns.each { |key, value| instance_variable_set("@#{key}", value) }
       end
 
-      def execute(template, local_assigns = {})
-        send(template.method(local_assigns), local_assigns) do |*names|
+      def execute(method, local_assigns = {})
+        send(method, local_assigns) do |*names|
           instance_variable_get "@content_for_#{names.first || 'layout'}"
         end
       end
diff --git a/actionpack/lib/action_view/renderable.rb b/actionpack/lib/action_view/renderable.rb
index 5090d0160a..4f865cbced 100644
--- a/actionpack/lib/action_view/renderable.rb
+++ b/actionpack/lib/action_view/renderable.rb
@@ -19,7 +19,7 @@ module ActionView
     memoize :handler
 
     def compiled_source
-      handler.new(nil).compile(self) if handler.compilable?
+      handler.call(self)
     end
     memoize :compiled_source
 
@@ -27,8 +27,8 @@ module ActionView
       view._first_render ||= self
       view._last_render = self
       view.send(:evaluate_assigns)
-      compile(local_assigns) if handler.compilable?
-      handler.new(view).render(self, local_assigns)
+      compile(local_assigns)
+      view.send(:execute, method(local_assigns), local_assigns)
     end
 
     def method(local_assigns)
diff --git a/actionpack/lib/action_view/template_handler.rb b/actionpack/lib/action_view/template_handler.rb
index e2dd305f93..d7e7c9b199 100644
--- a/actionpack/lib/action_view/template_handler.rb
+++ b/actionpack/lib/action_view/template_handler.rb
@@ -1,21 +1,14 @@
-module ActionView
-  class TemplateHandler
-    def self.compilable?
-      false
-    end
-
-    def initialize(view)
-      @view = view
-    end
+# Legacy TemplateHandler stub
 
-    def render(template, local_assigns = {})
-    end
-
-    def compile(template)
+module ActionView
+  module TemplateHandlers
+    module Compilable
     end
+  end
 
-    def compilable?
-      self.class.compilable?
+  class TemplateHandler
+    def self.call(template)
+      new.compile(template)
     end
   end
 end
diff --git a/actionpack/lib/action_view/template_handlers.rb b/actionpack/lib/action_view/template_handlers.rb
index 1471e99e01..6c8aa4c2a7 100644
--- a/actionpack/lib/action_view/template_handlers.rb
+++ b/actionpack/lib/action_view/template_handlers.rb
@@ -1,5 +1,4 @@
 require 'action_view/template_handler'
-require 'action_view/template_handlers/compilable'
 require 'action_view/template_handlers/builder'
 require 'action_view/template_handlers/erb'
 require 'action_view/template_handlers/rjs'
diff --git a/actionpack/lib/action_view/template_handlers/compilable.rb b/actionpack/lib/action_view/template_handlers/compilable.rb
deleted file mode 100644
index a0ebaefeef..0000000000
--- a/actionpack/lib/action_view/template_handlers/compilable.rb
+++ /dev/null
@@ -1,20 +0,0 @@
-module ActionView
-  module TemplateHandlers
-    module Compilable
-      def self.included(base)
-        base.extend ClassMethod
-      end
-
-      module ClassMethod
-        # If a handler is mixin this module, set compilable to true
-        def compilable?
-          true
-        end
-      end
-
-      def render(template, local_assigns = {})
-        @view.send(:execute, template, local_assigns)
-      end
-    end
-  end
-end
-- 
cgit v1.2.3