Minor refactoring of jsonify format handling in template

bsiggelkow · Aug 24, 2011 · 838de5d · 838de5d
1 parent 9336f24
commit 838de5d
Showing 1 changed file with 16 additions and 10 deletions.
diff --git a/lib/jsonify-rails/jsonify_builder.rb b/lib/jsonify-rails/jsonify_builder.rb
@@ -7,31 +7,37 @@ class JsonifyBuilder < Template::Handler
 
         self.default_format = Mime::JSON
 
-        def format
-          Rails.application.config.respond_to?(:jsonify_format) ? Rails.application.config.jsonify_format : 'plain'
-        end
-
         def compile(template)
-          "json = ::Jsonify::Builder.new(:format => :#{format});" +
+          "json = ::Jsonify::Builder.new(:format => :#{jsonify_format});" +
             template.source +
           ";json.compile!;"
         end
+
+        private 
+
+        def jsonify_format
+          Rails.application.config.respond_to?(:jsonify_format) ? Rails.application.config.jsonify_format : 'plain'
+        end
+
       end
     else
       class JsonifyBuilder
         def default_format
           Mime::JSON
         end
 
-        def self.format
-          Rails.application.config.respond_to?(:jsonify_format) ? Rails.application.config.jsonify_format : 'plain'
-        end
-
         def self.call(template)
-          "json = ::Jsonify::Builder.new(:format => :#{format});" +
+          "json = ::Jsonify::Builder.new(:format => :#{jsonify_format});" +
             template.source +
           ";json.compile!;"
         end
+
+        private 
+
+        def self.jsonify_format
+          Rails.application.config.respond_to?(:jsonify_format) ? Rails.application.config.jsonify_format : 'plain'
+        end
+
       end
     end
   end