additional consistency checks. making the error message for the number of nodes use...

[rails.git] / app / models / changeset.rb

diff --git a/app/models/changeset.rb b/app/models/changeset.rb
index 9b92c62ead7547a6b11def8998abf608e344b793..caa167c9eb19e52b32e277269820b056f918e962 100644 (file)
--- a/app/models/changeset.rb
+++ b/app/models/changeset.rb
@@ -4,6 +4,23 @@ class Changeset < ActiveRecord::Base
   belongs_to :user
 
   has_many :changeset_tags, :foreign_key => 'id'
+  
+  has_many :nodes
+  has_many :ways
+  has_many :relations
+  has_many :old_nodes
+  has_many :old_ways
+  has_many :old_relations
+  
+  validates_presence_of :user_id, :created_at, :open
+  validates_inclusion_of :open, :in => [ true, false ]
+  
+  # Use a method like this, so that we can easily change how we
+  # determine whether a changeset is open, without breaking code in at 
+  # least 6 controllers
+  def is_open?
+    return open
+  end
 
   def self.from_xml(xml, create=false)
     begin
@@ -54,6 +71,7 @@ class Changeset < ActiveRecord::Base
 
     Changeset.transaction do
       # fixme update modified_at time?
+      # FIXME there is no modified_at time, should it be added
       self.save!
     end
 
@@ -76,6 +94,7 @@ class Changeset < ActiveRecord::Base
     doc.root << to_xml_node()
     return doc
   end
+  
   def to_xml_node(user_display_name_cache = nil)
     el1 = XML::Node.new 'changeset'
     el1['id'] = self.id.to_s
@@ -100,6 +119,7 @@ class Changeset < ActiveRecord::Base
     end
     
     el1['created_at'] = self.created_at.xmlschema
+    el1['open'] = self.open.to_s
 
     # FIXME FIXME FIXME: This does not include changes yet! There is 
     # currently no changeset_id column in the tables as far as I can tell,