Add top_rentals, fixes #2

nmunson · Feb 23, 2015 · 088c9ca · 088c9ca
1 parent 0de6b54
commit 088c9ca
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/README.rdoc b/README.rdoc
@@ -53,7 +53,7 @@ Each object provides a find() method which accepts a number of options:
 
   RottenList.find(:type => "upcoming", :limit => 10, :expand_results => true)
 
-[:type]            specifies a list type to retrieve.  Valid options are 'box_office', 'in_theaters', 'opening', 'upcoming', 'new_releases'.
+[:type]            specifies a list type to retrieve.  Valid options are 'box_office', 'in_theaters', 'opening', 'upcoming', 'new_releases', 'top_rentals'.
 [:limit]           see RottenMovie
 [:expand_results]  see RottenMovie
 

diff --git a/lib/rottentomatoes/rotten_list.rb b/lib/rottentomatoes/rotten_list.rb
@@ -4,7 +4,7 @@ class RottenList
 
     def self.find(options)
       raise ArgumentError, ":type must be set" if (options[:type].nil?)
-      if !["box_office", "in_theaters", "opening", "upcoming", "new_releases"].include?(options[:type])
+      if !["box_office", "in_theaters", "opening", "upcoming", "new_releases", "top_rentals"].include?(options[:type])
         raise ArgumentError, ":type must be a recognized format"
       end
 

diff --git a/lib/rottentomatoes/rottentomatoes.rb b/lib/rottentomatoes/rottentomatoes.rb
@@ -34,7 +34,7 @@ def self.api_call(method, options)
       end
 
       if (method == "lists")
-        url += (options[:type] == "new_releases") ? "/dvds/" : "/movies/"
+        url += (options[:type] == "new_releases" || options[:type] == "top_rentals") ? "/dvds/" : "/movies/"
         url += options[:type]
       end