change relation.appendMember to use the undo system, and update various places as...
[potlatch2.git] / net / systemeD / halcyon / connection / actions / SplitWayAction.as
index c3d5509f078d4116119f38e6cbcb9cdd639e72f4..b6b815f8e2c20290e20393293939aac0b14880fc 100644 (file)
@@ -22,20 +22,70 @@ package net.systemeD.halcyon.connection.actions {
                                        selectedWay.sliceNodes(selectedWay.indexOfNode(selectedNode),selectedWay.length),
                                        push);
 
-                               selectedWay.deleteNodesFrom(selectedWay.indexOfNode(selectedNode)+1, push);
+                               // we reverse the list, which is already sorted by position. This way positions aren't affected
+                               // for previous inserts when all the inserts are eventually executed
+                               for each (var o:Object in selectedWay.memberships.reverse()) {
+                                       // don't add a turn restriction to the relation if it's no longer relevant
+                                       if (o.relation.tagIs('type','restriction')) {
+                                               var vias:Array=o.relation.findMembersByRole('via');
+                                               if (vias.length && vias[0] is Node) {
+                                                       if (newWay.indexOfNode(Node(vias[0]))==-1) { 
+                                                               continue;
+                                                       }
+                                               }
+                                       }
 
-                               // copy relations
-                               // FIXME make this reversible
-                               // FIXME should be more clever about the position (for ordered relations).
-                               //        It should either be before, or after, the selectedWay, depending
-                               //        on the relative sequence of the relation members compared to the 
-                               //        direction of selectedWay.
-                               // FIXME if we insert twice into the same relation, the position may become
-                               //        boggled (i.e. "10th position" is no longer valid if we previously
-                               //        inserted something earlier).
-                               for each (var o:Object in selectedWay.memberships) {
-                                       o.relation.insertMember(o.position, new RelationMember(newWay, o.role));
+                  // newWay should be added immediately after the selectedWay, unless the setup
+                  // is arse-backwards. By that I mean either:
+                  // a) The first node (0) of selectedWay is in the subsequentWay, or
+                  // b) The last node (N) of selectedWay is in the preceedingWay
+                  // Note that the code above means newWay is the tail of selectedWay S-->.-->N
+                  // i.e. preceedingWay x--x--x--x                             P-1   ↓
+                  //      selectedWay            N<--.<--S<--.<--0             P     ↓ relation members list
+                  //      subsequentWay                           x--x--x--x   P+1   ↓
+                  // There are some edge cases:
+                  // 1) If the immediately adjacent member isn't a way - handled fine
+                  // 2) If the selectedWay shares first/last node with non-adjacent ways - phooey
+                  
+                  var backwards:Boolean = false;
+                  // note that backwards is actually a ternary of 'true', 'false', and 'itdoesntmatter' (== 'false')
+                  
+                  var offset:int = 1; //work from o.position outwards along the length of the relationmembers
+                  while ((o.position - offset) >= 0 || (o.position + offset < o.relation.length)) {
+                    if ((o.position - offset >= 0) && o.relation.getMember(o.position - offset).entity is Way)  {
+                      var preceedingWay:Way = o.relation.getMember(o.position - offset).entity as Way;
+                      if(preceedingWay.indexOfNode(selectedWay.getLastNode()) >= 0) {
+                        backwards = true;
+                      }
+                    }
+                    if ((o.position + offset < o.relation.length) && o.relation.getMember(o.position + offset).entity is Way) {
+                      var subsequentWay:Way = o.relation.getMember(o.position + offset).entity as Way;
+                      if(subsequentWay.indexOfNode(selectedWay.getNode(0)) >= 0) {
+                        backwards = true;
+                      }
+                    }
+                    offset++;
+                  }
+                  if (backwards) {
+                    o.relation.insertMember(o.position, new RelationMember(newWay, o.role), push); //insert newWay before selectedWay
+                  } else {
+                    o.relation.insertMember(o.position + 1, new RelationMember(newWay, o.role), push); // insert after
+                  }
+                }
+                
+                // now that we're done with the selectedWay, remove the nodes
+                selectedWay.deleteNodesFrom(selectedWay.indexOfNode(selectedNode)+1, push);
+
+                               // and remove from any turn restrictions that aren't relevant
+                               for each (var r:Relation in selectedWay.findParentRelationsOfType('restriction')) {
+                                       vias=r.findMembersByRole('via');
+                                       if (vias.length && vias[0] is Node) {
+                                               if (selectedWay.indexOfNode(Node(vias[0]))==-1) { 
+                                                       r.removeMember(selectedWay,push);
+                                               }
+                                       }
                                }
+
             }
             newWay.suspend();
             selectedWay.suspend();