this is kind of a "suggested change" for the API controller to

support an additional "restriction" parameter for the "map"
call (only value currently supported: nodes_only to return only
nodes). i would have liked to implement "tagged_nodes_only" as
well but was unsure how to do that best. feel free to make up
some other way instead to improve node access for POI editors,
or drop this altogether if you have some other grand plan.

diff --git a/app/controllers/api_controller.rb b/app/controllers/api_controller.rb
index 3e9e627db7fe02c2b7f5777f32f1429b13e9ed1d..f84ae10764fa485fd4b05d004281ef9d5aa53a56 100644 (file)
--- a/app/controllers/api_controller.rb
+++ b/app/controllers/api_controller.rb
@@ -144,6 +144,11 @@ class ApiController < ApplicationController
       return
     end
 
       return
     end
 

+    if param[:restriction] and param[:restriction] != "nodes_only"
+      report_error("The parameter restriction may only take one of the following values: nodes_only")
+      return
+    end
+

     # FIXME um why is this area using a different order for the lat/lon from above???
     @nodes = Node.find_by_area(min_lat, min_lon, max_lat, max_lon, :conditions => {:visible => true}, :include => :node_tags, :limit => APP_CONFIG['max_number_of_nodes']+1)
     # get all the nodes, by tag not yet working, waiting for change from NickB
     # FIXME um why is this area using a different order for the lat/lon from above???
     @nodes = Node.find_by_area(min_lat, min_lon, max_lat, max_lon, :conditions => {:visible => true}, :include => :node_tags, :limit => APP_CONFIG['max_number_of_nodes']+1)
     # get all the nodes, by tag not yet working, waiting for change from NickB


@@ -170,6 +175,14 @@ class ApiController < ApplicationController
     bounds['maxlon'] = max_lon.to_s
     doc.root << bounds
 
     bounds['maxlon'] = max_lon.to_s
     doc.root << bounds
 

+    # bail out at this stage if user has indicated that he is 
+    # not interested in ways or relations.
+    if params[:restriction] == "nodes_only"
+        response.headers["Content-Disposition"] = "attachment; filename=\"map.osm\""
+        render :text => doc.to_s, :content_type => "text/xml"
+        return
+    end
+

     # get ways
     # find which ways are needed
     ways = Array.new
     # get ways
     # find which ways are needed
     ways = Array.new