X-Git-Url: https://gitweb.fperrin.net/?a=blobdiff_plain;f=tim%2Fprune%2FApp.java;h=205d88ae11c74165c23bd6aa7ef1fab275af4485;hb=4d5796d02a15808311c09448d79e6e7d1de9d636;hp=091a37cb4a70bb1e8f149c1cb286a9098c7f26f3;hpb=112bb0c9b46894adca9a33ed8c99ea712b253185;p=GpsPrune.git

diff --git a/tim/prune/App.java b/tim/prune/App.java
index 091a37c..205d88a 100644
--- a/tim/prune/App.java
+++ b/tim/prune/App.java
@@ -1,35 +1,43 @@
 package tim.prune;
 
+import java.io.File;
 import java.util.ArrayList;
 import java.util.EmptyStackException;
 import java.util.Set;
 import java.util.Stack;
-import java.io.File;
 
 import javax.swing.JFrame;
 import javax.swing.JOptionPane;
 
+import tim.prune.config.Config;
 import tim.prune.data.Altitude;
-import tim.prune.data.Coordinate;
+import tim.prune.data.Checker;
 import tim.prune.data.DataPoint;
 import tim.prune.data.Field;
 import tim.prune.data.LatLonRectangle;
-import tim.prune.data.Latitude;
-import tim.prune.data.Longitude;
 import tim.prune.data.NumberUtils;
 import tim.prune.data.Photo;
 import tim.prune.data.PhotoList;
+import tim.prune.data.RecentFile;
+import tim.prune.data.SourceInfo;
 import tim.prune.data.Track;
 import tim.prune.data.TrackInfo;
+import tim.prune.data.SourceInfo.FILE_TYPE;
+import tim.prune.function.AsyncMediaLoader;
+import tim.prune.function.SaveConfig;
+import tim.prune.function.SelectTracksFunction;
 import tim.prune.function.browser.BrowserLauncher;
 import tim.prune.function.browser.UrlGenerator;
 import tim.prune.function.edit.FieldEditList;
 import tim.prune.function.edit.PointEditor;
 import tim.prune.gui.MenuManager;
+import tim.prune.gui.SidebarController;
 import tim.prune.gui.UndoManager;
 import tim.prune.gui.Viewport;
 import tim.prune.load.FileLoader;
 import tim.prune.load.JpegLoader;
+import tim.prune.load.MediaLinkInfo;
+import tim.prune.load.TrackNameList;
 import tim.prune.save.ExifSaver;
 import tim.prune.save.FileSaver;
 import tim.prune.undo.*;
@@ -46,6 +54,7 @@ public class App
 	private TrackInfo _trackInfo = null;
 	private int _lastSavePosition = 0;
 	private MenuManager _menuManager = null;
+	private SidebarController _sidebarController = null;
 	private FileLoader _fileLoader = null;
 	private JpegLoader _jpegLoader = null;
 	private FileSaver _fileSaver = null;
@@ -53,7 +62,12 @@ public class App
 	private boolean _mangleTimestampsConfirmed = false;
 	private Viewport _viewport = null;
 	private ArrayList<File> _dataFiles = null;
