Use POST when editing GPX trace details

Fixes #1533
openstreetmap · May 1, 2017 · 3a16b68 · 3a16b68
1 parent 3039271
commit 3a16b68
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/app/views/trace/edit.html.erb b/app/views/trace/edit.html.erb
@@ -4,7 +4,7 @@
 
 <img src="<%= url_for :controller => 'trace', :action => 'picture', :id => @trace.id, :display_name => @trace.user.display_name %>">
 
-<%= form_for @trace, :url => { :action => "edit" } do |f| %>
+<%= form_for @trace, :method => :post, :url => { :action => "edit" } do |f| %>
 
 <div id='edit-trace-form' class='standard-form'>
   <fieldset>

diff --git a/config/routes.rb b/config/routes.rb
@@ -208,7 +208,7 @@
   match "/traces/mine" => "trace#mine", :via => :get
   match "/trace/create" => "trace#create", :via => [:get, :post]
   match "/trace/:id/data" => "trace#data", :via => :get, :id => /\d+/, :as => "trace_data"
-  match "/trace/:id/edit" => "trace#edit", :via => [:get, :post, :patch], :id => /\d+/, :as => "trace_edit"
+  match "/trace/:id/edit" => "trace#edit", :via => [:get, :post], :id => /\d+/, :as => "trace_edit"
   match "/trace/:id/delete" => "trace#delete", :via => :post, :id => /\d+/
 
   # diary pages

diff --git a/test/controllers/trace_controller_test.rb b/test/controllers/trace_controller_test.rb
@@ -157,10 +157,6 @@ def test_routes
       { :path => "/trace/1/edit", :method => :post },
       { :controller => "trace", :action => "edit", :id => "1" }
     )
-    assert_routing(
-      { :path => "/trace/1/edit", :method => :patch },
-      { :controller => "trace", :action => "edit", :id => "1" }
-    )
     assert_routing(
       { :path => "/trace/1/delete", :method => :post },
       { :controller => "trace", :action => "delete", :id => "1" }