- def import
- begin
- logger.info("GPX Import importing #{name} (#{id}) from #{user.email}")
-
- # TODO *nix specific, could do to work on windows... would be functionally inferior though - check for '.gz'
- filetype = `file -bz #{trace_name}`.chomp
- gzipped = filetype =~ /gzip compressed/
- bzipped = filetype =~ /bzip2 compressed/
- zipped = filetype =~ /Zip archive/
- tarred = filetype =~ /tar archive/
-
- if tarred and gzipped then
- filename = tempfile = "/tmp/#{rand}"
- system("tar -zxOf #{trace_name} > #{filename}")
- elsif tarred and bzipped then
- filename = tempfile = "/tmp/#{rand}"
- system("tar -jxOf #{trace_name} > #{filename}")
+ def update_from_xml(xml, create = false)
+ p = XML::Parser.string(xml, :options => XML::Parser::Options::NOERROR)
+ doc = p.parse
+
+ doc.find("//osm/gpx_file").each do |pt|
+ return update_from_xml_node(pt, create)
+ end
+
+ raise OSM::APIBadXMLError.new("trace", xml, "XML doesn't contain an osm/gpx_file element.")
+ rescue LibXML::XML::Error, ArgumentError => ex
+ raise OSM::APIBadXMLError.new("trace", xml, ex.message)
+ end
+
+ def update_from_xml_node(pt, create = false)
+ raise OSM::APIBadXMLError.new("trace", pt, "visibility missing") if pt["visibility"].nil?
+ self.visibility = pt["visibility"]
+
+ unless create
+ raise OSM::APIBadXMLError.new("trace", pt, "ID is required when updating.") if pt["id"].nil?
+ id = pt["id"].to_i
+ # .to_i will return 0 if there is no number that can be parsed.
+ # We want to make sure that there is no id with zero anyway
+ raise OSM::APIBadUserInput, "ID of trace cannot be zero when updating." if id.zero?
+ raise OSM::APIBadUserInput, "The id in the url (#{self.id}) is not the same as provided in the xml (#{id})" unless self.id == id
+ end
+
+ # We don't care about the time, as it is explicitly set on create/update/delete
+ # We don't care about the visibility as it is implicit based on the action
+ # and set manually before the actual delete
+ self.visible = true
+
+ description = pt.find("description").first
+ raise OSM::APIBadXMLError.new("trace", pt, "description missing") if description.nil?
+ self.description = description.content
+
+ self.tags = pt.find("tag").collect do |tag|
+ Tracetag.new(:tag => tag.content)
+ end
+ end
+
+ def xml_file
+ # TODO: *nix specific, could do to work on windows... would be functionally inferior though - check for '.gz'
+ filetype = `/usr/bin/file -Lbz #{trace_name}`.chomp
+ gzipped = filetype =~ /gzip compressed/
+ bzipped = filetype =~ /bzip2 compressed/
+ zipped = filetype =~ /Zip archive/
+ tarred = filetype =~ /tar archive/
+
+ if gzipped || bzipped || zipped || tarred
+ tmpfile = Tempfile.new("trace.#{id}")
+
+ if tarred && gzipped
+ system("tar -zxOf #{trace_name} > #{tmpfile.path}")
+ elsif tarred && bzipped
+ system("tar -jxOf #{trace_name} > #{tmpfile.path}")