-	private boolean _firstDataFile = true;
+	private boolean _autoAppendNextFile = false;
+	private boolean _busyLoading = false;
+	private AppMode _appMode = AppMode.NORMAL;
+
+	/** Enum for the app mode - currently only two options but may expand later */
+	public enum AppMode {NORMAL, DRAWRECT};
 
 
 	/**
@@ -113,14 +127,15 @@ public class App
 		if (inDataFiles == null || inDataFiles.size() == 0) {
 			_dataFiles = null;
 		}
-		else {
+		else
+		{
 			_dataFiles = inDataFiles;
 			File f = _dataFiles.get(0);
 			_dataFiles.remove(0);
 			// Start load of specified file
 			if (_fileLoader == null)
 				_fileLoader = new FileLoader(this, _frame);
-			_firstDataFile = true;
+			_autoAppendNextFile = false; // prompt for append
 			_fileLoader.openFile(f);
 		}
 	}
@@ -134,6 +149,7 @@ public class App
 	{
 		_undoStack.add(inUndo);
 		UpdateMessageBroker.informSubscribers(inConfirmText);
+		setCurrentMode(AppMode.NORMAL);
 	}
 
 	/**
@@ -178,7 +194,7 @@ public class App
 		else
 		{
 			if (_fileSaver == null) {
-				_fileSaver = new FileSaver(this, _frame, _track);
+				_fileSaver = new FileSaver(this, _frame);
 			}
 			char delim = ',';
 			if (_fileLoader != null) {delim = _fileLoader.getLastUsedDelimiter();}
@@ -203,6 +219,10 @@ public class App
 				JOptionPane.WARNING_MESSAGE, null, buttonTexts, buttonTexts[1])
 			== JOptionPane.YES_OPTION)
 		{
+			// save settings
+			if (Config.getConfigBoolean(Config.KEY_AUTOSAVE_SETTINGS)) {
+				new SaveConfig(this).silentSave();
+			}
 			System.exit(0);
 		}
 	}
@@ -239,7 +259,7 @@ public class App
 			// add information to undo stack
 			UndoOperation undo = new UndoEditPoint(currentPoint, inUndoList);
 			// pass to track for completion
-			if (_track.editPoint(currentPoint, inEditList))
+			if (_track.editPoint(currentPoint, inEditList, false))
 			{
 				_undoStack.push(undo);
 				// Confirm point edit
@@ -258,13 +278,14 @@ public class App
 		DataPoint currentPoint = _trackInfo.getCurrentPoint();
 		if (currentPoint != null)
 		{
+			// Check for photo
 			boolean deletePhoto = false;
 			Photo currentPhoto = currentPoint.getPhoto();
 			if (currentPhoto != null)
 			{
 				// Confirm deletion of photo or decoupling
 				int response = JOptionPane.showConfirmDialog(_frame,
-					I18nManager.getText("dialog.deletepoint.deletephoto") + " " + currentPhoto.getFile().getName(),
+					I18nManager.getText("dialog.deletepoint.deletephoto") + " " + currentPhoto.getName(),
 					I18nManager.getText("dialog.deletepoint.title"),
 					JOptionPane.YES_NO_CANCEL_OPTION);
 				if (response == JOptionPane.CANCEL_OPTION || response == JOptionPane.CLOSED_OPTION)
@@ -277,10 +298,11 @@ public class App
 			// store necessary information to undo it later
 			int pointIndex = _trackInfo.getSelection().getCurrentPointIndex();
 			int photoIndex = _trackInfo.getPhotoList().getPhotoIndex(currentPhoto);
+			int audioIndex = _trackInfo.getAudioList().getAudioIndex(currentPoint.getAudio());
 			DataPoint nextTrackPoint = _trackInfo.getTrack().getNextTrackPoint(pointIndex + 1);
 			// Construct Undo object
 			UndoOperation undo = new UndoDeletePoint(pointIndex, currentPoint, photoIndex,
-				nextTrackPoint != null && nextTrackPoint.getSegmentStart());
+				audioIndex, nextTrackPoint != null && nextTrackPoint.getSegmentStart());
 			// call track to delete point
 			if (_trackInfo.deletePoint())
 			{
@@ -298,95 +320,15 @@ public class App
 						// decouple photo from point
 						currentPhoto.setDataPoint(null);
 					}
+					UpdateMessageBroker.informSubscribers(DataSubscriber.PHOTOS_MODIFIED);
+				}
+				// Delete audio object (without bothering to ask)
+				if (audioIndex > -1) {
+					_trackInfo.getAudioList().deleteAudio(audioIndex);
 				}
 				// Confirm
 				UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.deletepoint.single"));
-			}
-		}
-	}
-
-
-	/**
-	 * Delete the currently selected range
-	 */
-	public void deleteSelectedRange()
-	{
-		if (_track != null)
-		{
-			// Find out if photos should be deleted or not
-			int selStart = _trackInfo.getSelection().getStart();
-			int selEnd = _trackInfo.getSelection().getEnd();
-			if (selStart >= 0 && selEnd >= selStart)
-			{
-				int numToDelete = selEnd - selStart + 1;
-				boolean[] deletePhotos = new boolean[numToDelete];
-				Photo[] photosToDelete = new Photo[numToDelete];
-				boolean deleteAll = false;
-				boolean deleteNone = false;
-				String[] questionOptions = {I18nManager.getText("button.yes"), I18nManager.getText("button.no"),
-					I18nManager.getText("button.yestoall"), I18nManager.getText("button.notoall"),
-					I18nManager.getText("button.cancel")};
-				DataPoint point = null;
-				for (int i=0; i<numToDelete; i++)
-				{
-					point = _trackInfo.getTrack().getPoint(i + selStart);
-					if (point != null && point.getPhoto() != null)
-					{
-						if (deleteAll)
-						{
-							deletePhotos[i] = true;
-							photosToDelete[i] = point.getPhoto();
-						}
-						else if (deleteNone) {deletePhotos[i] = false;}
-						else
-						{
-							int response = JOptionPane.showOptionDialog(_frame,
-								I18nManager.getText("dialog.deletepoint.deletephoto") + " " + point.getPhoto().getFile().getName(),
-								I18nManager.getText("dialog.deletepoint.title"),
-								JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.QUESTION_MESSAGE, null,
-								questionOptions, questionOptions[1]);
-							// check for cancel or close
-							if (response == 4 || response == -1) {return;}
-							// check for yes or yes to all
-							if (response == 0 || response == 2)
-							{
-								deletePhotos[i] = true;
-								photosToDelete[i] = point.getPhoto();
-								if (response == 2) {deleteAll = true;}
-							}
-							// check for no to all
-							if (response == 3) {deleteNone = true;}
-						}
-					}
-				}
-				// add information to undo stack
-				UndoDeleteRange undo = new UndoDeleteRange(_trackInfo);
-				// delete requested photos
-				for (int i=0; i<numToDelete; i++)
-				{
-					point = _trackInfo.getTrack().getPoint(i + selStart);
-					if (point != null && point.getPhoto() != null)
-					{
-						if (deletePhotos[i])
-						{
-							// delete photo from list
-							_trackInfo.getPhotoList().deletePhoto(_trackInfo.getPhotoList().getPhotoIndex(point.getPhoto()));
-						}
-						else
-						{
-							// decouple from point
-							point.getPhoto().setDataPoint(null);
-						}
-					}
-				}
-				// call track to delete range
-				if (_trackInfo.deleteRange())
-				{
-					_undoStack.push(undo);
-					// Confirm
-					UpdateMessageBroker.informSubscribers("" + numToDelete + " "
-						+ I18nManager.getText("confirm.deletepoint.multi"));
-				}
+				UpdateMessageBroker.informSubscribers(DataSubscriber.DATA_ADDED_OR_REMOVED);
 			}
 		}
 	}
