Add further details into the splitWay relations handling section

diff --git a/net/systemeD/halcyon/connection/actions/SplitWayAction.as b/net/systemeD/halcyon/connection/actions/SplitWayAction.as
index 0cf03d989069354f94d745ccbb44e2604b9d212b..ef1e8dbcdb03534619c4f4286a33a351c9eb8fdc 100644 (file)
--- a/net/systemeD/halcyon/connection/actions/SplitWayAction.as
+++ b/net/systemeD/halcyon/connection/actions/SplitWayAction.as
@@ -24,9 +24,14 @@ package net.systemeD.halcyon.connection.actions {
               selectedWay.deleteNodesFrom(selectedWay.indexOfNode(selectedNode)+1, push);
               
               // copy relations
               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) {
               for each (var r:Relation in selectedWay.parentRelations) {

-                  // ** needs to copy roles as well

                   r.appendMember(new RelationMember(newWay, ''));
               }
             }
                   r.appendMember(new RelationMember(newWay, ''));
               }
             }