Changeset 5211 in josm for trunk/src/org/openstreetmap/josm/gui/widgets/OsmIdTextField.java

Timestamp:

2012-05-04T19:15:59+02:00 (12 years ago)

Author:

simon04

Message:

fix #7666 - Download object exception with invalid value

File:

• trunk/src/org/openstreetmap/josm/gui/widgets/OsmIdTextField.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/widgets/OsmIdTextField.java

@@ -30 +30 @@
     }
 
-    public long getOsmId() {
-        return validator.getOsmId();
-    }
-
     /**
      * Get entered ID list - supports "1,2,3" "1 2   ,3" or even "1 2 3 v2 6 v8"
@@ -46 +42 @@
     }
 
+    public void performValidation() {
+        validator.validate();
+    }
 
     /**
@@ -66 +65 @@
         @Override
         public boolean isValid() {
-            return getOsmId() > 0 || readOsmIds() != false;
+            return readOsmIds();
         }
 
@@ -78 +77 @@
         }
 
-        public long getOsmId() {
-            String value  = getComponent().getText();
-            if (value == null || value.trim().length() == 0) return 0;
-            try {
-                long osmId = Long.parseLong(value.trim());
-                if (osmId > 0) 
-                    return osmId;
-                return 0;
-            } catch(NumberFormatException e) {
-                return 0;
-            }
-        }
-        
         public boolean readOsmIds() {
             String value = getComponent().getText();
@@ -110 +96 @@
                         ids.add(SimplePrimitiveId.fromString(s));
                     } catch (IllegalArgumentException ex) {
-                        if (type == OsmPrimitiveType.NODE) {
-                            ids.add(new SimplePrimitiveId(Long.parseLong(s), OsmPrimitiveType.NODE));
-                        } else if (type == OsmPrimitiveType.WAY) {
-                            ids.add(new SimplePrimitiveId(Long.parseLong(s), OsmPrimitiveType.WAY));
-                        } else if (type == OsmPrimitiveType.RELATION) {
-                            ids.add(new SimplePrimitiveId(Long.parseLong(s), OsmPrimitiveType.RELATION));
-                        } else {
+                        try {
+                            if (type == OsmPrimitiveType.NODE) {
+                                ids.add(new SimplePrimitiveId(Long.parseLong(s), OsmPrimitiveType.NODE));
+                            } else if (type == OsmPrimitiveType.WAY) {
+                                ids.add(new SimplePrimitiveId(Long.parseLong(s), OsmPrimitiveType.WAY));
+                            } else if (type == OsmPrimitiveType.RELATION) {
+                                ids.add(new SimplePrimitiveId(Long.parseLong(s), OsmPrimitiveType.RELATION));
+                            } else {
+                                return false;
+                            }
+                        } catch (IllegalArgumentException ex2) {
                             return false;
                         }