has_many :old_segments, :foreign_key => :id
belongs_to :user
- has_one :from_node, :class => 'Node', :foreign_key => 'node_a'
- has_one :to_node, :class => 'Node', :foreign_key => 'node_b'
+ # using belongs_to :foreign_key = 'node_*', since if use has_one :foreign_key = 'id', segment preconditions? fails checking for segment id in node table
+ belongs_to :from_node, :class_name => 'Node', :foreign_key => 'node_a'
+ belongs_to :to_node, :class_name => 'Node', :foreign_key => 'node_b'
def self.from_xml(xml, create=false)
p = XML::Parser.new
def save_with_history
begin
Segment.transaction do
+ self.timestamp = Time.now
self.save
old_segment = OldSegment.from_segment(self)
old_segment.save
end
def to_xml
- doc = XML::Document.new
- doc.encoding = 'UTF-8'
- root = XML::Node.new 'osm'
- root['version'] = API_VERSION
- root['generator'] = 'OpenStreetMap server'
- doc.root = root
- root << to_xml_node()
+ doc = OSM::API.new.get_xml_doc
+ doc.root << to_xml_node()
return doc
end
- def to_xml_node
+ def to_xml_node(user_display_name_cache = nil)
el1 = XML::Node.new 'segment'
el1['id'] = self.id.to_s
el1['from'] = self.node_a.to_s
el1['to'] = self.node_b.to_s
+ if user_display_name_cache and user_display_name_cache[self.user_id]
+ # use the cache if available
+ else
+ user_display_name_cache[self.user_id] = self.user.display_name
+ end
+
+ #el1['user'] = self.user.display_name if self.user.data_public?
+ el1['user'] = user_display_name_cache[self.user_id]
+
Segment.split_tags(el1, self.tags)
el1['visible'] = self.visible.to_s
el1['timestamp'] = self.timestamp.xmlschema
end
end
- def precondtions_ok?
- return from_node and from_node.visible and to_node and to_node.visible
+ def preconditions_ok?
+ from_node and from_node.visible and to_node and to_node.visible
end
end