From: Andy Allan <git@gravitystorm.co.uk>
Date: Wed, 26 Jun 2024 13:17:39 +0000 (+0100)
Subject: Merge pull request #4610 from AntonKhorev/no-old-note-api
X-Git-Tag: live~924
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/7f3cb3c6246e6e271761e714cceb081e85a4d245?hp=9702bb997ee429adf69e6eea45a9ebf2bc326ebf

Merge pull request #4610 from AntonKhorev/no-old-note-api

Remove old note api endpoints
---

diff --git a/config/routes.rb b/config/routes.rb
index acf2256a3..98c8ba15e 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -99,12 +99,6 @@ OpenStreetMap::Application.routes.draw do
       end
     end
 
-    post "notes/addPOIexec" => "notes#create"
-    post "notes/closePOIexec" => "notes#close"
-    post "notes/editPOIexec" => "notes#comment"
-    get "notes/getGPX" => "notes#index", :format => "gpx"
-    get "notes/getRSSfeed" => "notes#feed", :format => "rss"
-
     resources :user_blocks, :only => [:show], :constraints => { :id => /\d+/ }, :controller => "user_blocks", :as => :api_user_blocks
   end
 
diff --git a/test/controllers/api/notes_controller_test.rb b/test/controllers/api/notes_controller_test.rb
index 045e3bf43..42972e494 100644
--- a/test/controllers/api/notes_controller_test.rb
+++ b/test/controllers/api/notes_controller_test.rb
@@ -99,27 +99,6 @@ module Api
         { :path => "/api/0.6/notes/feed", :method => :get },
         { :controller => "api/notes", :action => "feed", :format => "rss" }
       )
-
-      assert_recognizes(
-        { :controller => "api/notes", :action => "create" },
-        { :path => "/api/0.6/notes/addPOIexec", :method => :post }
-      )
-      assert_recognizes(
-        { :controller => "api/notes", :action => "close" },
-        { :path => "/api/0.6/notes/closePOIexec", :method => :post }
-      )
-      assert_recognizes(
-        { :controller => "api/notes", :action => "comment" },
-        { :path => "/api/0.6/notes/editPOIexec", :method => :post }
-      )
-      assert_recognizes(
-        { :controller => "api/notes", :action => "index", :format => "gpx" },
-        { :path => "/api/0.6/notes/getGPX", :method => :get }
-      )
-      assert_recognizes(
-        { :controller => "api/notes", :action => "feed", :format => "rss" },
-        { :path => "/api/0.6/notes/getRSSfeed", :method => :get }
-      )
     end
 
     def test_create_success