@@ -397,7 +339,7 @@ public class App
 	 */
 	public void finishCompressTrack()
 	{
-		UndoCompress undo = new UndoCompress(_track);
+		UndoDeleteMarked undo = new UndoDeleteMarked(_track);
 		// call track to do compress
 		int numPointsDeleted = _trackInfo.deleteMarkedPoints();
 		// add to undo stack if successful
@@ -409,7 +351,7 @@ public class App
 				 + (numPointsDeleted==1?I18nManager.getText("confirm.deletepoint.single"):I18nManager.getText("confirm.deletepoint.multi")));
 		}
 		else {
-			showErrorMessage("function.compress", "dialog.compress.nonefound");
+			showErrorMessage("function.compress", "dialog.deletemarked.nonefound");
 		}
 	}
 
@@ -449,7 +391,7 @@ public class App
 		int selStart = _trackInfo.getSelection().getStart();
 		int selEnd = _trackInfo.getSelection().getEnd();
 		UndoAddTimeOffset undo = new UndoAddTimeOffset(selStart, selEnd, inTimeOffset);
-		if (_trackInfo.getTrack().addTimeOffset(selStart, selEnd, inTimeOffset))
+		if (_trackInfo.getTrack().addTimeOffset(selStart, selEnd, inTimeOffset, false))
 		{
 			_undoStack.add(undo);
 			UpdateMessageBroker.informSubscribers(DataSubscriber.DATA_EDITED);
@@ -485,6 +427,7 @@ public class App
 		if (success)
 		{
 			_undoStack.add(undo);
+			_trackInfo.getSelection().markInvalid();
 			UpdateMessageBroker.informSubscribers(DataSubscriber.DATA_EDITED);
 			UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.addaltitudeoffset"));
 		}
@@ -514,29 +457,6 @@ public class App
 	}
 
 
