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

Don't show 'Download XML' link for deleted elements

Browse source
This commit is contained in:
Anton Khorev 2022-09-22 14:51:31 +03:00
parent 09263bc4a1
commit 01d48694b4
2 changed files with 13 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
app/views/browse/feature.html.erb
View file

@ -5,7 +5,9 @@
<%= render :partial => @type, :object => @feature %>
<div class='secondary-actions'>
<%= link_to(t("browse.download_xml"), :controller => "api/#{@type.pluralize}", :action => :show) %>
&middot;
<% if @feature.visible? %>
<%= link_to(t("browse.download_xml"), :controller => "api/#{@type.pluralize}", :action => :show) %>
&middot;
<% end %>
<%= link_to(t("browse.view_history"), :action => "#{@type}_history") %>
</div>

10
test/controllers/browse_controller_test.rb
View file

@ -63,7 +63,15 @@ class BrowseControllerTest < ActionDispatch::IntegrationTest
end
def test_read_node
browse_check :node_path, create(:node).id, "browse/feature"
node = create :node
browse_check :node_path, node.id, "browse/feature"
assert_select "a[href='#{api_node_path node}']", :count => 1
end
def test_read_deleted_node
node = create :node, :visible => false
browse_check :node_path, node.id, "browse/feature"
assert_select "a[href='#{api_node_path node}']", :count => 0
end
def test_read_node_history