Changeset 17540 in osm for applications/editors/josm/plugins/validator/src/org

Timestamp:

2009-09-09T00:01:56+02:00 (15 years ago)

Author:

guggis

Message:

Updating to JOSM r2082

Location:

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator

Files:

• OSMValidatorPlugin.java (modified) (2 diffs)
• ValidateUploadHook.java (modified) (3 diffs)
• tests/UntaggedWay.java (modified) (1 diff)
• util/AgregatePrimitivesVisitor.java (modified) (1 diff)

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/OSMValidatorPlugin.java

@@ -156 +156 @@
     }
 
+    private ValidateUploadHook uploadHook;
+    
     @Override
     public void mapFrameInitialized(MapFrame oldFrame, MapFrame newFrame) {
@@ -168 +170 @@
             Layer.listeners.remove(this);
 
-        LinkedList<UploadHook> hooks = ((UploadAction) Main.main.menu.upload).uploadHooks;
-        Iterator<UploadHook> hooksIt = hooks.iterator();
-        while (hooksIt.hasNext()) {
-            if (hooksIt.next() instanceof ValidateUploadHook) {
-                if (newFrame == null)
-                    hooksIt.remove();
-                break;
-            }
-        }
-        if (newFrame != null)
-            hooks.add(0, new ValidateUploadHook(this));
+        if (newFrame != null) {
+                UploadAction.registerUploadHook(uploadHook = new ValidateUploadHook(this));
+        } else {
+                UploadAction.unregisterUploadHook(uploadHook);
+                uploadHook = null;
+        }
     }
 

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/ValidateUploadHook.java

@@ -14 +14 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.UploadAction.UploadHook;
+import org.openstreetmap.josm.data.APIDataSet;
 import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
@@ -42 +43 @@
      * Validate the modified data before uploading
      */
-    public boolean checkUpload(Collection<OsmPrimitive> add, Collection<OsmPrimitive> update, Collection<OsmPrimitive> delete)
+    public boolean checkUpload(APIDataSet apiDataSet)
     {
         Collection<Test> tests = OSMValidatorPlugin.getEnabledTests(true);
@@ -49 +50 @@
 
         AgregatePrimitivesVisitor v = new AgregatePrimitivesVisitor();
-        v.visit(add);
-        Collection<OsmPrimitive> selection = v.visit(update);
+        v.visit(apiDataSet.getPrimitivesToAdd());
+        Collection<OsmPrimitive> selection = v.visit(apiDataSet.getPrimitivesToUpdate());
 
         List<TestError> errors = new ArrayList<TestError>(30);

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/UntaggedWay.java

@@ -119 +119 @@
             if(r.isUsable() && "multipolygon".equals(r.get("type")))
             {
-                for (RelationMember m : r.members)
+                for (RelationMember m : r.getMembers())
                 {
-                    if(m.member != null && m.member instanceof Way &&
-                    m.member.isUsable() && !m.member.isTagged())
-                        multipolygonways.add((Way)m.member);
+                    if(m.getMember() != null && m.getMember() instanceof Way &&
+                    m.getMember().isUsable() && !m.getMember().isTagged())
+                        multipolygonways.add((Way)m.getMember());
                 }
             }

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/util/AgregatePrimitivesVisitor.java

@@ -65 +65 @@
         if (!aggregatedData.contains(r)) {
             aggregatedData.add(r);
-            for (RelationMember m : r.members) {
-                m.member.visit(this);
+            for (RelationMember m : r.getMembers()) {
+                m.getMember().visit(this);
             }
         }