-	/**
-	 * Interpolate the two selected points
-	 */
-	public void interpolateSelection()
-	{
-		// Get number of points to add
-		Object numPointsStr = JOptionPane.showInputDialog(_frame,
-			I18nManager.getText("dialog.interpolate.parameter.text"),
-			I18nManager.getText("dialog.interpolate.title"),
-			JOptionPane.QUESTION_MESSAGE, null, null, "");
-		int numPoints = parseNumber(numPointsStr);
-		if (numPoints <= 0) return;
-
-		UndoInsert undo = new UndoInsert(_trackInfo.getSelection().getStart() + 1,
-			numPoints);
-		// call track to interpolate
-		if (_trackInfo.interpolate(numPoints))
-		{
-			_undoStack.add(undo);
-		}
-	}
-
-
 	/**
 	 * Average the selected points
 	 */
@@ -556,21 +476,40 @@ public class App
 
 
 	/**
-	 * Create a new point at the given lat/long coordinates
-	 * @param inLat latitude
-	 * @param inLong longitude
+	 * Create a new point at the end of the track
+	 * @param inPoint point to add
 	 */
-	public void createPoint(double inLat, double inLong)
+	public void createPoint(DataPoint inPoint)
 	{
 		// create undo object
 		UndoCreatePoint undo = new UndoCreatePoint();
-		// create point and add to track
-		DataPoint point = new DataPoint(new Latitude(inLat, Coordinate.FORMAT_NONE), new Longitude(inLong, Coordinate.FORMAT_NONE), null);
-		point.setSegmentStart(true);
-		_track.appendPoints(new DataPoint[] {point});
+		_undoStack.add(undo);
+		// add point to track
+		inPoint.setSegmentStart(true);
+		_track.appendPoints(new DataPoint[] {inPoint});
+		// ensure track's field list contains point's fields
+		_track.extendFieldList(inPoint.getFieldList());
 		_trackInfo.selectPoint(_trackInfo.getTrack().getNumPoints()-1);
-		// add undo object to stack
+		// update listeners
+		UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.createpoint"));
+	}
+
+
+	/**
+	 * Create a new point before the given position
+	 * @param inPoint point to add
+	 * @param inIndex index of following point
+	 */
+	public void createPoint(DataPoint inPoint, int inIndex)
+	{
+		// create undo object
+		UndoInsert undo = new UndoInsert(inIndex, 1);
 		_undoStack.add(undo);
+		// add point to track
+		_track.insertPoint(inPoint, inIndex);
+		// ensure track's field list contains point's fields
+		_track.extendFieldList(inPoint.getFieldList());
+		_trackInfo.selectPoint(inIndex);
 		// update listeners
 		UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.createpoint"));
 	}
@@ -626,14 +565,33 @@ public class App
 	}
 
 	/**
-	 * Receive loaded data and optionally merge with current Track
+	 * Receive loaded data and determine whether to filter on tracks or not
 	 * @param inFieldArray array of fields
 	 * @param inDataArray array of data
 	 * @param inAltFormat altitude format
-	 * @param inFilename filename used
+	 * @param inSourceInfo information about the source of the data
+	 * @param inTrackNameList information about the track names
 	 */
