Changeset 26460 in osm for applications

Timestamp:

2011-08-06T08:05:16+02:00 (13 years ago)

Author:

akks

Message:

'FastDraw: fix NPE and relation bugs after editing'

Location:

applications/editors/josm/plugins/FastDraw

Files:

• build.xml (modified) (2 diffs)
• src/org/openstreetmap/josm/plugins/fastdraw/FastDrawingMode.java (modified) (2 diffs)

applications/editors/josm/plugins/FastDraw/build.xml

@@ -30 +30 @@
 <project name="FastDraw" default="dist" basedir=".">
     <!-- enter the SVN commit message -->
-    <property name="commit.message" value="FastDraw: fix NPE in last segment drawing"/>
+    <property name="commit.message" value="FastDraw: fix NPE and relation bugs after editing"/>
     <!-- enter the *lowest* JOSM version this plugin is currently compatible with -->
     <property name="plugin.main.version" value="4201"/>
@@ -248 +248 @@
     <target name="runjosm" depends="install">
         <java jar="${josm}" fork="true">
-            <arg line="../../core/data_nodist/restriction.osm"/>
+            <arg line="../../core/data_nodist/multipolygon.osm"/>
         </java>
     </target>

applications/editors/josm/plugins/FastDraw/src/org/openstreetmap/josm/plugins/fastdraw/FastDrawingMode.java

@@ -554 +554 @@
                     List<OsmPrimitive> refs = nd.getReferrers();
                     // does someone need this node? if no-delete it.
-                    if (refs.isEmpty()) cmds.add(new DeleteCommand(nd));                                       
+                    if (refs.isEmpty() && !nd.isDeleted() && nd.isUsable()) cmds.add(new DeleteCommand(nd));                                       
                 }
             }
@@ -663 +663 @@
         if (w.isClosed()) line.closeLine();
         oldNodes = w.getNodes();
-        cmds.add(new DeleteCommand(w));
+        List <OsmPrimitive> wl = new ArrayList(); wl.add(w);
+        
+        Command c = DeleteCommand.delete(getEditLayer(), wl, false);
+        if (c != null) cmds.add(c);
         delCmd = new SequenceCommand(tr("Convert way to FastDraw line"), cmds);
         Main.main.undoRedo.add(delCmd);