Remove unused parameter
authorJohn Firebaugh <john.firebaugh@gmail.com>
Sun, 25 Nov 2012 19:14:47 +0000 (15:14 -0400)
committerJohn Firebaugh <john.firebaugh@gmail.com>
Sun, 25 Nov 2012 19:14:47 +0000 (15:14 -0400)
app/controllers/api_controller.rb
app/controllers/relation_controller.rb
app/controllers/search_controller.rb
app/models/relation.rb

index 602b79d..455c054 100644 (file)
@@ -203,7 +203,7 @@ class ApiController < ApplicationController
     # this "uniq" may be slightly inefficient; it may be better to first collect and output
     # all node-related relations, then find the *not yet covered* way-related ones etc.
     relations.uniq.each do |relation|
-      doc.root << relation.to_xml_node(nil, changeset_cache, user_display_name_cache)
+      doc.root << relation.to_xml_node(changeset_cache, user_display_name_cache)
     end
 
     response.headers["Content-Disposition"] = "attachment; filename=\"map.osm\""
index 0b989d1..af3c075 100644 (file)
@@ -97,7 +97,6 @@ class RelationController < ApplicationController
       # create XML.
       doc = OSM::API.new.get_xml_doc
       visible_nodes = {}
-      visible_members = { "Node" => {}, "Way" => {}, "Relation" => {} }
       changeset_cache = {}
       user_display_name_cache = {}
       
@@ -105,23 +104,20 @@ class RelationController < ApplicationController
         if node.visible? # should be unnecessary if data is consistent.
           doc.root << node.to_xml_node(changeset_cache, user_display_name_cache)
           visible_nodes[node.id] = node
-          visible_members["Node"][node.id] = true
         end
       end
       ways.each do |way|
         if way.visible? # should be unnecessary if data is consistent.
           doc.root << way.to_xml_node(visible_nodes, changeset_cache, user_display_name_cache)
-          visible_members["Way"][way.id] = true
         end
       end
       relations.each do |rel|
         if rel.visible? # should be unnecessary if data is consistent.
-          doc.root << rel.to_xml_node(nil, changeset_cache, user_display_name_cache)
-          visible_members["Relation"][rel.id] = true
+          doc.root << rel.to_xml_node(changeset_cache, user_display_name_cache)
         end
       end
       # finally add self and output
-      doc.root << relation.to_xml_node(visible_members, changeset_cache, user_display_name_cache)
+      doc.root << relation.to_xml_node(changeset_cache, user_display_name_cache)
       render :text => doc.to_s, :content_type => "text/xml"
       
     else
index 622ebb1..78becc2 100644 (file)
@@ -90,7 +90,7 @@ class SearchController < ApplicationController
     end
 
     relations.each do |rel|
-      doc.root << rel.to_xml_node(nil, changeset_cache, user_display_name_cache)
+      doc.root << rel.to_xml_node(changeset_cache, user_display_name_cache)
     end
 
     render :text => doc.to_s, :content_type => "text/xml"
index 028d602..abe0143 100644 (file)
@@ -105,7 +105,7 @@ class Relation < ActiveRecord::Base
     return doc
   end
 
-  def to_xml_node(visible_members = nil, changeset_cache = {}, user_display_name_cache = {})
+  def to_xml_node(changeset_cache = {}, user_display_name_cache = {})
     el1 = XML::Node.new 'relation'
     el1['id'] = self.id.to_s
     el1['visible'] = self.visible.to_s