Remove unused POST method for message replies
authorAndy Allan <git@gravitystorm.co.uk>
Tue, 15 May 2018 09:33:40 +0000 (17:33 +0800)
committerAndy Allan <git@gravitystorm.co.uk>
Tue, 15 May 2018 09:33:40 +0000 (17:33 +0800)
This action just shows the pre-filled form, with the target for
the form being the same as for new messages.

config/routes.rb
test/controllers/messages_controller_test.rb

index ed5e687..44996fd 100644 (file)
@@ -266,7 +266,7 @@ OpenStreetMap::Application.routes.draw do
   match "/message/new/:display_name" => "messages#new", :via => [:get, :post], :as => "new_message"
   get "/message/read/:message_id" => "messages#read", :as => "read_message"
   post "/message/mark/:message_id" => "messages#mark", :as => "mark_message"
-  match "/message/reply/:message_id" => "messages#reply", :via => [:get, :post], :as => "reply_message"
+  get "/message/reply/:message_id" => "messages#reply", :as => "reply_message"
   post "/message/delete/:message_id" => "messages#delete", :as => "delete_message"
 
   # oauth admin pages (i.e: for setting up new clients, etc...)
index dd9df8f..d9a1d9b 100644 (file)
@@ -32,10 +32,6 @@ class MessagesControllerTest < ActionController::TestCase
       { :path => "/message/reply/1", :method => :get },
       { :controller => "messages", :action => "reply", :message_id => "1" }
     )
-    assert_routing(
-      { :path => "/message/reply/1", :method => :post },
-      { :controller => "messages", :action => "reply", :message_id => "1" }
-    )
     assert_routing(
       { :path => "/message/delete/1", :method => :post },
       { :controller => "messages", :action => "delete", :message_id => "1" }