]> git.openstreetmap.org Git - rails.git/commitdiff
Restore constraints on note id parameter
authorAnton Khorev <tony29@yandex.ru>
Wed, 1 May 2024 10:17:14 +0000 (13:17 +0300)
committerAnton Khorev <tony29@yandex.ru>
Wed, 1 May 2024 10:17:14 +0000 (13:17 +0300)
config/routes.rb
test/controllers/notes_controller_test.rb

index d1e4c74ae5e06c0d00736051520e1031bdd96411..8271e7e4dba1b95f05b4027af56a5d5cc0aac605 100644 (file)
@@ -122,7 +122,7 @@ OpenStreetMap::Application.routes.draw do
     match :subscribe, :unsubscribe, :on => :member, :via => [:get, :post]
   end
   get "/changeset/:id/comments/feed" => "changeset_comments#index", :as => :changeset_comments_feed, :id => /\d*/, :defaults => { :format => "rss" }
-  resources :notes, :path => "note", :only => [:show, :new]
+  resources :notes, :path => "note", :id => /\d+/, :only => [:show, :new]
 
   get "/user/:display_name/history" => "changesets#index"
   get "/user/:display_name/history/feed" => "changesets#feed", :defaults => { :format => :atom }
index e68a5f33beec2f4eb3c44a2d76da25c9f88788ec..ff1071897f79d197ba24cc8a16a40cec8b754fac 100644 (file)
@@ -179,9 +179,9 @@ class NotesControllerTest < ActionDispatch::IntegrationTest
       get path_method.call
     end
 
-    assert_raise ActionController::UrlGenerationError do
-      get path_method.call(:id => -10) # we won't have an id that's negative
-    end
+    assert_raise ActionController::UrlGenerationError do
+      get path_method.call(:id => -10) # we won't have an id that's negative
+    end
 
     get path_method.call(:id => 0)
     assert_response :not_found