Index: src/org/openstreetmap/josm/actions/SaveAction.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/actions/SaveAction.java b/trunk/src/org/openstreetmap/josm/actions/SaveAction.java
--- a/trunk/src/org/openstreetmap/josm/actions/SaveAction.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/actions/SaveAction.java (working copy)
@@ -14,14 +14,14 @@
import javax.swing.JPanel;
import javax.swing.SwingConstants;
-import org.openstreetmap.josm.data.gpx.GpxData.GpxDataChangeListener;
+import org.openstreetmap.josm.data.gpx.GpxConstants;
import org.openstreetmap.josm.gui.ExtendedDialog;
import org.openstreetmap.josm.gui.MainApplication;
+import org.openstreetmap.josm.gui.layer.AbstractModifiableLayer;
import org.openstreetmap.josm.gui.layer.GpxLayer;
import org.openstreetmap.josm.gui.layer.Layer;
import org.openstreetmap.josm.gui.layer.LayerManager.LayerAddEvent;
import org.openstreetmap.josm.gui.layer.LayerManager.LayerRemoveEvent;
-import org.openstreetmap.josm.gui.layer.OsmDataLayer;
import org.openstreetmap.josm.gui.layer.SaveToFile;
import org.openstreetmap.josm.gui.util.GuiHelper;
import org.openstreetmap.josm.spi.preferences.Config;
@@ -37,13 +37,11 @@
private static final SaveAction instance = new SaveAction();
private final PropertyChangeListener updateOnRequireSaveChange = evt -> {
- if (OsmDataLayer.REQUIRES_SAVE_TO_DISK_PROP.equals(evt.getPropertyName())) {
+ if (AbstractModifiableLayer.REQUIRES_SAVE_TO_DISK_PROP.equals(evt.getPropertyName())) {
updateEnabledState();
}
};
- private final GpxDataChangeListener updateOnRequireSaveChangeGpx = evt -> updateEnabledState();
-
/**
* Construct the action with "Save" as label.
*/
@@ -68,12 +66,9 @@
@Override
public void layerAdded(LayerAddEvent e) {
Layer l = e.getAddedLayer();
- if (l instanceof OsmDataLayer) {
+ if (l instanceof AbstractModifiableLayer) {
l.addPropertyChangeListener(updateOnRequireSaveChange);
}
- if (l instanceof GpxLayer) {
- ((GpxLayer) l).data.addWeakChangeListener(updateOnRequireSaveChangeGpx);
- }
super.layerAdded(e);
}
@@ -80,12 +75,9 @@
@Override
public void layerRemoving(LayerRemoveEvent e) {
Layer l = e.getRemovedLayer();
- if (l instanceof OsmDataLayer) {
+ if (l instanceof AbstractModifiableLayer) {
l.removePropertyChangeListener(updateOnRequireSaveChange);
}
- if (l instanceof GpxLayer) {
- ((GpxLayer) l).data.removeChangeListener(updateOnRequireSaveChangeGpx);
- }
super.layerRemoving(e);
}
};
@@ -113,11 +105,18 @@
}
// Ask for overwrite in case of GpxLayer
- if (f != null && layer instanceof GpxLayer && !Config.getPref().getBoolean("gpx.export.overwrite", false)) {
+ if (f != null
+ && layer instanceof GpxLayer
+ && (((GpxLayer) layer).data == null
+ || !GpxConstants.JOSM_CREATOR_NAME.equals(((GpxLayer) layer).data.creator))
+ && !Config.getPref().getBoolean("gpx.export.overwrite", false)) {
+
JPanel p = new JPanel(new GridBagLayout());
- JLabel label = new JLabel(tr("File {0} exists. Overwrite?", f.getName()));
+ JLabel label = new JLabel(""
+ + tr("The file \"{0}\" will be modified.
Would you like to overwrite the existing file?", f.getName())
+ + "");
label.setHorizontalAlignment(SwingConstants.CENTER);
- JCheckBox remember = new JCheckBox(tr("Remember choice"));
+ JCheckBox remember = new JCheckBox(tr("Always overwrite GPX files without asking"));
remember.setHorizontalAlignment(SwingConstants.CENTER);
p.add(label, GBC.eol().fill(GBC.HORIZONTAL).insets(5, 5, 5, 10));
p.add(remember, GBC.eop().fill(GBC.HORIZONTAL));
@@ -124,13 +123,16 @@
ExtendedDialog dialog = new ExtendedDialog(
MainApplication.getMainFrame(),
tr("Overwrite"),
- tr("Overwrite"), tr("Cancel"))
- .setButtonIcons("save_as", "cancel")
+ tr("Overwrite"), tr("New file"), tr("Cancel"))
+ .setButtonIcons("save", "save_as", "cancel")
.setContent(p);
- if (dialog.showDialog().getValue() != 1) {
+ int val = dialog.showDialog().getValue();
+ if (val == 1) {
+ Config.getPref().putBoolean("gpx.export.overwrite", remember.isSelected());
+ } else if (val == 2) {
f = null;
- } else if (remember.isSelected()) {
- Config.getPref().putBoolean("gpx.export.overwrite", true);
+ } else {
+ return null;
}
}
return f == null ? layer.createAndOpenSaveFileChooser() : f;
Index: src/org/openstreetmap/josm/actions/SessionSaveAsAction.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/actions/SessionSaveAsAction.java b/trunk/src/org/openstreetmap/josm/actions/SessionSaveAsAction.java
--- a/trunk/src/org/openstreetmap/josm/actions/SessionSaveAsAction.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/actions/SessionSaveAsAction.java (working copy)
@@ -19,6 +19,7 @@
import java.util.Map;
import java.util.Set;
import java.util.stream.Collectors;
+import java.util.stream.Stream;
import javax.swing.BorderFactory;
import javax.swing.JCheckBox;
@@ -32,11 +33,14 @@
import javax.swing.border.EtchedBorder;
import javax.swing.filechooser.FileFilter;
+import org.openstreetmap.josm.data.preferences.BooleanProperty;
import org.openstreetmap.josm.gui.ExtendedDialog;
import org.openstreetmap.josm.gui.HelpAwareOptionPane;
import org.openstreetmap.josm.gui.MainApplication;
import org.openstreetmap.josm.gui.MapFrame;
import org.openstreetmap.josm.gui.MapFrameListener;
+import org.openstreetmap.josm.gui.Notification;
+import org.openstreetmap.josm.gui.layer.AbstractModifiableLayer;
import org.openstreetmap.josm.gui.layer.Layer;
import org.openstreetmap.josm.gui.util.WindowGeometry;
import org.openstreetmap.josm.gui.widgets.AbstractFileChooser;
@@ -58,6 +62,8 @@
private transient Map exporters;
private transient MultiMap dependencies;
+ private static final BooleanProperty SAVE_LOCAL_FILES_PROPERTY = new BooleanProperty("session.savelocal", true);
+
/**
* Constructs a new {@code SessionSaveAsAction}.
*/
@@ -156,6 +162,36 @@
.filter(layer -> exporters.get(layer) != null && exporters.get(layer).shallExport())
.collect(Collectors.toList());
+ Stream layersToSaveStream = layersOut.stream()
+ .filter(layer -> layer.isSavable()
+ && layer instanceof AbstractModifiableLayer
+ && ((AbstractModifiableLayer) layer).requiresSaveToFile()
+ && exporters.get(layer) != null
+ && !exporters.get(layer).requiresZip());
+
+ if (SAVE_LOCAL_FILES_PROPERTY.get()) {
+ // individual files must be saved before the session file as the location may change
+
+ if (layersToSaveStream
+ .map(layer -> SaveAction.getInstance().doSave(layer, true))
+ .collect(Collectors.toList()) //force evaluation of all elements
+ .contains(false)) {
+
+ new Notification(tr("Not all local files referenced by the session file could be saved."
+ + "
Make sure you save them before closing JOSM."))
+ .setIcon(JOptionPane.WARNING_MESSAGE)
+ .setDuration(Notification.TIME_LONG)
+ .show();
+ }
+
+ } else if (layersToSaveStream.anyMatch(l -> true)) {
+ new Notification(tr("Not all local files referenced by the session file are saved yet."
+ + "
Make sure you save them before closing JOSM."))
+ .setIcon(JOptionPane.INFORMATION_MESSAGE)
+ .setDuration(Notification.TIME_LONG)
+ .show();
+ }
+
int active = -1;
Layer activeLayer = getLayerManager().getActiveLayer();
if (activeLayer != null) {
@@ -241,6 +277,7 @@
}
protected final Component build() {
+ JPanel op = new JPanel(new GridBagLayout());
JPanel ip = new JPanel(new GridBagLayout());
for (Layer layer : layers) {
JPanel wrapper = new JPanel(new GridBagLayout());
@@ -263,7 +300,13 @@
p.add(sp, GBC.eol().fill());
final JTabbedPane tabs = new JTabbedPane();
tabs.addTab(tr("Layers"), p);
- return tabs;
+ op.add(tabs, GBC.eol().fill());
+ JCheckBox chkSaveLocal = new JCheckBox(tr("Save all local files to disk"), SAVE_LOCAL_FILES_PROPERTY.get());
+ chkSaveLocal.addChangeListener(l -> {
+ SAVE_LOCAL_FILES_PROPERTY.put(chkSaveLocal.isSelected());
+ });
+ op.add(chkSaveLocal);
+ return op;
}
protected final Component getDisabledExportPanel(Layer layer) {
Index: src/org/openstreetmap/josm/data/gpx/GpxConstants.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/data/gpx/GpxConstants.java b/trunk/src/org/openstreetmap/josm/data/gpx/GpxConstants.java
--- a/trunk/src/org/openstreetmap/josm/data/gpx/GpxConstants.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/data/gpx/GpxConstants.java (working copy)
@@ -100,6 +100,11 @@
String META_BOUNDS = META_PREFIX + "bounds";
/**
+ * The creator element that will be written when exporting a GPX file
+ */
+ String JOSM_CREATOR_NAME = "JOSM GPX export";
+
+ /**
* Namespace for the XSD
*/
String XML_URI_XSD = "http://www.w3.org/2001/XMLSchema-instance";
Index: src/org/openstreetmap/josm/data/gpx/GpxData.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java b/trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java
--- a/trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java (working copy)
@@ -41,7 +41,7 @@
*
* @author Raphael Mack <ramack@raphael-mack.de>
*/
-public class GpxData extends WithAttributes implements Data {
+public class GpxData extends WithAttributes implements Data, IGpxLayerPrefs {
/**
* Constructs a new GpxData.
@@ -981,6 +981,7 @@
* @return Modifiable map
* @since 15496
*/
+ @Override
public Map getLayerPrefs() {
return layerPrefs;
}
@@ -1095,7 +1096,7 @@
suppressedInvalidate = true;
} else {
if (setModified) {
- modified = true;
+ setModified(true);
}
if (listeners.hasListeners()) {
GpxDataChangeEvent e = new GpxDataChangeEvent(this);
@@ -1137,6 +1138,14 @@
* @param e The event
*/
void gpxDataChanged(GpxDataChangeEvent e);
+
+ /**
+ * Called when the modified state of the data changed
+ * @param modified the new modified state
+ */
+ default void modifiedStateChanged(boolean modified) {
+ // Override if needed
+ }
}
/**
@@ -1175,8 +1184,14 @@
* @param value modified flag
* @since 15496
*/
+ @Override
public void setModified(boolean value) {
- modified = value;
+ if (!initializing && modified != value) {
+ modified = value;
+ if (listeners.hasListeners()) {
+ listeners.fireEvent(l -> l.modifiedStateChanged(modified));
+ }
+ }
}
/**
Index: src/org/openstreetmap/josm/data/gpx/IGpxLayerPrefs.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/data/gpx/IGpxLayerPrefs.java b/trunk/src/org/openstreetmap/josm/data/gpx/IGpxLayerPrefs.java
new file mode 100644
--- a/trunk/src/org/openstreetmap/josm/data/gpx/IGpxLayerPrefs.java (nonexistent)
+++ b/trunk/src/org/openstreetmap/josm/data/gpx/IGpxLayerPrefs.java (working copy)
@@ -0,0 +1,26 @@
+// License: GPL. For details, see LICENSE file.
+package org.openstreetmap.josm.data.gpx;
+
+import java.util.Map;
+
+/**
+ * Interface containing the layer preferences.
+ * Implemented by GpxLayer and MarkerLayer
+ * @since xxx
+ */
+public interface IGpxLayerPrefs {
+
+ /**
+ * The layer specific prefs formerly saved in the preferences, e.g. drawing options.
+ * NOT the track specific settings (e.g. color, width)
+ * @return Modifiable map
+ */
+ Map getLayerPrefs();
+
+ /**
+ * Sets the modified flag to the value.
+ * @param value modified flag
+ */
+ void setModified(boolean value);
+
+}
Index: src/org/openstreetmap/josm/gui/MainFrame.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/MainFrame.java b/trunk/src/org/openstreetmap/josm/gui/MainFrame.java
--- a/trunk/src/org/openstreetmap/josm/gui/MainFrame.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/MainFrame.java (working copy)
@@ -25,6 +25,7 @@
import javax.swing.JPanel;
import org.openstreetmap.josm.data.UserIdentityManager;
+import org.openstreetmap.josm.gui.layer.AbstractModifiableLayer;
import org.openstreetmap.josm.gui.layer.LayerManager.LayerAddEvent;
import org.openstreetmap.josm.gui.layer.LayerManager.LayerChangeListener;
import org.openstreetmap.josm.gui.layer.LayerManager.LayerOrderChangeEvent;
@@ -45,9 +46,9 @@
private final transient LayerStateChangeListener updateTitleOnLayerStateChange = (layer, newValue) -> onLayerChange(layer);
private final transient PropertyChangeListener updateTitleOnSaveChange = evt -> {
- if (evt.getPropertyName().equals(OsmDataLayer.REQUIRES_SAVE_TO_DISK_PROP)
+ if (evt.getPropertyName().equals(AbstractModifiableLayer.REQUIRES_SAVE_TO_DISK_PROP)
|| evt.getPropertyName().equals(OsmDataLayer.REQUIRES_UPLOAD_TO_SERVER_PROP)) {
- OsmDataLayer layer = (OsmDataLayer) evt.getSource();
+ AbstractModifiableLayer layer = (AbstractModifiableLayer) evt.getSource();
onLayerChange(layer);
}
};
@@ -186,7 +187,7 @@
getRootPane().putClientProperty("Window.documentModified", dirty);
}
- private void onLayerChange(OsmDataLayer layer) {
+ private void onLayerChange(AbstractModifiableLayer layer) {
if (layer == MainApplication.getLayerManager().getEditLayer()) {
refreshTitle();
}
Index: src/org/openstreetmap/josm/gui/io/importexport/GpxImporter.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/io/importexport/GpxImporter.java b/trunk/src/org/openstreetmap/josm/gui/io/importexport/GpxImporter.java
--- a/trunk/src/org/openstreetmap/josm/gui/io/importexport/GpxImporter.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/io/importexport/GpxImporter.java (working copy)
@@ -147,6 +147,7 @@
}
if (data.gpxLayer != null) {
MainApplication.getLayerManager().addLayer(data.gpxLayer);
+ MainApplication.getLayerManager().setActiveLayer(data.gpxLayer);
}
data.postLayerTask.run();
});
Index: src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java b/trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java
--- a/trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/layer/AbstractModifiableLayer.java (working copy)
@@ -16,6 +16,10 @@
*/
public abstract class AbstractModifiableLayer extends Layer implements DownloadFromServer, UploadToServer, SaveToFile, Lockable {
+ /** Property used to know if this layer has to be saved on disk */
+ public static final String REQUIRES_SAVE_TO_DISK_PROP = AbstractModifiableLayer.class.getName() + ".requiresSaveToDisk";
+ static final String IS_DIRTY_SYMBOL = "*";
+
/**
* Constructs a new {@code ModifiableLayer}.
* @param name Layer name
@@ -55,6 +59,17 @@
}
/**
+ * Determines if this layer is "dirty", i.e. requires save or upload
+ * @return if this layer is "dirty"
+ * @since 17626 in {@link OsmDataLayer}
+ * @since xxx in {@link AbstractModifiableLayer}
+ */
+ public boolean isDirty() {
+ // Override if needed
+ return requiresSaveToFile() || (requiresUploadToServer() && !isUploadDiscouraged());
+ }
+
+ /**
* Determines if data managed by this layer has been modified.
* @return true if data has been modified; false, otherwise
*/
@@ -130,4 +145,5 @@
// Override if needed;
return null;
}
+
}
Index: src/org/openstreetmap/josm/gui/layer/GpxLayer.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java b/trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java
--- a/trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java (working copy)
@@ -1,6 +1,28 @@
// License: GPL. For details, see LICENSE file.
package org.openstreetmap.josm.gui.layer;
+import static org.openstreetmap.josm.tools.I18n.tr;
+import static org.openstreetmap.josm.tools.I18n.trn;
+
+import java.awt.Color;
+import java.awt.Dimension;
+import java.awt.Graphics2D;
+import java.awt.event.ActionEvent;
+import java.io.File;
+import java.time.Instant;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
+import java.util.NoSuchElementException;
+import java.util.stream.Collectors;
+
+import javax.swing.AbstractAction;
+import javax.swing.Action;
+import javax.swing.Icon;
+import javax.swing.JScrollPane;
+import javax.swing.SwingUtilities;
+
import org.openstreetmap.josm.actions.AutoScaleAction;
import org.openstreetmap.josm.actions.ExpertToggleAction;
import org.openstreetmap.josm.actions.ExpertToggleAction.ExpertModeChangeListener;
@@ -11,6 +33,7 @@
import org.openstreetmap.josm.data.SystemOfMeasurement;
import org.openstreetmap.josm.data.gpx.GpxConstants;
import org.openstreetmap.josm.data.gpx.GpxData;
+import org.openstreetmap.josm.data.gpx.GpxData.GpxDataChangeEvent;
import org.openstreetmap.josm.data.gpx.GpxData.GpxDataChangeListener;
import org.openstreetmap.josm.data.gpx.IGpxTrack;
import org.openstreetmap.josm.data.gpx.IGpxTrackSegment;
@@ -35,6 +58,7 @@
import org.openstreetmap.josm.gui.layer.gpx.MarkersFromNamedPointsAction;
import org.openstreetmap.josm.gui.layer.markerlayer.MarkerLayer;
import org.openstreetmap.josm.gui.preferences.display.GPXSettingsPanel;
+import org.openstreetmap.josm.gui.util.GuiHelper;
import org.openstreetmap.josm.gui.widgets.HtmlPanel;
import org.openstreetmap.josm.tools.ImageProvider;
import org.openstreetmap.josm.tools.Logging;
@@ -41,27 +65,6 @@
import org.openstreetmap.josm.tools.Utils;
import org.openstreetmap.josm.tools.date.Interval;
-import javax.swing.AbstractAction;
-import javax.swing.Action;
-import javax.swing.Icon;
-import javax.swing.JScrollPane;
-import javax.swing.SwingUtilities;
-import java.awt.Color;
-import java.awt.Dimension;
-import java.awt.Graphics2D;
-import java.awt.event.ActionEvent;
-import java.io.File;
-import java.time.Instant;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.List;
-import java.util.NoSuchElementException;
-import java.util.stream.Collectors;
-
-import static org.openstreetmap.josm.tools.I18n.tr;
-import static org.openstreetmap.josm.tools.I18n.trn;
-
/**
* A layer that displays data from a Gpx file / the OSM gpx downloads.
*/
@@ -83,7 +86,19 @@
/**
* Added as field to be kept as reference.
*/
- private final GpxDataChangeListener dataChangeListener = e -> this.invalidate();
+ private final GpxDataChangeListener dataChangeListener = new GpxDataChangeListener() {
+ @Override
+ public void gpxDataChanged(GpxDataChangeEvent e) {
+ invalidate();
+ }
+
+ @Override
+ public void modifiedStateChanged(boolean modified) {
+ GuiHelper.runInEDT(() ->
+ propertyChangeSupport.firePropertyChange(REQUIRES_SAVE_TO_DISK_PROP, !modified, modified)
+ );
+ }
+ };
/**
* The MarkerLayer imported from the same file.
*/
@@ -356,6 +371,11 @@
}
@Override
+ public String getLabel() {
+ return isDirty() ? super.getLabel() + ' ' + IS_DIRTY_SYMBOL : super.getLabel();
+ }
+
+ @Override
public void visitBoundingBox(BoundingXYVisitor v) {
v.visit(data.recalculateBounds());
}
@@ -592,6 +612,9 @@
@Override
public synchronized void destroy() {
+ if (linkedMarkerLayer != null && MainApplication.getLayerManager().containsLayer(linkedMarkerLayer)) {
+ linkedMarkerLayer.data.transferLayerPrefs(data.getLayerPrefs());
+ }
data.clear();
data = null;
super.destroy();
Index: src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java b/trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java
--- a/trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java (working copy)
@@ -147,9 +147,6 @@
private static final int HATCHED_SIZE = 15;
// U+2205 EMPTY SET
private static final String IS_EMPTY_SYMBOL = "\u2205";
- private static final String IS_DIRTY_SYMBOL = "*";
- /** Property used to know if this layer has to be saved on disk */
- public static final String REQUIRES_SAVE_TO_DISK_PROP = OsmDataLayer.class.getName() + ".requiresSaveToDisk";
/** Property used to know if this layer has to be uploaded */
public static final String REQUIRES_UPLOAD_TO_SERVER_PROP = OsmDataLayer.class.getName() + ".requiresUploadToServer";
@@ -1065,15 +1062,6 @@
return getAssociatedFile() != null && requiresSaveToFile;
}
- /**
- * Determines if this layer is "dirty", i.e., requires save or upload
- * @return if this layer is "dirty"
- * @since 17626
- */
- public boolean isDirty() {
- return requiresSaveToFile() || (requiresUploadToServer() && !isUploadDiscouraged());
- }
-
@Override
public String getLabel() {
String label = super.getLabel();
Index: src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java b/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java
--- a/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java (working copy)
@@ -189,7 +189,9 @@
*/
public WayPoint convertToWayPoint() {
WayPoint wpt = new WayPoint(getCoor());
- wpt.setTimeInMillis((long) (time * 1000));
+ if (time != 0) {
+ wpt.setTimeInMillis((long) (time * 1000));
+ }
if (text != null) {
wpt.getExtensions().add("josm", "text", text);
} else if (dataProvider != null) {
Index: src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java b/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java
--- a/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java (working copy)
@@ -20,7 +20,9 @@
import java.util.ArrayList;
import java.util.Collection;
import java.util.Comparator;
+import java.util.HashMap;
import java.util.List;
+import java.util.Map;
import java.util.Optional;
import javax.swing.AbstractAction;
@@ -37,6 +39,7 @@
import org.openstreetmap.josm.data.gpx.GpxData;
import org.openstreetmap.josm.data.gpx.GpxExtension;
import org.openstreetmap.josm.data.gpx.GpxLink;
+import org.openstreetmap.josm.data.gpx.IGpxLayerPrefs;
import org.openstreetmap.josm.data.gpx.WayPoint;
import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
import org.openstreetmap.josm.data.preferences.IntegerProperty;
@@ -56,6 +59,7 @@
import org.openstreetmap.josm.gui.preferences.display.GPXSettingsPanel;
import org.openstreetmap.josm.io.audio.AudioPlayer;
import org.openstreetmap.josm.spi.preferences.Config;
+import org.openstreetmap.josm.tools.ColorHelper;
import org.openstreetmap.josm.tools.ImageProvider;
import org.openstreetmap.josm.tools.Logging;
import org.openstreetmap.josm.tools.Utils;
@@ -76,7 +80,7 @@
/**
* A list of markers.
*/
- public final List data;
+ public final MarkerData data;
private boolean mousePressed;
public GpxLayer fromLayer;
private Marker currentMarker;
@@ -100,17 +104,20 @@
public MarkerLayer(GpxData indata, String name, File associatedFile, GpxLayer fromLayer) {
super(name);
this.setAssociatedFile(associatedFile);
- this.data = new ArrayList<>();
+ this.data = new MarkerData();
this.fromLayer = fromLayer;
double firstTime = -1.0;
String lastLinkedFile = "";
+ if (fromLayer == null || fromLayer.data == null) {
+ data.ownLayerPrefs = indata.getLayerPrefs();
+ }
+
+ String cs = GPXSettingsPanel.tryGetDataPrefLocal(data, "markers.color");
Color c = null;
- String cs = GPXSettingsPanel.tryGetLayerPrefLocal(indata, "markers.color");
if (cs != null) {
- try {
- c = Color.decode(cs);
- } catch (NumberFormatException ex) {
+ c = ColorHelper.html2color(cs);
+ if (c == null) {
Logging.warn("Could not read marker color: " + cs);
}
}
@@ -459,7 +466,7 @@
* @return true if text should be shown, false otherwise.
*/
private boolean isTextOrIconShown() {
- return Boolean.parseBoolean(GPXSettingsPanel.getLayerPref(fromLayer, "markers.show-text"));
+ return Boolean.parseBoolean(GPXSettingsPanel.getDataPref(data, "markers.show-text"));
}
@Override
@@ -475,13 +482,11 @@
@Override
public void setColor(Color color) {
setPrivateColors(color);
- if (fromLayer != null) {
- String cs = null;
- if (color != null) {
- cs = String.format("#%02X%02X%02X", color.getRed(), color.getGreen(), color.getBlue());
- }
- GPXSettingsPanel.putLayerPrefLocal(fromLayer, "markers.color", cs);
+ String cs = null;
+ if (color != null) {
+ cs = ColorHelper.color2html(color);
}
+ GPXSettingsPanel.putDataPrefLocal(data, "markers.color", cs);
invalidate();
}
@@ -533,7 +538,7 @@
@Override
public void actionPerformed(ActionEvent e) {
- GPXSettingsPanel.putLayerPrefLocal(layer.fromLayer, "markers.show-text", Boolean.toString(!layer.isTextOrIconShown()));
+ GPXSettingsPanel.putDataPrefLocal(layer.data, "markers.show-text", Boolean.toString(!layer.isTextOrIconShown()));
layer.invalidate();
}
@@ -617,4 +622,41 @@
invalidate();
}
}
+
+ /**
+ * the data of a MarkerLayer
+ */
+ public class MarkerData extends ArrayList implements IGpxLayerPrefs {
+
+ private Map ownLayerPrefs;
+
+ @Override
+ public Map getLayerPrefs() {
+ if (ownLayerPrefs == null && fromLayer != null && fromLayer.data != null) {
+ return fromLayer.data.getLayerPrefs();
+ }
+ // fallback to own layerPrefs if the corresponding gpxLayer has already been deleted
+ // by the user or never existed when loaded from a session file
+ if (ownLayerPrefs == null) {
+ ownLayerPrefs = new HashMap<>();
+ }
+ return ownLayerPrefs;
+ }
+
+ /**
+ * Transfers the layerPrefs from the GpxData to MarkerData (when GpxData is deleted)
+ * @param gpxLayerPrefs the layerPrefs from the GpxData object
+ */
+ public void transferLayerPrefs(Map gpxLayerPrefs) {
+ ownLayerPrefs = new HashMap<>(gpxLayerPrefs);
+ }
+
+ @Override
+ public void setModified(boolean value) {
+ if (fromLayer != null && fromLayer.data != null) {
+ fromLayer.data.setModified(value);
+ }
+ }
+
+ }
}
Index: src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java b/trunk/src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java
--- a/trunk/src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java (working copy)
@@ -29,6 +29,7 @@
import org.apache.commons.jcs3.access.exception.InvalidArgumentException;
import org.openstreetmap.josm.actions.ExpertToggleAction;
import org.openstreetmap.josm.data.gpx.GpxData;
+import org.openstreetmap.josm.data.gpx.IGpxLayerPrefs;
import org.openstreetmap.josm.gui.MainApplication;
import org.openstreetmap.josm.gui.layer.GpxLayer;
import org.openstreetmap.josm.gui.layer.gpx.GpxDrawHelper;
@@ -172,6 +173,17 @@
* @return the value
*/
public static String getLayerPref(GpxLayer layer, String key) {
+ GpxData data = layer != null ? layer.data : null;
+ return getDataPref(data, key);
+ }
+
+ /**
+ * Reads the preference for the given layer or the default preference if not available
+ * @param data the data. Can be null, default preference will be returned then
+ * @param key the drawing key to be read, without "draw.rawgps."
+ * @return the value
+ */
+ public static String getDataPref(IGpxLayerPrefs data, String key) {
Object d = DEFAULT_PREFS.get(key);
String ds;
if (d != null) {
@@ -180,7 +192,7 @@
Logging.warn("No default value found for layer preference \"" + key + "\".");
ds = null;
}
- return Optional.ofNullable(tryGetLayerPrefLocal(layer, key)).orElse(Config.getPref().get("draw.rawgps." + key, ds));
+ return Optional.ofNullable(tryGetDataPrefLocal(data, key)).orElse(Config.getPref().get("draw.rawgps." + key, ds));
}
/**
@@ -190,7 +202,18 @@
* @return the integer value
*/
public static int getLayerPrefInt(GpxLayer layer, String key) {
- String s = getLayerPref(layer, key);
+ GpxData data = layer != null ? layer.data : null;
+ return getDataPrefInt(data, key);
+ }
+
+ /**
+ * Reads the integer preference for the given data or the default preference if not available
+ * @param data the data. Can be null, default preference will be returned then
+ * @param key the drawing key to be read, without "draw.rawgps."
+ * @return the integer value
+ */
+ public static int getDataPrefInt(IGpxLayerPrefs data, String key) {
+ String s = getDataPref(data, key);
if (s != null) {
try {
return Integer.parseInt(s);
@@ -213,7 +236,7 @@
* @return the value or null if not found
*/
public static String tryGetLayerPrefLocal(GpxLayer layer, String key) {
- return layer != null ? tryGetLayerPrefLocal(layer.data, key) : null;
+ return layer != null ? tryGetDataPrefLocal(layer.data, key) : null;
}
/**
@@ -222,7 +245,7 @@
* @param key the drawing key to be read, without "draw.rawgps."
* @return the value or null if not found
*/
- public static String tryGetLayerPrefLocal(GpxData data, String key) {
+ public static String tryGetDataPrefLocal(IGpxLayerPrefs data, String key) {
return data != null ? data.getLayerPrefs().get(key) : null;
}
@@ -236,7 +259,7 @@
String v = value == null ? null : value.toString();
if (layers != null) {
for (GpxLayer l : layers) {
- putLayerPrefLocal(l.data, key, v);
+ putDataPrefLocal(l.data, key, v);
}
} else {
Config.getPref().put("draw.rawgps." + key, v);
@@ -251,7 +274,7 @@
*/
public static void putLayerPrefLocal(GpxLayer layer, String key, String value) {
if (layer == null) return;
- putLayerPrefLocal(layer.data, key, value);
+ putDataPrefLocal(layer.data, key, value);
}
/**
@@ -260,7 +283,9 @@
* @param key the drawing key to be written, without "draw.rawgps."
* @param value the value or null to remove key
*/
- public static void putLayerPrefLocal(GpxData data, String key, String value) {
+ public static void putDataPrefLocal(IGpxLayerPrefs data, String key, String value) {
+ if (data == null) return;
+ data.setModified(true);
if (value == null || value.trim().isEmpty() ||
(getLayerPref(null, key).equals(value) && DEFAULT_PREFS.get(key) != null && DEFAULT_PREFS.get(key).toString().equals(value))) {
data.getLayerPrefs().remove(key);
Index: src/org/openstreetmap/josm/io/GpxWriter.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/io/GpxWriter.java b/trunk/src/org/openstreetmap/josm/io/GpxWriter.java
--- a/trunk/src/org/openstreetmap/josm/io/GpxWriter.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/io/GpxWriter.java (working copy)
@@ -127,9 +127,13 @@
validprefixes = namespaces.stream().map(n -> n.getPrefix()).collect(Collectors.toList());
+ data.creator = JOSM_CREATOR_NAME;
out.println("");
- out.println(" {
+ if (k != null && k.indexOf("markers.") == 0) {
+ data.getLayerPrefs().put(k, v);
+ }
+ });
data.put(GpxData.META_DESC, "exported JOSM marker layer");
for (Marker m : layer.data) {
data.waypoints.add(m.convertToWayPoint());
Index: src/org/openstreetmap/josm/io/session/MarkerSessionImporter.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/io/session/MarkerSessionImporter.java b/trunk/src/org/openstreetmap/josm/io/session/MarkerSessionImporter.java
--- a/trunk/src/org/openstreetmap/josm/io/session/MarkerSessionImporter.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/io/session/MarkerSessionImporter.java (working copy)
@@ -5,7 +5,6 @@
import java.io.IOException;
import java.io.InputStream;
-import java.util.List;
import javax.xml.xpath.XPath;
import javax.xml.xpath.XPathConstants;
@@ -14,7 +13,6 @@
import javax.xml.xpath.XPathFactory;
import org.openstreetmap.josm.gui.io.importexport.GpxImporter;
-import org.openstreetmap.josm.gui.layer.GpxLayer;
import org.openstreetmap.josm.gui.layer.Layer;
import org.openstreetmap.josm.gui.layer.markerlayer.MarkerLayer;
import org.openstreetmap.josm.gui.progress.ProgressMonitor;
@@ -49,21 +47,7 @@
support.addPostLayersTask(importData.getPostLayerTask());
- GpxLayer gpxLayer = null;
- List deps = support.getLayerDependencies();
- if (!deps.isEmpty()) {
- Layer layer = deps.get(0).getLayer();
- if (layer instanceof GpxLayer) {
- gpxLayer = (GpxLayer) layer;
- }
- }
-
- MarkerLayer markerLayer = importData.getMarkerLayer();
- if (markerLayer != null) {
- markerLayer.fromLayer = gpxLayer;
- }
-
- return markerLayer;
+ return importData.getMarkerLayer();
}
} catch (XPathExpressionException e) {
throw new IllegalDataException(e);
Index: src/org/openstreetmap/josm/io/session/SessionReader.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/io/session/SessionReader.java b/trunk/src/org/openstreetmap/josm/io/session/SessionReader.java
--- a/trunk/src/org/openstreetmap/josm/io/session/SessionReader.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/io/session/SessionReader.java (working copy)
@@ -3,7 +3,6 @@
import static org.openstreetmap.josm.tools.I18n.tr;
-import java.awt.Color;
import java.awt.GraphicsEnvironment;
import java.io.BufferedInputStream;
import java.io.File;
@@ -46,7 +45,6 @@
import org.openstreetmap.josm.io.Compression;
import org.openstreetmap.josm.io.IllegalDataException;
import org.openstreetmap.josm.tools.CheckParameterUtil;
-import org.openstreetmap.josm.tools.ColorHelper;
import org.openstreetmap.josm.tools.JosmRuntimeException;
import org.openstreetmap.josm.tools.Logging;
import org.openstreetmap.josm.tools.MultiMap;
@@ -619,15 +617,6 @@
Logging.warn(ex);
}
}
- String colorString = el.getAttribute("color");
- if (colorString != null) {
- try {
- Color color = ColorHelper.html2color(colorString);
- layer.setColor(color);
- } catch (RuntimeException ex) {
- Logging.warn("Cannot parse color " + colorString);
- }
- }
layer.setName(names.get(entry.getKey()));
layers.add(layer);
}
Index: src/org/openstreetmap/josm/io/session/SessionWriter.java
===================================================================
diff --git a/trunk/src/org/openstreetmap/josm/io/session/SessionWriter.java b/trunk/src/org/openstreetmap/josm/io/session/SessionWriter.java
--- a/trunk/src/org/openstreetmap/josm/io/session/SessionWriter.java (revision 17921)
+++ b/trunk/src/org/openstreetmap/josm/io/session/SessionWriter.java (working copy)
@@ -43,7 +43,6 @@
import org.openstreetmap.josm.gui.layer.geoimage.GeoImageLayer;
import org.openstreetmap.josm.gui.layer.markerlayer.MarkerLayer;
import org.openstreetmap.josm.gui.preferences.projection.ProjectionPreference;
-import org.openstreetmap.josm.tools.ColorHelper;
import org.openstreetmap.josm.tools.JosmRuntimeException;
import org.openstreetmap.josm.tools.Logging;
import org.openstreetmap.josm.tools.MultiMap;
@@ -240,9 +239,6 @@
if (!Utils.equalsEpsilon(layer.getOpacity(), 1.0)) {
el.setAttribute("opacity", Double.toString(layer.getOpacity()));
}
- if (layer.getColor() != null) {
- el.setAttribute("color", ColorHelper.color2html(layer.getColor()));
- }
Set deps = dependencies.get(layer);
final String depends = deps == null ? "" : deps.stream().map(depLayer -> {
int depIndex = layers.indexOf(depLayer);
Index: test/data/sessions/data_export.gpx
===================================================================
diff --git a/trunk/test/data/sessions/data_export.gpx b/trunk/test/data/sessions/data_export.gpx
new file mode 100644
--- a/trunk/test/data/sessions/data_export.gpx (nonexistent)
+++ b/trunk/test/data/sessions/data_export.gpx (working copy)
@@ -0,0 +1,18 @@
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
\ No newline at end of file
Index: test/data/sessions/gpx_markers.jos
===================================================================
diff --git a/trunk/test/data/sessions/gpx_markers.jos b/trunk/test/data/sessions/gpx_markers.jos
--- a/trunk/test/data/sessions/gpx_markers.jos (revision 17921)
+++ b/trunk/test/data/sessions/gpx_markers.jos (working copy)
@@ -15,7 +15,7 @@
layers/01/data.gpx
-
+
layers/02/data.gpx
Index: test/data/sessions/gpx_markers.joz
===================================================================
diff --git a/trunk/test/data/sessions/gpx_markers.joz b/trunk/test/data/sessions/gpx_markers.joz
GIT binary patch
literal 1636
zc$^FHW@Zs#;Nak3=q|eu#DD~ZfH)_yGPS5!-@s5mC9xz?FTJ2*Zji74V*`;r${*!*
z-Y;DKWAPSV7rz~Ey|gbVF$&$9ZC*J=XQS-Rwk7}Ta@(hgt#uIiIjQpVv$HnQbNW{~
zpHGO*F0p*va#rBQEt9oz(>LFq{r=|5H+!t?gkIO(xoDs+v-noimt)g@JYWCX`^)23
zdSLByb+GV4_p5nw|RL?@#-tTUqbrNROYXrmTsX~s4l78^R&C;L}CBtD8CbO>c>m8
ze`kGT<&5YFQ{fI5dv~zyIotZDx{O=4OMl^*E#$v0HX=V=$L;*4>L#Ij>qwpOpPKXE
zFKZ3^HE+j){5Kp`8;lLV$nEVcn~|;kB$`#!{&eKA3Jdcd(U*V3Os1-RRQMi$cgMBE
zbL(8gk6aJ<%N*d%&awE}Bf0sE3=9XD85jbHie4i`^oIE6&jCiS`>Fa3cMi|naQ1ke
zh-6XrQmsH`)v1f_#Z)MkFe=~Xe6xT3*=eyJx6dxSb8pYbJs)qh#eXerjdOSzWOnk9
zdQRhllPPg#uA6?8{XcnIXWgaK+;Lm}Hn!D1N}i%A;I@jT{%2Krv)z3=g%{B)dECM_
zUF57YTGp%ic2}Ro%~MXD$2Y0ZT(eR&c>b>+pO|juFF0<$bRB7jV0Pgi$
zZY?^Q5VPuZ1pnOZnViNfU(;A#__lrashYdE-0gGRze?kOa`*nPD>g2C_2{vw#6)i8
zOSLopwAbVn^}gD)vcf)rh?(&gCl)XIeYwG(UgNqF<+&mTz8L
zym*`PRzcGx?-C_1ut}|ugX8c^+pS!2w?&eS#w!#zpimkTvo#Zw4k0=mNsQXnP
z)iF)Q=DJhCtwRC53wc_9i|(?x91lt(SwEbnKLMr@OICbogd2#9Q;Ul;^Yiqw@{5;F
zI+%UPK;r28L;u<*RL@@aYR#l;lPjb+cE6dmbiUp!Gu6)6zwf8J-CdQOyupG0es%wG
z{rFSUVwVPRAM=WRKI0gtTL5?LsjBIZ7vFZ1j1yDdc70oy&JDY7qW(!;-R**(+P4bb
zlg_=9$~UXwR-QiZMNT`sYEZEKXXsY%;G
z^vA9hFYRwOe%j+Xr{(j^v?qKmrf%;YE%>-7!hhs%@cB6Tz5AVR-6s}D53fs5ikyDo
zXKF%7^&A=d!eWl>_h;h{d6>s|l)svv(>2lB)Gn-b#?kHXJEJn6uKu*o*!{rw@OxgY
zM`w1bOqp`$W5xB79+u=CEdsv;&GsHzXRdF)B;721hs7Uu-M`wmQa9D5r2Lp&{ScgM
zJ0+!+BY`m@2hO#OOd<@3I6=JI+-8?}x=pL~q_m@G-m4=(C5zboEg)P0xtmpRb1rwl3U#`k{^b
z`(mdXYc#n(C~rAAwMVwAgSo}$s^0X-H#O_#-R(5?ZeyMndRDSoB#I#rK^tU_Gm6EuwY=2`MARH=T(ol
z%1e#k2s(YN`nb*K`=Td#dc9ALLo+??~-4
z-qwPuwFZBk6*(sUk7s?qjp?v>|0VIe?_b@s{>d5u5B3+pVE;TP&cy;4>`iFFZUhhZ
zxf7zj4;zRazt8@guYJ##l{?Smma4wJCz9fPkn`~N8^`o^sHt{ttE;Y_xv9%Q;N0bt
z@9QlezFuZ?sC|o{%M44-`K&9M$l||y_6Mto<(E12S3P5Q<}qfOIrWhA>;=1j
zAJ@HO_r}p|>P9)o?3J-ieD}f)6-#52^R;H{${f~qER
z>t9lw^88Zj%YwCM7Vh6K^36BOqRT(i@@1p&pQL-G?-qpo(x_>AW-N1dTDRyc^Vs_n
zlDrmIZWlZ&xjBYswy~O1N`{Nu)QZ^3A4_FU?>k))dTPpAC8?lm=lU+~m@l^Y{G2c6
zogK1&E-$}c6Z3ojy*fN*IAHv=QfSD;F;=B1N1<{efLaQN==m%U`J?1tG(BsUnBt-7*N`{?uz
zHfipT?Ekfl>RY2%1}#-GymR;QyT%9Y3okBle?2AB`^0q~j%}B&tg@Y2@bP#Ke|y%^
z`@Pvl?RrVZ=dM|Xlxqu#y%()3Irb}iPLWV_|Fn#U%F~*8kyvhjt629~!p1j~j9jF0b2z`V-rf*9W5q40_6yt|+EO|XUp}4clhV`pVQ^+RN{iXvGVNZN9W;@udh~sVPslh{dh(zk484OXo;NQ(buNjhyoS-+O*c
z(#_j+{?NS3zt8SlCLHwBP%1C~oA052i{t-<{+Bx1{=DR;rqd(g0B=Snc?Mjig$e@%
zC_rh;7Ff3`j4-`R3B&?7`f|fu6yjj^mnwWu55-6y{0^$Jxc|SI;
Index: test/data/sessions/markers.gpx
===================================================================
diff --git a/trunk/test/data/sessions/markers.gpx b/trunk/test/data/sessions/markers.gpx
new file mode 100644
--- a/trunk/test/data/sessions/markers.gpx (nonexistent)
+++ b/trunk/test/data/sessions/markers.gpx (working copy)
@@ -0,0 +1,19 @@
+
+
+
+ exported JOSM marker layer
+
+
+
+
+
+
+
+
+
+
+
+
\ No newline at end of file
Index: test/unit/org/openstreetmap/josm/data/gpx/GpxTrackTest.java
===================================================================
diff --git a/trunk/test/unit/org/openstreetmap/josm/data/gpx/GpxTrackTest.java b/trunk/test/unit/org/openstreetmap/josm/data/gpx/GpxTrackTest.java
--- a/trunk/test/unit/org/openstreetmap/josm/data/gpx/GpxTrackTest.java (revision 17921)
+++ b/trunk/test/unit/org/openstreetmap/josm/data/gpx/GpxTrackTest.java (working copy)
@@ -8,8 +8,8 @@
import java.util.ArrayList;
import java.util.HashMap;
+import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.extension.RegisterExtension;
-import org.junit.jupiter.api.Test;
import org.openstreetmap.josm.TestUtils;
import org.openstreetmap.josm.testutils.JOSMTestRules;
import org.openstreetmap.josm.tools.ListenerList;
@@ -31,7 +31,7 @@
public JOSMTestRules test = new JOSMTestRules();
/**
- * Tests weather the track can read and write colors.
+ * Tests whether the track can read and write colors.
*/
@Test
void testColors() {
Index: test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java
===================================================================
diff --git a/trunk/test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java b/trunk/test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java
--- a/trunk/test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java (revision 17921)
+++ b/trunk/test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java (working copy)
@@ -1,19 +1,24 @@
// License: GPL. For details, see LICENSE file.
package org.openstreetmap.josm.io.session;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.fail;
+
import java.awt.Color;
import java.io.File;
import java.io.IOException;
-import java.io.InputStream;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
+import java.time.Instant;
import java.util.Arrays;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
+import java.util.stream.Collectors;
+import java.util.zip.ZipEntry;
import java.util.zip.ZipFile;
import org.junit.jupiter.api.BeforeEach;
@@ -42,8 +47,6 @@
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-import static org.junit.jupiter.api.Assertions.assertEquals;
-
/**
* Unit tests for Session writing.
*/
@@ -108,7 +111,7 @@
MainApplication.getLayerManager().addLayer(createOsmLayer());
}
- private byte[] testWrite(List layers, final boolean zip) throws IOException {
+ private Map testWrite(List layers, final boolean zip) throws IOException {
Map exporters = new HashMap<>();
if (zip) {
SessionWriter.registerSessionLayerExporter(OsmDataLayer.class, OsmHeadlessJozExporter.class);
@@ -127,9 +130,15 @@
if (!zip) {
return null;
}
- try (ZipFile zipFile = new ZipFile(file);
- InputStream input = zipFile.getInputStream(zipFile.getEntry("session.jos"))) {
- return Utils.readBytesFromStream(input);
+ try (ZipFile zipFile = new ZipFile(file)) {
+ return Collections.list(zipFile.entries()).stream().collect(Collectors.toMap(ZipEntry::getName, e -> {
+ try {
+ return Utils.readBytesFromStream(zipFile.getInputStream(e));
+ } catch (IOException ex) {
+ fail(ex);
+ }
+ return null;
+ }));
}
} finally {
if (file.exists()) {
@@ -146,7 +155,9 @@
private GpxLayer createGpxLayer() {
GpxData data = new GpxData();
- data.waypoints.add(new WayPoint(new LatLon(42.72665, -0.00747)));
+ WayPoint wp = new WayPoint(new LatLon(42.72665, -0.00747));
+ wp.setInstant(Instant.parse("2021-01-01T10:15:30.00Z"));
+ data.waypoints.add(wp);
data.waypoints.add(new WayPoint(new LatLon(42.72659, -0.00749)));
GpxLayer layer = new GpxLayer(data, "GPX layer name");
layer.setAssociatedFile(new File("data.gpx"));
@@ -232,11 +243,22 @@
@Test
void testWriteGpxAndMarkerJoz() throws IOException {
GpxLayer gpx = createGpxLayer();
- byte[] bytes = testWrite(Arrays.asList(gpx, createMarkerLayer(gpx)), true);
+ Map bytes = testWrite(Arrays.asList(gpx, createMarkerLayer(gpx)), true);
+
Path path = Paths.get(TestUtils.getTestDataRoot() + "/sessions/gpx_markers.jos");
String expected = new String(Files.readAllBytes(path), StandardCharsets.UTF_8).replace("\r", "");
- String actual = new String(bytes, StandardCharsets.UTF_8).replace("\r", "");
+ String actual = new String(bytes.get("session.jos"), StandardCharsets.UTF_8).replace("\r", "");
assertEquals(expected, actual);
+
+ path = Paths.get(TestUtils.getTestDataRoot() + "/sessions/data_export.gpx");
+ expected = new String(Files.readAllBytes(path), StandardCharsets.UTF_8).replace("\r", "");
+ actual = new String(bytes.get("layers/01/data.gpx"), StandardCharsets.UTF_8).replace("\r", "");
+ assertEquals(expected, actual);
+
+ path = Paths.get(TestUtils.getTestDataRoot() + "/sessions/markers.gpx");
+ expected = new String(Files.readAllBytes(path), StandardCharsets.UTF_8).replace("\r", "");
+ actual = new String(bytes.get("layers/02/data.gpx"), StandardCharsets.UTF_8).replace("\r", "");
+ assertEquals(expected, actual);
}
/**