]> git.openstreetmap.org Git - rails.git/commitdiff
Merge changes from trunk 7673:8632.
authorShaun McDonald <shaun@shaunmcdonald.me.uk>
Thu, 3 Jul 2008 13:06:24 +0000 (13:06 +0000)
committerShaun McDonald <shaun@shaunmcdonald.me.uk>
Thu, 3 Jul 2008 13:06:24 +0000 (13:06 +0000)
1  2 
app/controllers/api_controller.rb
app/controllers/application.rb
app/models/old_relation.rb
app/models/old_way.rb
app/models/relation.rb
app/models/way.rb
config/environment.rb
config/routes.rb

Simple merge
Simple merge
Simple merge
Simple merge
index 984732c71c98f6b0bc9c1463029135fb2345f64a,9ee118f6e0c22fb26f4bff0012e30e3cb2c92173..eb3b06a130bc1a22c0f27170aabb0d2dc21c6c0b
@@@ -205,37 -181,19 +185,49 @@@ class Relation < ActiveRecord::Bas
      end
    end
  
 +  def delete_with_history(user)
 +    if self.visible
 +      if RelationMember.find(:first, :joins => "INNER JOIN current_relations ON current_relations.id=current_relation_members.id", :conditions => [ "visible = 1 AND member_type='relation' and member_id=?", self.id ])
 +      raise OSM::APIPreconditionFailedError.new
 +      else
 +      self.user_id = user.id
 +      self.tags = []
 +      self.members = []
 +      self.visible = false
 +      save_with_history!
 +      end
 +    else
 +      raise OSM::APIAlreadyDeletedError.new
 +    end
 +  end
 +
 +  def update_from(new_relation, user)
 +    if !new_relation.preconditions_ok?
 +      raise OSM::APIPreconditionFailedError.new
 +    elsif new_relation.version != version
 +      raise OSM::APIVersionMismatchError.new(new_relation.version, version)
 +    else
 +      self.user_id = user.id
 +      self.tags = new_relation.tags
 +      self.members = new_relation.members
 +      self.visible = true
 +      save_with_history!
 +    end
 +  end
 +
    def preconditions_ok?
+     # These are hastables that store an id in the index of all 
+     # the nodes/way/relations that have already been added.
+     # Once we know the id of the node/way/relation exists
+     # we check to see if it is already existing in the hashtable
+     # if it does, then we return false. Otherwise
+     # we add it to the relevant hash table, with the value true..
+     # Thus if you have nodes with the ids of 50 and 1 already in the
+     # relation, then the hash table nodes would contain:
+     # => {50=>true, 1=>true}
+     nodes = Hash.new
+     ways = Hash.new
+     relations = Hash.new
      self.members.each do |m|
        if (m[0] == "node")
          n = Node.find(:first, :conditions => ["id = ?", m[1]])
index ea027fb47908d2148ae7d07476843972aeeebd86,64b11cf672aabebe946e5473a7da601a500d7201..34afc6585041e7fa3c723dc08bde40a8b24af83c
@@@ -267,7 -254,12 +271,12 @@@ class Way < ActiveRecord::Bas
      
      self.user_id = user.id
  
 -    self.delete_with_relations_and_history(user)
 +    self.delete_with_history(user)
  
    end
+   # Temporary method to match interface to nodes
+   def tags_as_hash
+     return self.tags
+   end
  end
index 8aaab730289311ff083812694406de99cf4ec8ce,495f94d80a63f08b22f639bdcbaaace8c6d946e0..fb7573d2a5b13b7177c6381435f77e25c98ec3db
@@@ -12,11 -11,16 +11,16 @@@ RAILS_GEM_VERSION = '2.0.2' unless defi
  SERVER_URL = ENV['OSM_SERVER_URL'] || 'www.openstreetmap.org'
  
  # Application constants needed for routes.rb - must go before Initializer call
 -API_VERSION = ENV['OSM_API_VERSION'] || '0.5'
 +API_VERSION = ENV['OSM_API_VERSION'] || '0.6'
  
- # Set to :readonly to put the API in read-only mode or :offline to
- # take it completely offline
- API_STATUS = :online
+ # Set application status - possible settings are:
+ #
+ #   :online - online and operating normally
+ #   :api_readonly - site online but API in read-only mode
+ #   :api_offline - site online but API offline
+ #   :database_offline - database offline with site in emergency mode
+ #
+ OSM_STATUS = :online
  
  # Bootstrap the Rails environment, frameworks, and default configuration
  require File.join(File.dirname(__FILE__), 'boot')
Simple merge