-	public void informDataLoaded(Field[] inFieldArray, Object[][] inDataArray, Altitude.Format inAltFormat,
-		String inFilename)
+	public void informDataLoaded(Field[] inFieldArray, Object[][] inDataArray,
+		Altitude.Format inAltFormat, SourceInfo inSourceInfo, TrackNameList inTrackNameList)
+	{
+		// no link array given
+		informDataLoaded(inFieldArray, inDataArray, inAltFormat, inSourceInfo,
+			inTrackNameList, null);
+	}
+
+	/**
+	 * Receive loaded data and determine whether to filter on tracks or not
+	 * @param inFieldArray array of fields
+	 * @param inDataArray array of data
+	 * @param inAltFormat altitude format
+	 * @param inSourceInfo information about the source of the data
+	 * @param inTrackNameList information about the track names
+	 * @param inLinkInfo links to photo/audio clips
+	 */
+	public void informDataLoaded(Field[] inFieldArray, Object[][] inDataArray,
+		Altitude.Format inAltFormat, SourceInfo inSourceInfo,
+		TrackNameList inTrackNameList, MediaLinkInfo inLinkInfo)
 	{
 		// Check whether loaded array can be properly parsed into a Track
 		Track loadedTrack = new Track();
@@ -645,69 +603,113 @@ public class App
 			loadNextFile();
 			return;
 		}
