Remove unused POST method for message replies

This action just shows the pre-filled form, with the target for
the form being the same as for new messages.

diff --git a/config/routes.rb b/config/routes.rb
index ed5e6876be3138b45c74b8640e9f6f05ed9befb5..44996fd8f8f81c9609610c9eea35b18ee1a40c0c 100644 (file)
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -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...)

diff --git a/test/controllers/messages_controller_test.rb b/test/controllers/messages_controller_test.rb
index dd9df8f451db8fb3c666f845b64f34e3ca615398..d9a1d9b0caf634b072468663bdc20a9ac5bb9c32 100644 (file)
--- a/test/controllers/messages_controller_test.rb
+++ b/test/controllers/messages_controller_test.rb
@@ -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" }