From: Xin Zheng <xin@zenpow.com>
Date: Fri, 25 Jan 2008 13:57:42 +0000 (+0000)
Subject: minor changes
X-Git-Tag: live~8516
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/e7087af6bc0d7db9ca3f3c21be76e9c8a9c343b4

minor changes
---

diff --git a/app/models/node.rb b/app/models/node.rb
index 2a073f542..37143ff00 100644
--- a/app/models/node.rb
+++ b/app/models/node.rb
@@ -27,6 +27,32 @@ class Node < GeoRecord
     return true
   end
 
+  #
+  # Search for nodes matching tags within bounding_box
+  #
+  # Also adheres to limitations such as within max_number_of_nodes
+  #
+  def self.search(bounding_box, tags = {})
+    min_lon, min_lat, max_lon, max_lat = *bounding_box
+    # @fixme a bit of a hack to search for only visible nodes
+    # couldn't think of another to add to tags condition
+    #conditions_hash = tags.merge({ 'visible' => 1 })
+  
+    # using named placeholders http://www.robbyonrails.com/articles/2005/10/21/using-named-placeholders-in-ruby
+    #keys = []
+    #values = {}
+
+    #conditions_hash.each do |key,value|
+    #  keys <<  "#{key} = :#{key}"
+    #  values[key.to_sym] = value
+    #end 
+    #conditions = keys.join(' AND ')
+ 
+    find_by_area(min_lat, min_lon, max_lat, max_lon,
+                    :conditions => 'visible = 1',
+                    :limit => APP_CONFIG['max_number_of_nodes']+1)  
+  end
+
   # Read in xml as text and return it's Node object representation
   def self.from_xml(xml, create=false)
     begin