Allow InfoRequest.find_old_unclassified to accept an offset param so it can be used in conjunction with pagination.

2 files changed, 11 insertions, 2 deletions

diff --git a/app/models/info_request.rb b/app/models/info_request.rb
index ed54da840..2e16d0f58 100644
--- a/app/models/info_request.rb
+++ b/app/models/info_request.rb
@@ -977,8 +977,9 @@ public
 
     def InfoRequest.find_old_unclassified(extra_params={})
         params = old_unclassified_params(extra_params, include_last_response_time=true)
-        params[:limit] = extra_params[:limit] if extra_params[:limit]
-        params[:include] = extra_params[:include] if extra_params[:include]
+        [:limit, :include, :offset].each do |extra|
+            params[extra] = extra_params[extra] if extra_params[extra]
+        end
         if extra_params[:order]
             params[:order] = extra_params[:order]
             params.delete(:select)
diff --git a/spec/models/info_request_spec.rb b/spec/models/info_request_spec.rb
index 76be32e0c..204c600d9 100644
--- a/spec/models/info_request_spec.rb
+++ b/spec/models/info_request_spec.rb
@@ -341,6 +341,14 @@ describe InfoRequest do
             InfoRequest.find_old_unclassified(:limit => 5)
         end
 
+        it 'should ask for requests using any offset param supplied' do
+            InfoRequest.should_receive(:find).with(:all, {:select => anything,
+                                                          :order => anything,
+                                                          :conditions=> anything,
+                                                          :offset => 100})
+            InfoRequest.find_old_unclassified(:offset => 100)
+        end
+
         it 'should not limit the number of requests returned by default' do
             InfoRequest.should_not_receive(:find).with(:all, {:select => anything,
                                                               :order => anything,