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

Use resourceful route for new message

Browse source
This commit is contained in:
Anton Khorev 2025-01-02 10:28:03 +03:00
parent d38f6484e1
commit d57033fa87
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
config/routes.rb
View file

@ -311,7 +311,7 @@ OpenStreetMap::Application.routes.draw do
get "/export/embed" => "export#embed"
# messages
resources :messages, :id => /\d+/, :only => [:create, :show, :destroy] do
resources :messages, :path_names => { :new => "new/:display_name" }, :id => /\d+/, :only => [:new, :create, :show, :destroy] do
post :mark
patch :unmute
@ -324,7 +324,7 @@ OpenStreetMap::Application.routes.draw do
end
get "/user/:display_name/inbox", :to => redirect(:path => "/messages/inbox")
get "/user/:display_name/outbox", :to => redirect(:path => "/messages/outbox")
get "/message/new/:display_name" => "messages#new", :as => "new_message"
get "/message/new/:display_name", :to => redirect(:path => "/messages/new/%{display_name}")
get "/message/read/:message_id", :to => redirect(:path => "/messages/%{message_id}")
# muting users

2
test/controllers/messages_controller_test.rb
View file

@ -5,7 +5,7 @@ class MessagesControllerTest < ActionDispatch::IntegrationTest
# test all routes which lead to this controller
def test_routes
assert_routing(
{ :path => "/message/new/username", :method => :get },
{ :path => "/messages/new/username", :method => :get },
{ :controller => "messages", :action => "new", :display_name => "username" }
)
assert_routing(