From 8508c4f0a77b92b1378ee1204e8e352362cde8cf Mon Sep 17 00:00:00 2001 From: Anton Khorev Date: Sat, 1 Feb 2025 19:35:46 +0300 Subject: [PATCH] Remove unused visible_nodes hashes from api element controllers --- app/controllers/api/relations_controller.rb | 3 --- app/controllers/api/ways_controller.rb | 7 +------ 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/app/controllers/api/relations_controller.rb b/app/controllers/api/relations_controller.rb index a9b85de7d..ae101f373 100644 --- a/app/controllers/api/relations_controller.rb +++ b/app/controllers/api/relations_controller.rb @@ -102,14 +102,11 @@ module Api node_ids += way_node_ids.flatten nodes = Node.where(:id => node_ids.uniq).includes(:node_tags) - visible_nodes = {} - @nodes = [] nodes.each do |node| next unless node.visible? # should be unnecessary if data is consistent. @nodes << node - visible_nodes[node.id] = node end @ways = [] diff --git a/app/controllers/api/ways_controller.rb b/app/controllers/api/ways_controller.rb index 27c4a1fcc..632fdb9a6 100644 --- a/app/controllers/api/ways_controller.rb +++ b/app/controllers/api/ways_controller.rb @@ -76,15 +76,10 @@ module Api @way = Way.includes(:nodes => :node_tags).find(params[:id]) if @way.visible - visible_nodes = {} - @nodes = [] @way.nodes.uniq.each do |node| - if node.visible - @nodes << node - visible_nodes[node.id] = node - end + @nodes << node if node.visible end # Render the result -- 2.39.5