From 66ee8fb0cd5641e604a44d6b0f05a13481b0d56b Mon Sep 17 00:00:00 2001
From: Ahmad Ansori Palembani <palembani@gmail.com>
Date: Sat, 3 Aug 2024 21:09:27 +0700
Subject: [PATCH] refactor(backup/creator): Reformat code

---
 .../data/backup/create/BackupCreator.kt       | 39 +++++++++----------
 1 file changed, 18 insertions(+), 21 deletions(-)

diff --git a/app/src/main/java/eu/kanade/tachiyomi/data/backup/create/BackupCreator.kt b/app/src/main/java/eu/kanade/tachiyomi/data/backup/create/BackupCreator.kt
index 173bc97fcc..56fcd2eb3c 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/data/backup/create/BackupCreator.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/data/backup/create/BackupCreator.kt
@@ -69,33 +69,30 @@ class BackupCreator(
 
         var file: UniFile? = null
         try {
-            file = (
-                if (isAutoBackup) {
-                    // Get dir of file and create
-                    val dir = storageManager.getAutomaticBackupsDirectory()
+            file = if (isAutoBackup) {
+                // Get dir of file and create
+                val dir = storageManager.getAutomaticBackupsDirectory()
 
-                    // Delete older backups
-                    val numberOfBackups = preferences.numberOfBackups().get()
-                    dir?.listFiles { _, filename -> Backup.filenameRegex.matches(filename) }
-                        .orEmpty()
-                        .sortedByDescending { it.name }
-                        .drop(numberOfBackups - 1)
-                        .forEach { it.delete() }
+                // Delete older backups
+                val numberOfBackups = preferences.numberOfBackups().get()
+                dir?.listFiles { _, filename -> Backup.filenameRegex.matches(filename) }
+                    .orEmpty()
+                    .sortedByDescending { it.name }
+                    .drop(numberOfBackups - 1)
+                    .forEach { it.delete() }
 
-                    // Create new file to place backup
-                    dir?.createFile(Backup.getBackupFilename())
-                } else {
-                    UniFile.fromUri(context, uri)
-                }
-                )
-                ?: throw Exception("Couldn't create backup file")
+                // Create new file to place backup
+                dir?.createFile(Backup.getBackupFilename())
+            } else {
+                UniFile.fromUri(context, uri)
+            }
 
-            if (!file.isFile) {
+            if (file == null || !file.isFile) {
                 throw IllegalStateException("Failed to get handle on file")
             }
 
-            val byteArray = parser.encodeToByteArray(Backup.serializer(), backup!!)
-            if (byteArray.isEmpty()) {
+            val byteArray = backup?.let { parser.encodeToByteArray(Backup.serializer(), it) }
+            if (byteArray == null || byteArray.isEmpty()) {
                 throw IllegalStateException(context.getString(MR.strings.empty_backup_error))
             }