JAL-3628 Fixed oddly worded and non-formatted error popups when saving fails

[jalview.git] / src / jalview / gui / AlignFrame.java

diff --git a/src/jalview/gui/AlignFrame.java b/src/jalview/gui/AlignFrame.java
index 5b7a201..4da30ea 100644 (file)
--- a/src/jalview/gui/AlignFrame.java
+++ b/src/jalview/gui/AlignFrame.java
@@ -1250,7 +1250,7 @@ public class AlignFrame extends GAlignFrame implements DropTargetListener,
 
       statusBar.setText(MessageManager.formatMessage(
               "label.successfully_saved_to_file_in_format", new Object[]
-              { fileName, format }));
+              { file, format }));
 
       return;
     }
@@ -1301,7 +1301,7 @@ public class AlignFrame extends GAlignFrame implements DropTargetListener,
 
             if (backupfiles != null)
             {
-              Cache.log.debug("ALIGNFRAME using temp file "
+              Cache.log.debug("ALIGNFRAME about to write to temp file "
                       + backupfiles.getTempFilePath());
             }
 
@@ -1321,21 +1321,21 @@ public class AlignFrame extends GAlignFrame implements DropTargetListener,
             Cache.log.error(
                     "ALIGNFRAME Something happened writing the temp file");
             Cache.log.error(e.getMessage());
-            Cache.log.debug(e.getStackTrace());
+            Cache.log.debug(Cache.getStackTraceString(e));
           } catch (Exception ex)
           {
             lastSaveSuccessful = false;
             Cache.log.error(
                     "ALIGNFRAME Something else happened writing the temp file");
             Cache.log.error(ex.getMessage());
-            Cache.log.debug(ex.getStackTrace());
+            Cache.log.debug(Cache.getStackTraceString(ex));
           }
 
           if (doBackup)
           {
             backupfiles.setWriteSuccess(lastSaveSuccessful);
             Cache.log.debug("ALIGNFRAME writing temp file was "
-                    + (lastSaveSuccessful ? "" : "not ") + "successful");
+                    + (lastSaveSuccessful ? "" : "NOT ") + "successful");
             // do the backup file roll and rename the temp file to actual file
             Cache.log.debug(
                     "ALIGNFRAME about to rollBackupsAndRenameTempFile");