Changing the note search limit from 9999 to 10000
authorToby Murray <toby.murray@gmail.com>
Thu, 16 Jan 2014 08:28:34 +0000 (02:28 -0600)
committerTom Hughes <tom@compton.nu>
Sun, 19 Jan 2014 13:40:11 +0000 (13:40 +0000)
The documentation says the limit is 10000 which seems like a
nicer round number anyway.

app/controllers/notes_controller.rb
test/functional/notes_controller_test.rb

index 1f9d10b0967204d75435870e6b7ed55916d57e1d..76c97ba5b3799770c3e3f55fe212efa426b489d3 100644 (file)
@@ -309,10 +309,10 @@ private
   # Get the maximum number of results to return
   def result_limit
     if params[:limit]
-      if params[:limit].to_i > 0 and params[:limit].to_i < 10000
+      if params[:limit].to_i > 0 and params[:limit].to_i <= 10000
         params[:limit].to_i
       else
-        raise OSM::APIBadUserInput.new("Note limit must be between 1 and 9999")
+        raise OSM::APIBadUserInput.new("Note limit must be between 1 and 10000")
       end
     else
       100
index 49c3cd0edb699af6cb46cab3882cd1af5bc8d30a..374c879d696af6fba53d9830fb52a3cdacc7292d 100644 (file)
@@ -632,7 +632,7 @@ class NotesControllerTest < ActionController::TestCase
     get :index, {:bbox => '1,1,1.7,1.7', :limit => '0', :format => 'json'}
     assert_response :bad_request
 
-    get :index, {:bbox => '1,1,1.7,1.7', :limit => '10000', :format => 'json'}
+    get :index, {:bbox => '1,1,1.7,1.7', :limit => '10001', :format => 'json'}
     assert_response :bad_request
   end
 
@@ -709,7 +709,7 @@ class NotesControllerTest < ActionController::TestCase
     get :search, {:q => 'no match', :limit => '0', :format => 'json'}
     assert_response :bad_request
 
-    get :search, {:q => 'no match', :limit => '10000', :format => 'json'}
+    get :search, {:q => 'no match', :limit => '10001', :format => 'json'}
     assert_response :bad_request
   end
 
@@ -743,7 +743,7 @@ class NotesControllerTest < ActionController::TestCase
     get :feed, {:bbox => "1,1,1.2,1.2", :limit => '0', :format => "rss"}
     assert_response :bad_request
 
-    get :feed, {:bbox => "1,1,1.2,1.2", :limit => '10000', :format => "rss"}
+    get :feed, {:bbox => "1,1,1.2,1.2", :limit => '10001', :format => "rss"}
     assert_response :bad_request
   end