Skip to content

Commit 86ff7f5

Browse files
committed
BackupAndRestore: bit refactor
1 parent 72e4bd9 commit 86ff7f5

File tree

1 file changed

+19
-18
lines changed

1 file changed

+19
-18
lines changed

common/src/main/java/com/liskovsoft/smartyoutubetv2/common/misc/BackupAndRestoreHelper.java

Lines changed: 19 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -42,28 +42,24 @@ public void exportAppMediaFolder() {
4242
File dataDir = new File(mediaDir, "data");
4343
if (!dataDir.exists() || FileHelpers.isEmpty(dataDir) || VERSION.SDK_INT < 29) return;
4444

45-
String oldBackupZipName = getGeneralData().getBackupZipName();
46-
if (oldBackupZipName == null || !oldBackupZipName.endsWith(".zip")) {
47-
oldBackupZipName = createBackupZipNameWithTimestamp();
48-
getGeneralData().setBackupZipName(oldBackupZipName);
49-
}
45+
String backupZipName = getSavedBackupZipName();
5046

51-
MediaStoreFile file = new MediaStoreFile(mContext, oldBackupZipName, BACKUP_FOLDER_NAME);
47+
MediaStoreFile file = new MediaStoreFile(mContext, backupZipName, BACKUP_FOLDER_NAME);
5248
if (!file.isWritable()) {
53-
oldBackupZipName = createBackupZipNameWithTimestamp();
54-
getGeneralData().setBackupZipName(oldBackupZipName);
55-
file = new MediaStoreFile(mContext, oldBackupZipName, BACKUP_FOLDER_NAME);
49+
backupZipName = createBackupZipNameWithTimestamp();
50+
getGeneralData().setBackupZipName(backupZipName);
51+
file = new MediaStoreFile(mContext, backupZipName, BACKUP_FOLDER_NAME);
5652
}
5753

5854
if (!file.isWritable()) {
5955
deleteTimeStamp(); // User copied full old media directory (with the old timestamp)
60-
oldBackupZipName = createBackupZipNameWithTimestamp();
61-
getGeneralData().setBackupZipName(oldBackupZipName);
62-
file = new MediaStoreFile(mContext, oldBackupZipName, BACKUP_FOLDER_NAME);
56+
backupZipName = createBackupZipNameWithTimestamp();
57+
getGeneralData().setBackupZipName(backupZipName);
58+
file = new MediaStoreFile(mContext, backupZipName, BACKUP_FOLDER_NAME);
6359
}
6460

6561
if (file.isWritable()) {
66-
final File zipFile = new File(mediaDir, oldBackupZipName);
62+
final File zipFile = new File(mediaDir, backupZipName);
6763
ZipHelper2.zipDirectory(dataDir, zipFile);
6864

6965
if (zipFile.exists()) {
@@ -196,11 +192,7 @@ private void unpackTempZip(Uri zipUri, Runnable onSuccess, Runnable onError) {
196192
File mediaDir = FileHelpers.getExternalMediaDirectory(mContext);
197193

198194
// Copy ZIP from URI to the temporary file
199-
String backupZipName = getGeneralData().getBackupZipName();
200-
if (backupZipName == null || !backupZipName.endsWith(".zip")) {
201-
backupZipName = createBackupZipNameWithTimestamp();
202-
getGeneralData().setBackupZipName(backupZipName);
203-
}
195+
String backupZipName = getSavedBackupZipName();
204196
File tempZip = new File(mediaDir, backupZipName);
205197
copyUriToFile(zipUri, tempZip);
206198

@@ -303,4 +295,13 @@ private File getTimestampFile() {
303295
File timestampFile = new File(mediaDir, "timestamp.txt");
304296
return timestampFile;
305297
}
298+
299+
private String getSavedBackupZipName() {
300+
String oldBackupZipName = getGeneralData().getBackupZipName();
301+
if (oldBackupZipName == null || !oldBackupZipName.endsWith(".zip")) {
302+
oldBackupZipName = createBackupZipNameWithTimestamp();
303+
getGeneralData().setBackupZipName(oldBackupZipName);
304+
}
305+
return oldBackupZipName;
306+
}
306307
}

0 commit comments

Comments
 (0)