Changeset 18753 in josm for trunk/src

Timestamp:

2023-06-13T22:36:12+02:00 (21 months ago)

Author:

taylor.smock

Message:

Fix #23001: ClassCastException in GpxDrawHelper#calculateColors

This occurs when a working GPX file with HDOP information is converted to an OSM
Data Layer and back to a GPX Data Layer. This is fixed by (a) converting casts
for get operations to Number instead of Float/Integer and (b) actually
putting the number in the attribute map instead of a string.

Location:

trunk/src/org/openstreetmap/josm/gui/layer

Files:

• OsmDataLayer.java (modified) (2 diffs)
• gpx/GpxDrawHelper.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

@@ -945 +945 @@
         if (value != null) {
             try {
-                int i = Integer.parseInt(value);
+                final int i = Integer.parseInt(value);
                 // Sanity checks
                 if ((!GpxConstants.PT_SAT.equals(gpxKey) || i >= 0) &&
                         (!GpxConstants.PT_DGPSID.equals(gpxKey) || (0 <= i && i <= 1023))) {
-                    wpt.put(gpxKey, value);
+                    wpt.put(gpxKey, i);
                 }
             } catch (NumberFormatException e) {
@@ -964 +964 @@
         if (value != null) {
             try {
-                double d = Double.parseDouble(value);
+                final double d = Double.parseDouble(value);
                 // Sanity checks
                 if (!GpxConstants.PT_MAGVAR.equals(gpxKey) || (0.0 <= d && d < 360.0)) {
-                    wpt.put(gpxKey, value);
+                    wpt.put(gpxKey, d);
                 }
             } catch (NumberFormatException e) {

trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDrawHelper.java

@@ -626 +626 @@
         }
 
-        ArrayList<String> refs = new ArrayList<String>();
+        ArrayList<String> refs = new ArrayList<>();
         if (colored == ColorMode.REF) {
             for (Line segment : getLinesIterable(null)) {
@@ -639 +639 @@
                 }
             }
-            if (refs.size() > 0) {
+            if (!refs.isEmpty()) {
                 Collections.sort(refs);
                 String[] a = {};
@@ -660 +660 @@
 
                 if (colored == ColorMode.HDOP) {
-                    color = hdopScale.getColor((Float) trkPnt.get(GpxConstants.PT_HDOP));
+                    color = hdopScale.getColor((Number) trkPnt.get(GpxConstants.PT_HDOP));
                 } else if (colored == ColorMode.QUALITY) {
-                    color = qualityScale.getColor((Integer) trkPnt.get(GpxConstants.RTKLIB_Q));
+                    color = qualityScale.getColor((Number) trkPnt.get(GpxConstants.RTKLIB_Q));
                 } else if (colored == ColorMode.FIX) {
                     Object fixval = trkPnt.get(GpxConstants.PT_FIX);
@@ -671 +671 @@
                         }
                     }
-                } else if (colored == ColorMode.REF) {
-                    if (trkPnt.get(GpxConstants.PT_DGPSID) != null) {
-                        String refval = trkPnt.get(GpxConstants.PT_DGPSID).toString();
-                        int i = refs.indexOf(refval);
-                        if (i >= 0) {
-                            color = refScale.getColor(i);
-                        }
+                } else if (colored == ColorMode.REF && trkPnt.get(GpxConstants.PT_DGPSID) != null) {
+                    String refVal = trkPnt.get(GpxConstants.PT_DGPSID).toString();
+                    int i = refs.indexOf(refVal);
+                    if (i >= 0) {
+                        color = refScale.getColor(i);
                     }
                 }
@@ -852 +850 @@
                 if (hdopCircle && trkPnt.get(GpxConstants.PT_HDOP) != null) {
                     // hdop value
-                    float hdop = (Float) trkPnt.get(GpxConstants.PT_HDOP);
+                    float hdop = ((Number) trkPnt.get(GpxConstants.PT_HDOP)).floatValue();
                     if (hdop < 0) {
                         hdop = 0;