import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.HashMap;
+import java.util.List;
import java.util.Map;
import java.util.TreeMap;
+import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.Future;
import jalview.bin.Cache;
+import jalview.bin.Console;
import jalview.gui.Desktop;
import jalview.gui.JvOptionPane;
import jalview.util.MessageManager;
private static final String oldTempFileSuffix = "_oldfile_tobedeleted";
+ // thread pool used for completablefutures
+ private static final ExecutorService executorService = Executors
+ .newFixedThreadPool(3);
+
+ private static List<BackupFiles> savesInProgress = new ArrayList<>();
+
+ private CompletableFuture<Boolean> myFuture = null;
+
+ private boolean addSaveInProgress()
+ {
+ if (savesInProgress.contains(this))
+ {
+ return false;
+ }
+ else
+ {
+ this.setMyFuture();
+ savesInProgress.add(this);
+ return true;
+ }
+ }
+
+ private boolean removeSaveInProgress(boolean ret)
+ {
+ if (savesInProgress.contains(this))
+ {
+ this.getMyFuture().complete(ret);
+ // remove all occurrences
+ while (savesInProgress.remove(this))
+ {
+ }
+ return true;
+ }
+ return false;
+ }
+
+ private static CompletableFuture<Boolean> getNewFuture()
+ {
+ return new CompletableFuture<Boolean>()
+ {
+ };
+ }
+
+ private CompletableFuture<Boolean> getMyFuture()
+ {
+ return this.myFuture;
+ }
+
+ private void setMyFuture()
+ {
+ this.myFuture = getNewFuture();
+ }
+
+ public static boolean hasSavesInProgress()
+ {
+ boolean has = false;
+ for (CompletableFuture cf : savesInProgressCompletableFutures(true))
+ {
+ has |= !cf.isDone();
+ }
+ return has;
+ }
+
+ public static List<File> savesInProgressFiles(boolean all)
+ {
+ List<File> files = new ArrayList<>();
+ for (BackupFiles bfile : savesInProgress)
+ {
+ if (all || !bfile.getMyFuture().isDone())
+ files.add(bfile.getFile());
+ }
+ return files;
+ }
+
+ public static List<CompletableFuture<Boolean>> savesInProgressCompletableFutures(
+ boolean all)
+ {
+ List<CompletableFuture<Boolean>> cfs = new ArrayList<>();
+ for (BackupFiles bfile : savesInProgress)
+ {
+ if (all || !bfile.getMyFuture().isDone())
+ cfs.add(bfile.getMyFuture());
+ }
+ return cfs;
+ }
+
+ public static Future<Boolean> allSaved()
+ {
+ CompletableFuture<Boolean> f = new CompletableFuture<>();
+
+ executorService.submit(() -> {
+ for (BackupFiles buf : savesInProgress)
+ {
+ boolean allSaved = true;
+ try
+ {
+ allSaved &= buf.getMyFuture().get();
+ } catch (InterruptedException e)
+ {
+ Console.debug("InterruptedException waiting for files to save",
+ e);
+ } catch (ExecutionException e)
+ {
+ Console.debug("ExecutionException waiting for files to save", e);
+ }
+ f.complete(allSaved);
+ }
+ });
+ return f;
+ }
+
public BackupFiles(String filename)
{
this(new File(filename));
{
classInit();
this.file = file;
+
+ // add this file from the save in progress stack
+ addSaveInProgress();
+
BackupFilesPresetEntry bfpe = BackupFilesPresetEntry
.getSavedBackupEntry();
this.suffix = bfpe.suffix;
{
String tempfilename = file.getName();
File tempdir = file.getParentFile();
- Cache.log.debug(
+ tempdir.mkdirs();
+ Console.trace(
"BACKUPFILES [file!=null] attempting to create temp file for "
+ tempfilename + " in dir " + tempdir);
temp = File.createTempFile(tempfilename,
TEMP_FILE_EXT + newTempFileSuffix, tempdir);
- Cache.log.debug(
+ Console.debug(
"BACKUPFILES using temp file " + temp.getAbsolutePath());
}
else
{
- Cache.log.debug(
+ Console.trace(
"BACKUPFILES [file==null] attempting to create default temp file "
+ DEFAULT_TEMP_FILE + " with extension "
+ TEMP_FILE_EXT);
}
} catch (IOException e)
{
- Cache.log
- .error("Could not create temp file to save to (IOException)");
- Cache.log.error(e.getMessage());
- Cache.log.debug(Cache.getStackTraceString(e));
+ Console.error("Could not create temp file to save to (IOException)");
+ Console.error(e.getMessage());
+ Console.debug(Cache.getStackTraceString(e));
} catch (Exception e)
{
- Cache.log.error("Exception ctreating temp file for saving");
- Cache.log.debug(Cache.getStackTraceString(e));
+ Console.error("Exception creating temp file for saving");
+ Console.debug(Cache.getStackTraceString(e));
}
this.setTempFile(temp);
}
public static void classInit()
{
- Cache.log.debug("BACKUPFILES classInit");
- boolean e = Cache.getDefault(ENABLED, true);
+ Console.initLogger();
+ Console.trace("BACKUPFILES classInit");
+ boolean e = Cache.getDefault(ENABLED, !Platform.isJS());
setEnabled(e);
- Cache.log.debug("BACKUPFILES " + (e ? "enabled" : "disabled"));
+ Console.trace("BACKUPFILES " + (e ? "enabled" : "disabled"));
BackupFilesPresetEntry bfpe = BackupFilesPresetEntry
.getSavedBackupEntry();
- Cache.log.debug("BACKUPFILES preset scheme " + bfpe.toString());
+ Console.trace("BACKUPFILES preset scheme " + bfpe.toString());
setConfirmDelete(bfpe.confirmDelete);
- Cache.log.debug("BACKUPFILES confirm delete " + bfpe.confirmDelete);
+ Console.trace("BACKUPFILES confirm delete " + bfpe.confirmDelete);
}
public static void setEnabled(boolean flag)
path = this.getTempFile().getCanonicalPath();
} catch (IOException e)
{
- Cache.log.error(
- "IOException when getting Canonical Path of temp file '"
- + this.getTempFile().getName() + "'");
- Cache.log.debug(Cache.getStackTraceString(e));
+ Console.error("IOException when getting Canonical Path of temp file '"
+ + this.getTempFile().getName() + "'");
+ Console.debug(Cache.getStackTraceString(e));
}
return path;
}
|| suffix.length() == 0)
{
// nothing to do
- Cache.log.debug("BACKUPFILES rollBackupFiles nothing to do." + ", "
+ Console.debug("BACKUPFILES rollBackupFiles nothing to do." + ", "
+ "filename: " + (file != null ? file.getName() : "null")
+ ", " + "file exists: " + file.exists() + ", " + "enabled: "
+ enabled + ", " + "max: " + max + ", " + "suffix: '" + suffix
return true;
}
- Cache.log.debug("BACKUPFILES rollBackupFiles starting");
+ Console.trace("BACKUPFILES rollBackupFiles starting");
String dir = "";
File dirFile;
{
dirFile = file.getParentFile();
dir = dirFile.getCanonicalPath();
- Cache.log.debug("BACKUPFILES dir: " + dir);
+ Console.trace("BACKUPFILES dir: " + dir);
} catch (Exception e)
{
- Cache.log.error(
- "Could not get canonical path for file '" + file + "'");
- Cache.log.error(e.getMessage());
- Cache.log.debug(Cache.getStackTraceString(e));
+ Console.error("Could not get canonical path for file '" + file + "'");
+ Console.error(e.getMessage());
+ Console.debug(Cache.getStackTraceString(e));
return false;
}
String filename = file.getName();
String basename = filename;
- Cache.log.debug("BACKUPFILES filename is " + filename);
+ Console.trace("BACKUPFILES filename is " + filename);
boolean ret = true;
// Create/move backups up one
File[] backupFiles = dirFile.listFiles(bff);
int nextIndexNum = 0;
- Cache.log
- .debug("BACKUPFILES backupFiles.length: " + backupFiles.length);
+ Console.trace("BACKUPFILES backupFiles.length: " + backupFiles.length);
if (backupFiles.length == 0)
{
// No other backup files. Just need to move existing file to backupfile_1
- Cache.log.debug(
+ Console.trace(
"BACKUPFILES no existing backup files, setting index to 1");
nextIndexNum = 1;
}
if (reverseOrder)
{
// backup style numbering
- Cache.log.debug("BACKUPFILES rolling files in reverse order");
+ Console.trace("BACKUPFILES rolling files in reverse order");
int tempMax = noMax ? -1 : max;
// noMax == true means no limits
// no "oldest" file to delete
previousFile = backupfile_n;
fileToBeDeleted = null;
- Cache.log.debug("BACKUPFILES No oldest file to delete");
+ Console.trace("BACKUPFILES No oldest file to delete");
continue;
}
File replacementFile = backupfile_n;
long fileToBeDeletedLMT = fileToBeDeleted.lastModified();
long replacementFileLMT = replacementFile.lastModified();
- Cache.log.debug("BACKUPFILES fileToBeDeleted is "
+ Console.trace("BACKUPFILES fileToBeDeleted is "
+ fileToBeDeleted.getAbsolutePath());
- Cache.log.debug("BACKUPFILES replacementFile is "
+ Console.trace("BACKUPFILES replacementFile is "
+ backupfile_n.getAbsolutePath());
try
.format(fileToBeDeletedLMT);
String replacementFileLMTString = sdf
.format(replacementFileLMT);
- Cache.log.warn("WARNING! I am set to delete backupfile "
+ Console.warn("WARNING! I am set to delete backupfile "
+ fileToBeDeleted.getName()
+ " has modification time "
+ fileToBeDeletedLMTString
boolean delete = confirmNewerDeleteFile(fileToBeDeleted,
replacementFile, true);
- Cache.log.debug("BACKUPFILES "
+ Console.trace("BACKUPFILES "
+ (delete ? "confirmed" : "not") + " deleting file "
+ fileToBeDeleted.getAbsolutePath()
+ " which is newer than "
}
else
{
- Cache.log.debug("BACKUPFILES moving "
+ Console.debug("BACKUPFILES moving "
+ fileToBeDeleted.getAbsolutePath() + " to "
+ oldestTempFile.getAbsolutePath());
moveFileToFile(fileToBeDeleted, oldestTempFile);
}
else
{
- Cache.log.debug("BACKUPFILES going to move "
+ Console.debug("BACKUPFILES going to move "
+ fileToBeDeleted.getAbsolutePath() + " to "
+ oldestTempFile.getAbsolutePath());
moveFileToFile(fileToBeDeleted, oldestTempFile);
} catch (Exception e)
{
- Cache.log.error(
+ Console.error(
"Error occurred, probably making new temp file for '"
+ fileToBeDeleted.getName() + "'");
- Cache.log.error(Cache.getStackTraceString(e));
+ Console.error(Cache.getStackTraceString(e));
}
// reset
}
bfsb.append(backupFiles[i].getName());
}
- Cache.log.debug("BACKUPFILES backupFiles: " + bfsb.toString());
+ Console.trace("BACKUPFILES backupFiles: " + bfsb.toString());
// noMax == true means keep all backup files
if ((!noMax) && bfTreeMap.size() >= max)
{
- Cache.log.debug("BACKUPFILES noMax: " + noMax + ", " + "max: "
- + max + ", " + "bfTreeMap.size(): " + bfTreeMap.size());
+ Console.trace("BACKUPFILES noMax: " + noMax + ", " + "max: " + max
+ + ", " + "bfTreeMap.size(): " + bfTreeMap.size());
// need to delete some files to keep number of backups to designated
// max.
// Note that if the suffix is not numbered then do not delete any
int numToDelete = suffix.indexOf(NUM_PLACEHOLDER) > -1
? bfTreeMap.size() - max + 1
: 0;
- Cache.log.debug("BACKUPFILES numToDelete: " + numToDelete);
+ Console.trace("BACKUPFILES numToDelete: " + numToDelete);
// the "replacement" file is the latest backup file being kept (it's
// not replacing though)
File replacementFile = numToDelete < backupFiles.length
File fileToBeDeleted = backupFiles[i];
boolean delete = true;
- Cache.log.debug(
+ Console.trace(
"BACKUPFILES fileToBeDeleted: " + fileToBeDeleted);
boolean newer = false;
String replacementFileLMTString = sdf
.format(replacementFileLMT);
- Cache.log.warn("WARNING! I am set to delete backupfile '"
+ Console.warn("WARNING! I am set to delete backupfile '"
+ fileToBeDeleted.getName()
+ "' has modification time "
+ fileToBeDeletedLMTString
{
// User has confirmed delete -- no need to add it to the list
fileToBeDeleted.delete();
- Cache.log.debug("BACKUPFILES deleting fileToBeDeleted: "
+ Console.debug("BACKUPFILES deleting fileToBeDeleted: "
+ fileToBeDeleted);
delete = false;
}
else
{
// keeping file, nothing to do!
- Cache.log.debug("BACKUPFILES keeping fileToBeDeleted: "
+ Console.debug("BACKUPFILES keeping fileToBeDeleted: "
+ fileToBeDeleted);
}
}
if (delete)
{
addDeleteFile(fileToBeDeleted);
- Cache.log.debug("BACKUPFILES addDeleteFile(fileToBeDeleted): "
+ Console.debug("BACKUPFILES addDeleteFile(fileToBeDeleted): "
+ fileToBeDeleted);
}
String latestBackupFilename = dir + File.separatorChar
+ BackupFilenameParts.getBackupFilename(nextIndexNum, basename,
suffix, digits);
- Cache.log.debug("BACKUPFILES Moving old file [" + file
+ Console.trace("BACKUPFILES Moving old file [" + file
+ "] to latestBackupFilename [" + latestBackupFilename + "]");
// using boolean '&' instead of '&&' as don't want moveFileToFile attempt to
// be conditional (short-circuit)
ret = ret & moveFileToFile(file, new File(latestBackupFilename));
- Cache.log.debug(
+ Console.debug(
"BACKUPFILES moving " + file + " to " + latestBackupFilename
+ " was " + (ret ? "" : "NOT ") + "successful");
if (tidyUp)
{
- Cache.log.debug("BACKUPFILES tidying up files");
+ Console.debug("BACKUPFILES tidying up files");
tidyUpFiles();
}
saveFile = nextTempFile(ftbd.getName(), ftbd.getParentFile());
} catch (Exception e)
{
- Cache.log.error(
+ Console.error(
"Error when confirming to keep backup file newer than other backup files.");
e.printStackTrace();
}
for (int i = 0; i < deleteFiles.size(); i++)
{
File fileToDelete = deleteFiles.get(i);
- Cache.log.debug(
- "BACKUPFILES deleting fileToDelete:" + fileToDelete);
+ Console.trace("BACKUPFILES about to delete fileToDelete:"
+ + fileToDelete);
fileToDelete.delete();
- Cache.log.warn("deleting '" + fileToDelete.getName() + "'");
+ Console.warn("deleted '" + fileToDelete.getName() + "'");
}
}
tidyUpFiles();
}
+ // remove this file from the save in progress stack
+ removeSaveInProgress(rename);
+
return rename;
}
dirFile = file.getParentFile();
} catch (Exception e)
{
- Cache.log.error(
- "Could not get canonical path for file '" + file + "'");
+ Console.error("Could not get canonical path for file '" + file + "'");
return new TreeMap<>();
}
int pos = deleteFiles.indexOf(fileToBeDeleted);
if (pos > -1)
{
- Cache.log.debug("BACKUPFILES not adding file "
+ Console.debug("BACKUPFILES not adding file "
+ fileToBeDeleted.getAbsolutePath()
+ " to the delete list (already at index" + pos + ")");
return true;
}
else
{
- Cache.log.debug("BACKUPFILES adding file "
+ Console.debug("BACKUPFILES adding file "
+ fileToBeDeleted.getAbsolutePath() + " to the delete list");
deleteFiles.add(fileToBeDeleted);
}
return ret;
}
+ public File getFile()
+ {
+ return file;
+ }
+
public static boolean moveFileToFile(File oldFile, File newFile)
{
+ Console.initLogger();
boolean ret = false;
Path oldPath = Paths.get(oldFile.getAbsolutePath());
Path newPath = Paths.get(newFile.getAbsolutePath());
try
{
// delete destination file - not usually necessary but Just In Case...
- Cache.log.debug("BACKUPFILES deleting " + newFile.getAbsolutePath());
+ Console.trace("BACKUPFILES deleting " + newFile.getAbsolutePath());
newFile.delete();
- Cache.log.debug("BACKUPFILES moving " + oldFile.getAbsolutePath()
+ Console.trace("BACKUPFILES moving " + oldFile.getAbsolutePath()
+ " to " + newFile.getAbsolutePath());
Files.move(oldPath, newPath, StandardCopyOption.REPLACE_EXISTING);
ret = true;
- Cache.log.debug("BACKUPFILES move seems to have succeeded");
+ Console.trace("BACKUPFILES move seems to have succeeded");
} catch (IOException e)
{
- Cache.log.warn("Could not move file '" + oldPath.toString() + "' to '"
+ Console.warn("Could not move file '" + oldPath.toString() + "' to '"
+ newPath.toString() + "'");
- Cache.log.error(e.getMessage());
- Cache.log.debug(Cache.getStackTraceString(e));
+ Console.error(e.getMessage());
+ Console.debug(Cache.getStackTraceString(e));
ret = false;
} catch (Exception e)
{
- Cache.log.error(e.getMessage());
- Cache.log.debug(Cache.getStackTraceString(e));
+ Console.error(e.getMessage());
+ Console.debug(Cache.getStackTraceString(e));
ret = false;
}
return ret;