Removes unneeded ApiTemplate.create in favor of ApiTemplate.new

benedikt · Mar 29, 2012 · b05ce9d · b05ce9d
1 parent 298742e
commit b05ce9d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/lib/acts_as_api/api_template.rb b/lib/acts_as_api/api_template.rb
@@ -11,18 +11,11 @@ class ApiTemplate < Hash
 
     # The name of the api template as a Symbol.
     attr_accessor :api_template
-
-    attr_reader :options
 
-    # Returns a new ApiTemplate with the api template name
-    # set to the passed template.
-    def self.create(template)
-      t = ApiTemplate.new
-      t.api_template = template
-      return t
-    end
+    attr_reader :options
 
-    def initialize
+    def initialize(api_template = nil)
+      self.api_template = api_template
       @options ||= {}
     end
 

diff --git a/lib/acts_as_api/base.rb b/lib/acts_as_api/base.rb
@@ -34,7 +34,7 @@ def acts_as_api?#:nodoc:
       # be contained in the api responses.
       def api_accessible(api_template, options = {}, &block)
 
-        attributes = api_accessible_attributes(api_template) || ApiTemplate.create(api_template)
+        attributes = api_accessible_attributes(api_template) || ApiTemplate.new(api_template)
 
         attributes.merge!(api_accessible_attributes(options[:extend])) if options[:extend]