+		// Check for doubled track
+		if (Checker.isDoubledTrack(loadedTrack)) {
+			JOptionPane.showMessageDialog(_frame, I18nManager.getText("dialog.open.contentsdoubled"),
+				I18nManager.getText("function.open"), JOptionPane.WARNING_MESSAGE);
+		}
+
+		_busyLoading = true;
+		// Attach photos and/or audio clips to points
+		if (inLinkInfo != null)
+		{
+			String[] linkArray = inLinkInfo.getLinkArray();
+			if (linkArray != null) {
+				new AsyncMediaLoader(this, inLinkInfo.getZipFile(), linkArray, loadedTrack, inSourceInfo.getFile()).begin();
+			}
+		}
+		// Look at TrackNameList, decide whether to filter or not
+		if (inTrackNameList != null && inTrackNameList.getNumTracks() > 1)
+		{
+			// Launch a dialog to let the user choose which tracks to load, then continue
+			new SelectTracksFunction(this, loadedTrack, inSourceInfo, inTrackNameList).begin();
+		}
+		else {
+			// go directly to load
+			informDataLoaded(loadedTrack, inSourceInfo);
+		}
+		setCurrentMode(AppMode.NORMAL);
+	}
+
+
+	/**
+	 * Receive loaded data and optionally merge with current Track
+	 * @param inLoadedTrack loaded track
+	 * @param inSourceInfo information about the source of the data
+	 */
+	public void informDataLoaded(Track inLoadedTrack, SourceInfo inSourceInfo)
+	{
 		// Decide whether to load or append
 		if (_track.getNumPoints() > 0)
 		{
 			// ask whether to replace or append
 			int answer = 0;
-			if (_dataFiles == null || _firstDataFile) {
+			if (_autoAppendNextFile) {
+				// Automatically append the next file
+				answer = JOptionPane.YES_OPTION;
+			}
+			else {
+				// Ask whether to append or not
 				answer = JOptionPane.showConfirmDialog(_frame,
 					I18nManager.getText("dialog.openappend.text"),
 					I18nManager.getText("dialog.openappend.title"),
 					JOptionPane.YES_NO_CANCEL_OPTION);
 			}
-			else {
-				// Automatically append if there's a file load queue
-				answer = JOptionPane.YES_OPTION;
-			}
+			_autoAppendNextFile = false; // reset flag to cancel autoappend
+
 			if (answer == JOptionPane.YES_OPTION)
 			{
 				// append data to current Track
-				_undoStack.add(new UndoLoad(_track.getNumPoints(), loadedTrack.getNumPoints()));
-				_track.combine(loadedTrack);
-				// set filename if currently empty
-				if (_trackInfo.getFileInfo().getNumFiles() == 0)
-				{
-					_trackInfo.getFileInfo().setFile(inFilename);
-				}
-				else
-				{
-					_trackInfo.getFileInfo().addFile();
-				}
+				UndoLoad undo = new UndoLoad(_track.getNumPoints(), inLoadedTrack.getNumPoints());
+				undo.setNumPhotosAudios(_trackInfo.getPhotoList().getNumPhotos(), _trackInfo.getAudioList().getNumAudios());
+				_undoStack.add(undo);
+				_track.combine(inLoadedTrack);
+				// set source information
+				inSourceInfo.populatePointObjects(_track, inLoadedTrack.getNumPoints());
+				_trackInfo.getFileInfo().addSource(inSourceInfo);
 			}
 			else if (answer == JOptionPane.NO_OPTION)
 			{
 				// Don't append, replace data
 				PhotoList photos = null;
-				if (_trackInfo.getPhotoList().hasCorrelatedPhotos())
-				{
+				if (_trackInfo.getPhotoList().hasCorrelatedPhotos()) {
 					photos = _trackInfo.getPhotoList().cloneList();
 				}
-				_undoStack.add(new UndoLoad(_trackInfo, inDataArray.length, photos));
+				UndoLoad undo = new UndoLoad(_trackInfo, inLoadedTrack.getNumPoints(), photos);
+				undo.setNumPhotosAudios(_trackInfo.getPhotoList().getNumPhotos(), _trackInfo.getAudioList().getNumAudios());
+				_undoStack.add(undo);
 				_lastSavePosition = _undoStack.size();
 				_trackInfo.getSelection().clearAll();
-				_track.load(loadedTrack);
-				_trackInfo.getFileInfo().setFile(inFilename);
-				if (photos != null)
-				{
-					_trackInfo.getPhotoList().removeCorrelatedPhotos();
-				}
+				_track.load(inLoadedTrack);
+				inSourceInfo.populatePointObjects(_track, _track.getNumPoints());
+				_trackInfo.getFileInfo().replaceSource(inSourceInfo);
+				_trackInfo.getPhotoList().removeCorrelatedPhotos();
+				_trackInfo.getAudioList().removeCorrelatedAudios();
 			}
 		}
 		else
 		{
 			// Currently no data held, so transfer received data
-			_undoStack.add(new UndoLoad(_trackInfo, inDataArray.length, null));
+			UndoLoad undo = new UndoLoad(_trackInfo, inLoadedTrack.getNumPoints(), null);
+			undo.setNumPhotosAudios(_trackInfo.getPhotoList().getNumPhotos(), _trackInfo.getAudioList().getNumAudios());
+			_undoStack.add(undo);
 			_lastSavePosition = _undoStack.size();
 			_trackInfo.getSelection().clearAll();
-			_track.load(loadedTrack);
-			_trackInfo.getFileInfo().setFile(inFilename);
+			_track.load(inLoadedTrack);
+			inSourceInfo.populatePointObjects(_track, _track.getNumPoints());
+			_trackInfo.getFileInfo().addSource(inSourceInfo);
 		}
+		// Update config before subscribers are told
+		boolean isRegularLoad = (inSourceInfo.getFileType() != FILE_TYPE.GPSBABEL);
+		Config.getRecentFileList().addFile(new RecentFile(inSourceInfo.getFile(), isRegularLoad));
 		UpdateMessageBroker.informSubscribers();
 		// Update status bar
-		UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.loadfile") + " '" + inFilename + "'");
+		UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.loadfile")
+			+ " '" + inSourceInfo.getName() + "'");
 		// update menu
 		_menuManager.informFileLoaded();
+		// Remove busy lock
+		_busyLoading = false;
 		// load next file if there's a queue
 		loadNextFile();
 	}
@@ -722,12 +724,20 @@ public class App
 		loadNextFile();
 	}
 
