Add further details into the splitWay relations handling section
authorAndy Allan <gravitystorm@gmail.com>
Sun, 2 May 2010 14:03:46 +0000 (14:03 +0000)
committerAndy Allan <gravitystorm@gmail.com>
Sun, 2 May 2010 14:03:46 +0000 (14:03 +0000)
net/systemeD/halcyon/connection/actions/SplitWayAction.as

index 0cf03d9..ef1e8db 100644 (file)
@@ -24,9 +24,14 @@ package net.systemeD.halcyon.connection.actions {
               selectedWay.deleteNodesFrom(selectedWay.indexOfNode(selectedNode)+1, push);
               
               // copy relations
-              // TODO make this reversible
+              // FIXME make this reversible
+              // FIXME needs to copy roles as well
+              // FIXME needs to insert the new way in the correct position in 
+              //        the relation, in order to not destroy ordered route relations.
+              //        This will either be before, or after, the selectedWay, depending
+              //        on the relative sequence of the relation members compared to the 
+              //        direction of selectedWay.
               for each (var r:Relation in selectedWay.parentRelations) {
-                  // ** needs to copy roles as well
                   r.appendMember(new RelationMember(newWay, ''));
               }
             }