From 1c6d68ee303ad3d4b1df5c3d433bf972256f0c8e Mon Sep 17 00:00:00 2001 From: Keuin Date: Wed, 13 Jan 2021 13:23:31 +0800 Subject: Refactor for a better code quality --- src/main/java/com/keuin/kbackupfabric/operation/BackupOperation.java | 4 ++-- .../keuin/kbackupfabric/operation/backup/method/BackupMethod.java | 5 +++-- .../operation/backup/method/IncrementalBackupMethod.java | 2 +- 3 files changed, 6 insertions(+), 5 deletions(-) (limited to 'src') diff --git a/src/main/java/com/keuin/kbackupfabric/operation/BackupOperation.java b/src/main/java/com/keuin/kbackupfabric/operation/BackupOperation.java index 3183cf0..0ff2c40 100644 --- a/src/main/java/com/keuin/kbackupfabric/operation/BackupOperation.java +++ b/src/main/java/com/keuin/kbackupfabric/operation/BackupOperation.java @@ -52,8 +52,8 @@ public class BackupOperation extends InvokableAsyncBlockingOperation { String levelPath = getLevelPath(server); String backupFileName = getBackupFileName(customBackupName); - BackupFeedback result = backupMethod.backup(customBackupName,levelPath,backupSaveDirectory); - if(result.isSuccess()) { + BackupFeedback result = backupMethod.backup(customBackupName, levelPath, backupSaveDirectory); + if (result.isSuccess()) { // Restore old auto-save switch stat server.getWorlds().forEach(world -> world.savingDisabled = oldWorldsSavingDisabled.getOrDefault(world, true)); diff --git a/src/main/java/com/keuin/kbackupfabric/operation/backup/method/BackupMethod.java b/src/main/java/com/keuin/kbackupfabric/operation/backup/method/BackupMethod.java index 25a5952..7afbabd 100644 --- a/src/main/java/com/keuin/kbackupfabric/operation/backup/method/BackupMethod.java +++ b/src/main/java/com/keuin/kbackupfabric/operation/backup/method/BackupMethod.java @@ -12,10 +12,11 @@ public interface BackupMethod { /** * Perform a backup with given method. The backup will be saved as the given name. * Note: real file name depends on the backup type. - * @param backupName the backup name. + * + * @param customBackupName the custom backup name. * @return if the backup operation succeed. */ - BackupFeedback backup(String backupName, String levelPath, String backupSaveDirectory) throws IOException; + BackupFeedback backup(String customBackupName, String levelPath, String backupSaveDirectory) throws IOException; boolean restore(String backupName, String levelPath, String backupSaveDirectory) throws IOException; diff --git a/src/main/java/com/keuin/kbackupfabric/operation/backup/method/IncrementalBackupMethod.java b/src/main/java/com/keuin/kbackupfabric/operation/backup/method/IncrementalBackupMethod.java index 1dacc44..25e5731 100644 --- a/src/main/java/com/keuin/kbackupfabric/operation/backup/method/IncrementalBackupMethod.java +++ b/src/main/java/com/keuin/kbackupfabric/operation/backup/method/IncrementalBackupMethod.java @@ -6,7 +6,7 @@ import java.io.IOException; public class IncrementalBackupMethod implements BackupMethod { @Override - public IncrementalBackupFeedback backup(String backupName, String levelPath, String backupSaveDirectory) throws IOException { + public IncrementalBackupFeedback backup(String customBackupName, String levelPath, String backupSaveDirectory) throws IOException { return null; } -- cgit v1.2.3