Changeset 14055 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2018-07-28T01:03:54+02:00 (6 years ago)

Author:

Don-vip

Message:

fix #14103 - GPX → OSM: convert additional tags

Location:

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

Files:

• OsmDataLayer.java (modified) (1 diff)
• gpx/ConvertToDataLayerAction.java (modified) (3 diffs)
• markerlayer/Marker.java (modified) (4 diffs)

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

@@ -773 +773 @@
 
         if (time > 0) {
+            wpt.put(GpxConstants.PT_TIME, DateUtils.fromTimestamp(time));
             wpt.setTime(time);
+        } else if (n.hasKey(GpxConstants.PT_TIME)) {
+            wpt.put(GpxConstants.PT_TIME, DateUtils.fromString(n.get(GpxConstants.PT_TIME)));
+            wpt.setTime();
         } else if (!n.isTimestampEmpty()) {
-            wpt.put("time", DateUtils.fromTimestamp(n.getRawTimestamp()));
+            wpt.put(GpxConstants.PT_TIME, DateUtils.fromTimestamp(n.getRawTimestamp()));
             wpt.setTime();
         }

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

@@ -8 +8 @@
 import java.awt.event.ActionEvent;
 import java.io.File;
+import java.text.DateFormat;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.Date;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Map.Entry;
 import java.util.Optional;
 
@@ -67 +70 @@
     public static class FromGpxLayer extends ConvertToDataLayerAction<GpxLayer> {
 
+        private final DateFormat timeFormatter = DateUtils.getGpxFormat();
+
         /**
          * Creates a new {@code FromGpxLayer}.
@@ -83 +88 @@
                     for (WayPoint p : segment.getWayPoints()) {
                         Node n = new Node(p.getCoor());
-                        String timestr = p.getString(GpxConstants.PT_TIME);
-                        if (timestr != null) {
-                            try {
-                                n.setTimestamp(DateUtils.fromString(timestr));
-                            } catch (UncheckedParseException e) {
-                                Logging.log(Logging.LEVEL_WARN, e);
+                        for (Entry<String, Object> entry : p.attr.entrySet()) {
+                            String key = entry.getKey();
+                            String str = p.getString(key);
+                            if (str != null) {
+                                n.put(key, str);
+                                if (GpxConstants.PT_TIME.equals(key)) {
+                                    try {
+                                        n.setTimestamp(DateUtils.fromString(str));
+                                    } catch (UncheckedParseException e) {
+                                        Logging.log(Logging.LEVEL_WARN, e);
+                                    }
+                                }
+                            } else {
+                                Object obj = p.get(key);
+                                if (obj instanceof Date && GpxConstants.PT_TIME.equals(key)) {
+                                    Date date = (Date) obj;
+                                    n.put(key, timeFormatter.format(date));
+                                    n.setTimestamp(date);
+                                }
                             }
-                        }
-                        String elestr = p.getString(GpxConstants.PT_ELE);
-                        if (elestr != null) {
-                            n.put("ele", elestr);
                         }
                         ds.addPrimitive(n);

trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java

@@ -11 +11 @@
 import java.io.File;
 import java.text.DateFormat;
-import java.text.SimpleDateFormat;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -35 +34 @@
 import org.openstreetmap.josm.tools.ImageProvider;
 import org.openstreetmap.josm.tools.Logging;
+import org.openstreetmap.josm.tools.date.DateUtils;
 import org.openstreetmap.josm.tools.template_engine.ParseError;
 import org.openstreetmap.josm.tools.template_engine.TemplateEngineDataProvider;
@@ -209 +209 @@
     public static final String LABEL_PATTERN_DESC = "{desc}";
 
-    private final DateFormat timeFormatter = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss.SSS'Z'");
+    private final DateFormat timeFormatter = DateUtils.getGpxFormat();
     private final TemplateEngineDataProvider dataProvider;
     private final String text;
@@ -262 +262 @@
     public WayPoint convertToWayPoint() {
         WayPoint wpt = new WayPoint(getCoor());
-        wpt.put("time", timeFormatter.format(new Date(Math.round(time * 1000))));
+        wpt.put(GpxConstants.PT_TIME, timeFormatter.format(new Date(Math.round(time * 1000))));
         if (text != null) {
             wpt.addExtension("text", text);