fix #1833: Export / Import GPX

1 files changed, 8 insertions, 4 deletions

diff --git a/main/src/cgeo/geocaching/files/GPXParser.java b/main/src/cgeo/geocaching/files/GPXParser.java
index f8ec61c..454c494 100644
--- a/main/src/cgeo/geocaching/files/GPXParser.java
+++ b/main/src/cgeo/geocaching/files/GPXParser.java
@@ -255,8 +255,13 @@ public abstract class GPXParser extends FileParser {
             public void start(Attributes attrs) {
                 try {
                     if (attrs.getIndex("lat") > -1 && attrs.getIndex("lon") > -1) {
-                        cache.setCoords(new Geopoint(Double.valueOf(attrs.getValue("lat")),
-                                Double.valueOf(attrs.getValue("lon"))));
+                        final String latitude = attrs.getValue("lat");
+                        final String longitude = attrs.getValue("lon");
+                        // latitude and longitude are required attributes, but we export them empty for waypoints without coordinates
+                        if (StringUtils.isNotBlank(latitude) && StringUtils.isNotBlank(longitude)) {
+                            cache.setCoords(new Geopoint(Double.valueOf(latitude),
+                                    Double.valueOf(longitude)));
+                        }
                     }
                 } catch (Exception e) {
                     Log.w("Failed to parse waypoint's latitude and/or longitude.");
@@ -300,8 +305,7 @@ public abstract class GPXParser extends FileParser {
                     result.put(key, cache);
                     showProgressMessage(progressHandler, progressStream.getProgress());
                 } else if (StringUtils.isNotBlank(cache.getName())
-                        && cache.getCoords() != null
-                        && StringUtils.contains(type, "waypoint")) {
+                        && StringUtils.containsIgnoreCase(type, "waypoint")) {
                     addWaypointToCache();
                 }