Add ability to specify which arguments constitute the collection request

Signed-off-by: David Souza <david@souza.net>

lib/cached_resource/caching.rb

@@ -61,20 +61,20 @@ def update_singles_cache(updates)
       # Update the "mother" collection with an array of updates.
       def update_collection_cache(updates)
         updates = Array(updates)
-        collection = cache_read(:all)
+        collection = cache_read(cached_resource.collection_arguments)
 
         if collection && !updates.empty?
           store = RUBY_VERSION.to_f < 1.9 ? ActiveSupport::OrderedHash.new : {}
           index = collection.inject(store) {|hash, object| hash[object.send(primary_key)] = object; hash}
           updates.each {|object| index[object.send(primary_key)] = object}
-          cache_write(:all, index.values)
+          cache_write(cached_resource.collection_arguments, index.values)
         end
       end
 
       # Determine if the given arguments represent
       # the entire collection of objects.
       def is_collection?(*arguments)
-        arguments.length == 1 && arguments[0] == :all
+        arguments == cached_resource.collection_arguments
       end
 
       # Read a entry from the cache for the given key.

lib/cached_resource/configuration.rb

@@ -24,6 +24,7 @@ def initialize(options={})
         :enabled => true,
         :ttl => 604800,
         :collection_synchronize => false,
+        :collection_arguments => [:all],
         :cache => defined?(Rails.cache)  && Rails.cache || CACHE,
         :logger => defined?(Rails.logger) && Rails.logger || LOGGER
       }.merge(options))

spec/cached_resource/configuration_spec.rb

@@ -17,6 +17,10 @@
       configuration.collection_synchronize.should == false
     end
 
+    it "should default to :all for collection arguments" do
+      configuration.collection_arguments.should == [:all]
+    end
+
     describe "outside a Rails environment" do
       it "should be logging to a buffered logger attached to a NilIO" do
         configuration.logger.class.should == ActiveSupport::BufferedLogger
@@ -55,7 +59,13 @@
   describe "when initialized through cached resource" do
     before(:each) do
       class Foo < ActiveResource::Base
-        cached_resource :ttl => 1, :cache => "cache", :logger => "logger", :enabled => false,  :collection_synchronize => true, :custom => "irrelevant"
+        cached_resource :ttl => 1,
+                        :cache => "cache",
+                        :logger => "logger",
+                        :enabled => false,
+                        :collection_synchronize => true,
+                        :collection_arguments => [:every],
+                        :custom => "irrelevant"
       end
     end
 
@@ -69,6 +79,7 @@ class Foo < ActiveResource::Base
       Foo.cached_resource.logger.should == "logger"
       Foo.cached_resource.enabled.should == false
       Foo.cached_resource.collection_synchronize.should == true
+      Foo.cached_resource.collection_arguments.should == [:every]
       Foo.cached_resource.custom.should == "irrelevant"
     end
   end