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

Remove unused visible_nodes hashes from api element controllers

Browse source
This commit is contained in:
Anton Khorev 2025-02-01 19:35:46 +03:00
parent b8924ed702
commit 8508c4f0a7
2 changed files with 1 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

3
app/controllers/api/relations_controller.rb
View file

@ -102,14 +102,11 @@ module Api
node_ids += way_node_ids.flatten node_ids += way_node_ids.flatten
nodes = Node.where(:id => node_ids.uniq).includes(:node_tags) nodes = Node.where(:id => node_ids.uniq).includes(:node_tags)
visible_nodes = {}
@nodes = [] @nodes = []
nodes.each do |node| nodes.each do |node|
next unless node.visible? # should be unnecessary if data is consistent. next unless node.visible? # should be unnecessary if data is consistent.
@nodes << node @nodes << node
visible_nodes[node.id] = node
end end
@ways = [] @ways = []

7
app/controllers/api/ways_controller.rb
View file

@ -76,15 +76,10 @@ module Api
@way = Way.includes(:nodes => :node_tags).find(params[:id]) @way = Way.includes(:nodes => :node_tags).find(params[:id])
if @way.visible if @way.visible
visible_nodes = {}
@nodes = [] @nodes = []
@way.nodes.uniq.each do |node| @way.nodes.uniq.each do |node|
if node.visible @nodes << node if node.visible
@nodes << node
visible_nodes[node.id] = node
end
end end
# Render the result # Render the result