+	/**
+	 * External trigger to automatically append the next loaded file
+	 * instead of prompting to replace or append
+	 */
+	public void autoAppendNextFile()
+	{
+		_autoAppendNextFile = true;
+	}
+
 	/**
 	 * Load the next file in the waiting list, if any
 	 */
 	private void loadNextFile()
 	{
-		_firstDataFile = false;
 		if (_dataFiles == null || _dataFiles.size() == 0) {
 			_dataFiles = null;
 		}
@@ -736,6 +746,7 @@ public class App
 				public void run() {
 					File f = _dataFiles.get(0);
 					_dataFiles.remove(0);
+					_autoAppendNextFile = true;
 					_fileLoader.openFile(f);
 				}
 			}).start();
@@ -759,125 +770,16 @@ public class App
 				// Save numbers so load can be undone
 				_undoStack.add(new UndoLoadPhotos(numPhotosAdded, numPointsAdded));
 			}
-			if (numPhotosAdded == 1)
-			{
+			if (numPhotosAdded == 1) {
 				UpdateMessageBroker.informSubscribers("" + numPhotosAdded + " " + I18nManager.getText("confirm.jpegload.single"));
 			}
-			else
-			{
+			else {
 				UpdateMessageBroker.informSubscribers("" + numPhotosAdded + " " + I18nManager.getText("confirm.jpegload.multi"));
 			}
 			// MAYBE: Improve message when photo(s) fail to load (eg already added)
 			UpdateMessageBroker.informSubscribers();
 			// update menu
-			_menuManager.informFileLoaded();
-		}
-	}
-
-
-	/**
-	 * Connect the current photo to the current point
-	 */
-	public void connectPhotoToPoint()
-	{
-		Photo photo = _trackInfo.getCurrentPhoto();
-		DataPoint point = _trackInfo.getCurrentPoint();
-		if (photo != null && point != null)
-		{
-			if (point.getPhoto() != null)
-			{
-				// point already has a photo, confirm cloning of new point
-				if (JOptionPane.showConfirmDialog(_frame,
-					I18nManager.getText("dialog.connectphoto.clonepoint"),
-					I18nManager.getText("dialog.connect.title"),
-					JOptionPane.OK_CANCEL_OPTION) == JOptionPane.OK_OPTION)
-				{
-					// Create undo, clone point and attach
-					int pointIndex = _trackInfo.getSelection().getCurrentPointIndex() + 1;
-					// insert new point after current one
-					point = point.clonePoint();
-					UndoConnectPhotoWithClone undo = new UndoConnectPhotoWithClone(
-						point, photo.getFile().getName(), pointIndex);
-					_track.insertPoint(point, pointIndex);
-					photo.setDataPoint(point);
-					point.setPhoto(photo);
-					_undoStack.add(undo);
-					UpdateMessageBroker.informSubscribers(DataSubscriber.SELECTION_CHANGED);
-					UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.photo.connect"));
-				}
-			}
-			else
-			{
-				// point doesn't currently have a photo, so just connect it
-				_undoStack.add(new UndoConnectPhoto(point, photo.getFile().getName()));
-				photo.setDataPoint(point);
-				point.setPhoto(photo);
-				UpdateMessageBroker.informSubscribers(DataSubscriber.SELECTION_CHANGED);
-				UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.photo.connect"));
-			}
-		}
-	}
-
-
-	/**
-	 * Disconnect the current photo from its point
-	 */
-	public void disconnectPhotoFromPoint()
-	{
-		Photo photo = _trackInfo.getCurrentPhoto();
-		if (photo != null && photo.getDataPoint() != null)
-		{
-			DataPoint point = photo.getDataPoint();
-			_undoStack.add(new UndoDisconnectPhoto(point, photo.getFile().getName()));
-			// disconnect
-			photo.setDataPoint(null);
-			point.setPhoto(null);
-			UpdateMessageBroker.informSubscribers(DataSubscriber.SELECTION_CHANGED);
-			UpdateMessageBroker.informSubscribers(I18nManager.getText("confirm.photo.disconnect"));
-		}
-	}
-
-
-	/**
-	 * Remove the current photo, if any
-	 */
-	public void deleteCurrentPhoto()
-	{
-		// Delete the current photo, and optionally its point too, keeping undo information
-		Photo currentPhoto = _trackInfo.getCurrentPhoto();
-		if (currentPhoto != null)
-		{
-			// Photo is selected, see if it has a point or not
-			boolean photoDeleted = false;
-			UndoDeletePhoto undoAction = null;
-			if (currentPhoto.getDataPoint() == null)
-			{
-				// no point attached, so just delete photo
-				undoAction = new UndoDeletePhoto(currentPhoto, _trackInfo.getSelection().getCurrentPhotoIndex(),
-					null, -1);
-				photoDeleted = _trackInfo.deleteCurrentPhoto(false);
-			}
-			else
-			{
-				// point is attached, so need to confirm point deletion
-				undoAction = new UndoDeletePhoto(currentPhoto, _trackInfo.getSelection().getCurrentPhotoIndex(),
-					currentPhoto.getDataPoint(), _trackInfo.getTrack().getPointIndex(currentPhoto.getDataPoint()));
-				int response = JOptionPane.showConfirmDialog(_frame,
-					I18nManager.getText("dialog.deletephoto.deletepoint"),
-					I18nManager.getText("dialog.deletephoto.title"),
-					JOptionPane.YES_NO_CANCEL_OPTION);
-				boolean deletePointToo = (response == JOptionPane.YES_OPTION);
-				// Cancel delete if cancel pressed or dialog closed
-				if (response == JOptionPane.YES_OPTION || response == JOptionPane.NO_OPTION)
-				{
-					photoDeleted = _trackInfo.deleteCurrentPhoto(deletePointToo);
-				}
-			}
-			// Add undo information to stack if necessary
-			if (photoDeleted)
-			{
-				_undoStack.add(undoAction);
-			}
+			if (numPointsAdded > 0) _menuManager.informFileLoaded();
 		}
 	}
 
