]> git.openstreetmap.org Git - rails.git/commitdiff
Don't link to self from element version pages
authorAnton Khorev <tony29@yandex.ru>
Wed, 17 Jan 2024 00:34:42 +0000 (03:34 +0300)
committerAnton Khorev <tony29@yandex.ru>
Tue, 23 Jan 2024 19:08:39 +0000 (22:08 +0300)
app/views/browse/_common_details.html.erb
test/controllers/old_nodes_controller_test.rb
test/controllers/old_relations_controller_test.rb
test/controllers/old_ways_controller_test.rb

index fa8eeb3a5fef1da55fa28918c0678a637219b594..7d3f8e829c672d3bc0c05ebc1acaf178cfda4e73 100644 (file)
@@ -1,6 +1,6 @@
 <h4>
   <%= t "browse.version" %>
-  #<%= link_to common_details.version, :controller => "old_#{@type.pluralize}", :action => :show, :version => common_details.version %>
+  #<%= link_to_unless_current common_details.version, :controller => "old_#{@type.pluralize}", :action => :show, :version => common_details.version %>
 </h4>
 
 <p class="fst-italic">
index 80b8da946cfcf51f702587382af9cb851a790d5b..68a6c1f3efec417b2acbeddbfd7a77f8392a30f5 100644 (file)
@@ -14,6 +14,9 @@ class OldNodesControllerTest < ActionDispatch::IntegrationTest
     assert_response :success
     assert_template "old_nodes/show"
     assert_template :layout => "map"
+    assert_select "h4", /^Version/ do
+      assert_select "a[href='#{old_node_path node, 1}']", :count => 0
+    end
   end
 
   def test_not_found
index 87ee50db47e6c1d48657c6e953354d18573afffc..2e791f227f586d1c669c52121fc5da3192784415 100644 (file)
@@ -14,6 +14,9 @@ class OldRelationsControllerTest < ActionDispatch::IntegrationTest
     assert_response :success
     assert_template "old_relations/show"
     assert_template :layout => "map"
+    assert_select "h4", /^Version/ do
+      assert_select "a[href='#{old_relation_path relation, 1}']", :count => 0
+    end
   end
 
   def test_visible_with_members
index 897bdba1f98134dde887f9b3fc1d4266754801cc..022e4e8207ea10f0fa4625f94b13529afc2dc28c 100644 (file)
@@ -14,6 +14,9 @@ class OldWaysControllerTest < ActionDispatch::IntegrationTest
     assert_response :success
     assert_template "old_ways/show"
     assert_template :layout => "map"
+    assert_select "h4", /^Version/ do
+      assert_select "a[href='#{old_way_path way, 1}']", :count => 0
+    end
   end
 
   def test_visible_with_shared_nodes