Allow users to load traces that aren't theirs
[potlatch2.git] / net / systemeD / halcyon / connection / Trace.as
index 17bb572..8a4199e 100644 (file)
@@ -8,7 +8,7 @@ package net.systemeD.halcyon.connection {
     import net.systemeD.halcyon.connection.*;
     import net.systemeD.halcyon.Map;
     import net.systemeD.halcyon.Globals;
-    import net.systemeD.halcyon.VectorLayer;
+    import net.systemeD.halcyon.MapPaint;
 
     /**
     * Implements trace objects loaded from the OSM API.
@@ -16,23 +16,32 @@ package net.systemeD.halcyon.connection {
     * loading GPX files (and other formats) from arbitrary urls.
     */
     public class Trace extends EventDispatcher {
-        private var _id:Number;
-        private var _description:String;
+        private var _id:Number; // id of the trace, as reported by the server
+        private var _description:String; // description, as reported by the server
         private var tags:Array = []; // N.B. trace tags are NOT k/v pairs
-        private var _isLoaded:Boolean;
-        private var _filename:String;
-        private var _traceData:String;
+        private var _isLoaded:Boolean; // Flag for when the data has been downloaded and parsed
+        private var _filename:String; // The original name of the file, as reported by the server
+        private var _traceData:String; // the trace data, saved as a string
         private var map:Map;
-        private var _layer:VectorLayer;
+        private var _layer:MapPaint;
+        private var masterConnection:Connection; // The authenticated connection
+        private var _connection:Connection; // The one we store our fake nodes/ways in.
         private var simplify:Boolean = false;
 
         private static const STYLESHEET:String="stylesheets/gpx.css";
 
-        public function Trace() {
-            map = Globals.vars.root;
+        /** Create a new trace.
+        * @param masterConnection The authenticated connection to communicate with the server
+        */
+        public function Trace(masterConnection:Connection, id:int=0) {
+            this.masterConnection = masterConnection;
+            map = Globals.vars.root; // REFACTOR this prevents traces being added to arbitrary maps
+                       if (id!=0) _id=id;
         }
 
-        /* Create a new trace, from the XML description given by the user/traces call */
+        /** Create a new trace, from the XML description given by the user/traces call.
+        * This only creates the object itself, the actual trace contents (trkseg etc) are
+        * lazily downloaded later. */
         public function fromXML(xml:XML):Trace {
             _id = Number(xml.@id);
             _filename = xml.@name;
@@ -61,7 +70,7 @@ package net.systemeD.halcyon.connection {
 
         private function fetchFromServer():void {
             // todo - needs proper error handling
-            Connection.getConnection().fetchTrace(id, saveTraceData);
+            masterConnection.fetchTrace(id, saveTraceData);
             dispatchEvent(new Event("loading_data"));
         }
 
@@ -70,10 +79,18 @@ package net.systemeD.halcyon.connection {
             dispatchEvent(new Event("loaded_data"));
         }
 
-        private function get layer():VectorLayer {
+        private function get connection():Connection {
+            if (!_connection) {
+                // create a new connection so that the ids don't impact the main layer.
+                _connection = new Connection(filename, null, null, null);
+            }
+            return _connection
+        }
+
+        private function get layer():MapPaint {
             if (!_layer) {
-                _layer=new VectorLayer(filename,map,STYLESHEET);
-                map.addVectorLayer(_layer);
+                // create a new layer for every trace, so they can be turned on/off individually
+                _layer = map.addLayer(connection, STYLESHEET);
             }
             return _layer;
         }
@@ -96,11 +113,13 @@ package net.systemeD.halcyon.connection {
 
         private function processEvent(e:Event):void {
             removeEventListener("loaded_data", processEvent);
+            _isLoaded=true;
             process();
         }
 
         private function process():void {
             var file:XML = new XML(_traceData);
+            var action:CompositeUndoableAction = new CompositeUndoableAction("add trace objects");
                        for each (var ns:Namespace in file.namespaceDeclarations()) {
                                if (ns.uri.match(/^http:\/\/www\.topografix\.com\/GPX\/1\/[01]$/)) {
                                        default xml namespace = ns;
@@ -111,10 +130,10 @@ package net.systemeD.halcyon.connection {
                 var way:Way;
                 var nodestring:Array = [];
                 for each (var trkpt:XML in trkseg.trkpt) {
-                    nodestring.push(layer.createNode({}, trkpt.@lat, trkpt.@lon));
+                    nodestring.push(connection.createNode({}, trkpt.@lat, trkpt.@lon, action.push));
                 }
                 if (nodestring.length > 0) {
-                    way = layer.createWay({}, nodestring);
+                    way = connection.createWay({}, nodestring, action.push);
                     //if (simplify) { Simplify.simplify(way, paint.map, false); }
                 }
             }
@@ -124,12 +143,13 @@ package net.systemeD.halcyon.connection {
                 for each (var tag:XML in wpt.children()) {
                     tags[tag.name().localName]=tag.toString();
                 }
-                var node:Node = layer.createNode(tags, wpt.@lat, wpt.@lon);
-                               layer.registerPOI(node);
+                var node:Node = connection.createNode(tags, wpt.@lat, wpt.@lon, action.push);
+                               connection.registerPOI(node);
             }
 
+            action.doAction(); /* just do it, don't add to undo stack */
                        default xml namespace = new Namespace("");
-            layer.paint.updateEntityUIs(true, false);
+            layer.updateEntityUIs(true, false);
         }
     }
 }