@@ -965,32 +867,12 @@ public class App
 			showErrorMessageNoLookup("error.undofailed.title",
 				I18nManager.getText("error.undofailed.text") + " : " + ue.getMessage());
 			_undoStack.clear();
-			UpdateMessageBroker.informSubscribers();
 		}
 		catch (EmptyStackException empty) {}
+		UpdateMessageBroker.informSubscribers();
 	}
 
 
-	/**
-	 * Helper method to parse an Object into an integer
-	 * @param inObject object, eg from dialog
-	 * @return int value given
-	 */
-	private static int parseNumber(Object inObject)
-	{
-		int num = 0;
-		if (inObject != null)
-		{
-			try
-			{
-				num = Integer.parseInt(inObject.toString());
-			}
-			catch (NumberFormatException nfe)
-			{}
-		}
-		return num;
-	}
-
 	/**
 	 * Show a map url in an external browser
 	 * @param inSourceIndex index of map source to use
@@ -1037,4 +919,36 @@ public class App
 	{
 		return _viewport;
 	}
+
+	/**
+	 * Set the controller for the full screen mode
+	 * @param inController controller object
+	 */
+	public void setSidebarController(SidebarController inController)
+	{
+		_sidebarController = inController;
+	}
+
+	/**
+	 * Toggle sidebars on and off
+	 */
+	public void toggleSidebars()
+	{
+		_sidebarController.toggle();
+	}
+
+	/** @return true if App is currently busy with loading data */
+	public boolean isBusyLoading() {
+		return _busyLoading;
+	}
+
+	/** @return current app mode */
+	public AppMode getCurrentMode() {
+		return _appMode;
+	}
+
+	/** @param inMode the current app mode */
+	public void setCurrentMode(AppMode inMode) {
+		_appMode = inMode;
+	}
 }