X-Git-Url: https://gitweb.fperrin.net/?a=blobdiff_plain;f=tim%2Fprune%2Fsave%2FExifSaver.java;h=a5ae40295054d5eace532c3a542c53d99d510f07;hb=ca9bdb3916f9c39adbbf95d06ac95c21dafbb4e6;hp=1e597201d5f2842197531159ec3452f542fb428c;hpb=5625a1abadb5f2ca5f017fe7dbda1d5141cb637b;p=GpsPrune.git

diff --git a/tim/prune/save/ExifSaver.java b/tim/prune/save/ExifSaver.java
index 1e59720..a5ae402 100644
--- a/tim/prune/save/ExifSaver.java
+++ b/tim/prune/save/ExifSaver.java
@@ -21,6 +21,7 @@ import javax.swing.JTable;
 
 import tim.prune.ExternalTools;
 import tim.prune.I18nManager;
+import tim.prune.UpdateMessageBroker;
 import tim.prune.data.Altitude;
 import tim.prune.data.Coordinate;
 import tim.prune.data.DataPoint;
@@ -35,9 +36,11 @@ public class ExifSaver implements Runnable
 {
 	private Frame _parentFrame = null;
 	private JDialog _dialog = null;
+	private JButton _okButton = null;
 	private JCheckBox _overwriteCheckbox = null;
 	private JProgressBar _progressBar = null;
 	private PhotoTableModel _photoTableModel = null;
+	private boolean _saveCancelled = false;
 
 
 	// To preserve timestamps of file use parameter -P
@@ -72,6 +75,7 @@ public class ExifSaver implements Runnable
 	 * Save exif information to all photos in the list
 	 * whose coordinate information has changed since loading
 	 * @param inPhotoList list of photos to save
+	 * @return true if saved
 	 */
 	public boolean saveExifInformation(PhotoList inPhotoList)
 	{
@@ -166,19 +170,22 @@ public class ExifSaver implements Runnable
 		// Lower panel with ok and cancel buttons
 		JPanel buttonPanel = new JPanel();
 		buttonPanel.setLayout(new FlowLayout(FlowLayout.RIGHT));
-		JButton okButton = new JButton(I18nManager.getText("button.ok"));
-		okButton.addActionListener(new ActionListener() {
+		_okButton = new JButton(I18nManager.getText("button.ok"));
+		_okButton.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e)
 			{
+				// disable ok button
+				_okButton.setEnabled(false);
 				// start new thread to do save
 				new Thread(ExifSaver.this).start();
 			}
 		});
-		buttonPanel.add(okButton);
+		buttonPanel.add(_okButton);
 		JButton cancelButton = new JButton(I18nManager.getText("button.cancel"));
 		cancelButton.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e)
 			{
+				_saveCancelled = true;
 				_dialog.dispose();
 			}
 		});
@@ -208,6 +215,7 @@ public class ExifSaver implements Runnable
 	 */
 	public void run()
 	{
+		_saveCancelled = false;
 		PhotoTableEntry entry = null;
 		Photo photo = null;
 		int numPhotos = _photoTableModel.getRowCount();
@@ -220,7 +228,7 @@ public class ExifSaver implements Runnable
 		for (int i=0; i<numPhotos; i++)
 		{
 			entry = _photoTableModel.getPhotoTableEntry(i);
-			if (entry != null && entry.getSaveFlag())
+			if (entry != null && entry.getSaveFlag() && !_saveCancelled)
 			{
 				// Only look at photos which are selected and whose status has changed since load
 				photo = entry.getPhoto();
@@ -237,10 +245,9 @@ public class ExifSaver implements Runnable
 			_progressBar.setValue(i + 1);
 		}
 		_progressBar.setVisible(false);
-		// Show confirmation dialog
-		JOptionPane.showMessageDialog(_dialog, I18nManager.getText("dialog.saveexif.ok1") + " "
-			+ numSaved + " " + I18nManager.getText("dialog.saveexif.ok2"),
-			I18nManager.getText("dialog.saveexif.title"), JOptionPane.INFORMATION_MESSAGE);
+		// Show confirmation
+		UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.saveexif.ok1") + " "
+			+ numSaved + " " + I18nManager.getText("confirm.saveexif.ok2"));
 		// close dialog, all finished
 		_dialog.dispose();
 	}
@@ -297,7 +304,12 @@ public class ExifSaver implements Runnable
 		// Execute exif command
 		try
 		{
-			Runtime.getRuntime().exec(command);
+			Process process = Runtime.getRuntime().exec(command);
+			// Wait for process to finish so not too many run in parallel
+			try {
+				process.waitFor();
+			}
+			catch (InterruptedException ie) {}
 		}
 		catch (Exception e)
 		{