cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove unused POST method for message replies

Browse source 
This action just shows the pre-filled form, with the target for
the form being the same as for new messages.
This commit is contained in:
Andy Allan 2018-05-15 17:33:40 +08:00
parent b204b9dc64
commit 40cab845fb
2 changed files with 1 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
config/routes.rb
View 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...)

4
test/controllers/messages_controller_test.rb
View 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" }