diff --git a/src/main/java/stirling/software/SPDF/config/security/database/DatabaseBackupHelper.java b/src/main/java/stirling/software/SPDF/config/security/database/DatabaseBackupHelper.java index 1da9b108bea..b40f78c27fa 100644 --- a/src/main/java/stirling/software/SPDF/config/security/database/DatabaseBackupHelper.java +++ b/src/main/java/stirling/software/SPDF/config/security/database/DatabaseBackupHelper.java @@ -42,6 +42,7 @@ public class DatabaseBackupHelper implements DatabaseBackupInterface { private Path backupPath = Paths.get("configs/db/backup/"); + // fixMe: should check if backups exist without returning the whole list @Override public boolean hasBackup() { // Check if there is at least one backup @@ -51,6 +52,7 @@ public boolean hasBackup() { @Override public List getBackupList() { // Check if the backup directory exists, and create it if it does not + // todo: as this should always exist, can we make this an 'init' method so we do not have to check all the time? ensureBackupDirectoryExists(); List backupFiles = new ArrayList<>(); @@ -92,6 +94,7 @@ public boolean importDatabaseFromUI(String fileName) throws IOException { } // Imports a database backup from the specified path. + // todo: make private? public boolean importDatabaseFromUI(Path tempTemplatePath) throws IOException { boolean success = executeDatabaseScript(tempTemplatePath); if (success) {