Changeset 210 in josm for src/org/openstreetmap/josm/io

Timestamp:

2007-04-04T13:21:40+02:00 (18 years ago)

Author:

imi

Message:

• added the possibility for <wpt> tags in GPX files to construct markers from relative ​file:// urls
• fixed NPE when illegal characters in HTTP response from the server (fix for the fix)

Location:

src/org/openstreetmap/josm/io

Files:

• BoundingBoxDownloader.java (modified) (2 diffs)
• RawGpsReader.java (modified) (5 diffs)

src/org/openstreetmap/josm/io/BoundingBoxDownloader.java

@@ -49 +49 @@
                                 break;
                         // Use only track points, since the server mix everything together 
-                        Collection<Collection<GpsPoint>> allWays = new RawGpsReader(in).trackData;
+                        Collection<Collection<GpsPoint>> allWays = new RawGpsReader(in, null).trackData;
 
                         boolean foundSomething = false;
@@ -70 +70 @@
                 if (cancel)
                         return null;
-                throw e;
+                throw new SAXException("Illegal characters within the HTTP-header response", e);
         } catch (IOException e) {
                 if (cancel)

src/org/openstreetmap/josm/io/RawGpsReader.java

@@ -3 +3 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
+import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
@@ -15 +16 @@
 import org.openstreetmap.josm.gui.layer.RawGpsLayer.GpsPoint;
 import org.openstreetmap.josm.gui.layer.markerlayer.Marker;
+import org.openstreetmap.josm.gui.layer.markerlayer.MarkerProducers;
 import org.xml.sax.Attributes;
 import org.xml.sax.SAXException;
@@ -26 +28 @@
  */
 public class RawGpsReader {
+
+        /**
+         * The relative path when constructing markers from wpt-tags. Passed to 
+         * {@link MarkerProducers#createMarker(LatLon, java.util.Map, String)}
+         */
+        private File relativeMarkerPath;
 
         /**
@@ -90 +98 @@
                                 currentTagValues.clear();
                         } else if (qName.equals("wpt")) {
-                                markerData.add(Marker.createMarker(currentLatLon, currentTagValues));
+                                markerData.add(Marker.createMarker(currentLatLon, currentTagValues, relativeMarkerPath));
                                 currentTagValues.clear();
                         } else if (qName.equals("trkseg") || qName.equals("trk") || qName.equals("gpx")) {
@@ -107 +115 @@
         }
 
-
         /**
-         * Parse the input stream and store the result in trackData and markerData 
+         * Parse the input stream and store the result in trackData and markerData
+         * 
+         * @param relativeMarkerPath The directory to use as relative path for all <wpt> 
+         *    marker tags. Maybe <code>null</code>, in which case no relative urls are constructed for the markers. 
          */
-        public RawGpsReader(InputStream source) throws SAXException, IOException {
+        public RawGpsReader(InputStream source, File relativeMarkerPath) throws SAXException, IOException {
+                this.relativeMarkerPath = relativeMarkerPath;
                 Parser parser = new Parser();
                 parser.parse(new InputStreamReader(source, "UTF-8"));