From cca07cf0b54ca16ec7bafcfa6d3920b6a7c087e4 Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Wed, 5 Mar 2025 20:12:39 +0300
Subject: [PATCH] Rename browse controller to feature queries controller

---
 app/abilities/ability.rb                                  | 2 +-
 ...browse_controller.rb => feature_queries_controller.rb} | 6 +++---
 .../query.html.erb => feature_queries/show.html.erb}      | 0
 config/locales/en.yml                                     | 3 ++-
 config/routes.rb                                          | 2 +-
 ...troller_test.rb => feature_queries_controller_test.rb} | 8 ++++----
 6 files changed, 11 insertions(+), 10 deletions(-)
 rename app/controllers/{browse_controller.rb => feature_queries_controller.rb} (68%)
 rename app/views/{browse/query.html.erb => feature_queries/show.html.erb} (100%)
 rename test/controllers/{browse_controller_test.rb => feature_queries_controller_test.rb} (65%)

diff --git a/app/abilities/ability.rb b/app/abilities/ability.rb
index 7f47b578c..bd3ea28bb 100644
--- a/app/abilities/ability.rb
+++ b/app/abilities/ability.rb
@@ -4,7 +4,7 @@ class Ability
   include CanCan::Ability
 
   def initialize(user)
-    can :query, :browse
+    can :read, :feature_query
     can :read, [Node, Way, Relation, OldNode, OldWay, OldRelation]
     can [:show, :create], Note
     can :search, :direction
diff --git a/app/controllers/browse_controller.rb b/app/controllers/feature_queries_controller.rb
similarity index 68%
rename from app/controllers/browse_controller.rb
rename to app/controllers/feature_queries_controller.rb
index a3e65a1b7..f0f238c7a 100644
--- a/app/controllers/browse_controller.rb
+++ b/app/controllers/feature_queries_controller.rb
@@ -1,13 +1,13 @@
-class BrowseController < ApplicationController
+class FeatureQueriesController < ApplicationController
   layout :map_layout
 
   before_action :authorize_web
   before_action :set_locale
   before_action -> { check_database_readable(:need_api => true) }
   before_action :require_oauth
-  before_action :update_totp, :only => [:query]
+  before_action :update_totp
   around_action :web_timeout
   authorize_resource :class => false
 
-  def query; end
+  def show; end
 end
diff --git a/app/views/browse/query.html.erb b/app/views/feature_queries/show.html.erb
similarity index 100%
rename from app/views/browse/query.html.erb
rename to app/views/feature_queries/show.html.erb
diff --git a/config/locales/en.yml b/config/locales/en.yml
index b56fb6410..8b60b094c 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -417,7 +417,8 @@ en:
       telephone_link: "Call %{phone_number}"
       colour_preview: "Colour %{colour_value} preview"
       email_link: "Email %{email}"
-    query:
+  feature_queries:
+    show:
       title: "Query Features"
       introduction: "Click on the map to find nearby features."
       nearby: "Nearby features"
diff --git a/config/routes.rb b/config/routes.rb
index 3f4a12bd7..8d42db69b 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -204,7 +204,7 @@ OpenStreetMap::Application.routes.draw do
   get "/offline" => "site#offline"
   get "/key" => "site#key"
   get "/id" => "site#id"
-  get "/query" => "browse#query"
+  resource :feature_query, :path => "query", :only => :show
   post "/user/:display_name/confirm/resend" => "confirmations#confirm_resend", :as => :user_confirm_resend
   match "/user/:display_name/confirm" => "confirmations#confirm", :via => [:get, :post]
   match "/user/confirm" => "confirmations#confirm", :via => [:get, :post]
diff --git a/test/controllers/browse_controller_test.rb b/test/controllers/feature_queries_controller_test.rb
similarity index 65%
rename from test/controllers/browse_controller_test.rb
rename to test/controllers/feature_queries_controller_test.rb
index 1844dcc6c..ad2b5ad4d 100644
--- a/test/controllers/browse_controller_test.rb
+++ b/test/controllers/feature_queries_controller_test.rb
@@ -6,13 +6,13 @@ class BrowseControllerTest < ActionDispatch::IntegrationTest
   def test_routes
     assert_routing(
       { :path => "/query", :method => :get },
-      { :controller => "browse", :action => "query" }
+      { :controller => "feature_queries", :action => "show" }
     )
   end
 
-  def test_query
-    get query_path
+  def test_show
+    get feature_query_path
     assert_response :success
-    assert_template "browse/query"
+    assert_template "feature_queries/show"
   end
 end
-- 
2.39.5