style: consistent braces on if statements

owncloud · Jan 28, 2025 · 025440b · 025440b
1 parent 231a761
commit 025440b
Show file tree

Hide file tree

Showing 13 changed files with 42 additions and 17 deletions.
diff --git a/owncloudApp/src/main/java/com/owncloud/android/presentation/authentication/LoginActivity.kt b/owncloudApp/src/main/java/com/owncloud/android/presentation/authentication/LoginActivity.kt
@@ -802,7 +802,9 @@ class LoginActivity : AppCompatActivity(), SslUntrustedCertDialog.OnSslUntrusted
                     setResult(Activity.RESULT_CANCELED)
                     goToUrl(url = getString(R.string.welcome_link_url))
                 }
-            } else isVisible = false
+            } else {
+                isVisible = false
+            }
         }
 
         val legacyWebfingerLookupServer = mdmProvider.getBrandingString(NO_MDM_RESTRICTION_YET, R.string.webfinger_lookup_server)

diff --git a/.../src/main/java/com/owncloud/android/presentation/documentsprovider/cursors/SpaceCursor.kt b/.../src/main/java/com/owncloud/android/presentation/documentsprovider/cursors/SpaceCursor.kt
@@ -42,7 +42,9 @@ class SpaceCursor(projection: Array<String>?) : MatrixCursor(projection ?: DEFAU
     fun addSpace(space: OCSpace, rootFolder: OCFile, context: Context?) {
         val flags = if (rootFolder.hasAddFilePermission && rootFolder.hasAddSubdirectoriesPermission) {
             Document.FLAG_DIR_SUPPORTS_CREATE
-        } else 0
+        } else {
+            0
+        }
 
         val name = if (space.isPersonal) context?.getString(R.string.bottom_nav_personal) else space.name
 

diff --git a/...p/src/main/java/com/owncloud/android/presentation/files/filelist/MainFileListViewModel.kt b/...p/src/main/java/com/owncloud/android/presentation/files/filelist/MainFileListViewModel.kt
@@ -227,14 +227,18 @@ class MainFileListViewModel(
                         parentDir =
                             if (fileById != null && (!fileById.sharedByLink || fileById.sharedWithSharee != true) && fileById.spaceId == null) {
                                 getFileByRemotePathUseCase(GetFileByRemotePathUseCase.Params(fileById.owner, ROOT_PATH)).getDataOrNull()
-                            } else fileById
+                            } else {
+                                fileById
+                            }
                     }
 
                     FileListOption.AV_OFFLINE -> {
                         val fileById = fileByIdResult.getDataOrNull()
                         parentDir = if (fileById != null && (!fileById.isAvailableOffline)) {
                             getFileByRemotePathUseCase(GetFileByRemotePathUseCase.Params(fileById.owner, ROOT_PATH)).getDataOrNull()
-                        } else fileById
+                        } else {
+                            fileById
+                        }
                     }
 
                     FileListOption.SPACES_LIST -> {

diff --git a/.../src/main/java/com/owncloud/android/presentation/security/biometric/BiometricViewModel.kt b/.../src/main/java/com/owncloud/android/presentation/security/biometric/BiometricViewModel.kt
@@ -149,7 +149,9 @@ class BiometricViewModel(
     fun isBiometricLockAvailable(): Boolean =
         if (!BiometricManager.isHardwareDetected()) { // Biometric not supported
             false
-        } else BiometricManager.hasEnrolledBiometric() // Biometric not enrolled
+        } else {
+            BiometricManager.hasEnrolledBiometric() // Biometric not enrolled
+        }
 
     companion object {
         private const val ANDROID_KEY_STORE = "AndroidKeyStore"

diff --git a/...main/java/com/owncloud/android/presentation/settings/security/SettingsSecurityFragment.kt b/...main/java/com/owncloud/android/presentation/settings/security/SettingsSecurityFragment.kt
@@ -59,7 +59,7 @@ class SettingsSecurityFragment : PreferenceFragmentCompat() {
 
     private val enablePasscodeLauncher =
         registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
-            if (result.resultCode != Activity.RESULT_OK) return@registerForActivityResult
+            if (result.resultCode != Activity.RESULT_OK) { return@registerForActivityResult }
             else {
                 prefPasscode?.isChecked = true
                 prefBiometric?.isChecked = securityViewModel.getBiometricsState()
@@ -71,7 +71,7 @@ class SettingsSecurityFragment : PreferenceFragmentCompat() {
 
     private val disablePasscodeLauncher =
         registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
-            if (result.resultCode != Activity.RESULT_OK) return@registerForActivityResult
+            if (result.resultCode != Activity.RESULT_OK) { return@registerForActivityResult }
             else {
                 prefPasscode?.isChecked = false
 
@@ -83,7 +83,7 @@ class SettingsSecurityFragment : PreferenceFragmentCompat() {
 
     private val enablePatternLauncher =
         registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
-            if (result.resultCode != Activity.RESULT_OK) return@registerForActivityResult
+            if (result.resultCode != Activity.RESULT_OK) { return@registerForActivityResult }
             else {
                 prefPattern?.isChecked = true
                 prefBiometric?.isChecked = securityViewModel.getBiometricsState()
@@ -95,7 +95,7 @@ class SettingsSecurityFragment : PreferenceFragmentCompat() {
 
     private val disablePatternLauncher =
         registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
-            if (result.resultCode != Activity.RESULT_OK) return@registerForActivityResult
+            if (result.resultCode != Activity.RESULT_OK) { return@registerForActivityResult }
             else {
                 prefPattern?.isChecked = false
 

diff --git a/...c/main/java/com/owncloud/android/presentation/sharing/shares/PublicShareDialogFragment.kt b/...c/main/java/com/owncloud/android/presentation/sharing/shares/PublicShareDialogFragment.kt
@@ -148,7 +148,9 @@ class PublicShareDialogFragment : DialogFragment() {
                 capabilities?.filesSharingPublicExpireDateDays!!
             )
                 .time
-        } else -1
+        } else {
+            -1
+        }
 
     private val capabilityViewModel: CapabilityViewModel by viewModel {
         parametersOf(

diff --git a/owncloudApp/src/main/java/com/owncloud/android/providers/MdmProvider.kt b/owncloudApp/src/main/java/com/owncloud/android/providers/MdmProvider.kt
@@ -112,7 +112,9 @@ class MdmProvider(
 
         return if (isMdmFlavor()) {
             preferencesProvider.getString(key = mdmKey, defaultValue = setupValue) ?: throw IllegalStateException("Key $stringKey is not supported")
-        } else setupValue
+        } else {
+            setupValue
+        }
     }
 
     fun getBrandingBoolean(

diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImagePagerAdapter.kt b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImagePagerAdapter.kt
@@ -160,7 +160,9 @@ class PreviewImagePagerAdapter(
                 // trigger the creation of new PreviewImageFragment to replace current FileDownloadFragment
                 // only if the download succeeded. If not trigger an error
                 notifyDataSetChanged()
-            } else fragment?.onSyncEvent(action, success, file)
+            } else {
+                fragment?.onSyncEvent(action, success, file)
+            }
         }
     }
 }
diff --git a/owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/utils/OCFileLoggingTree.kt b/owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/utils/OCFileLoggingTree.kt
@@ -104,7 +104,7 @@ class OCFileLoggingTree(
                 element.lineNumber // format ensures line numbers have at least 3 places to align consecutive output from the same file
             )
             return "(${element.fileName}:${element.lineNumber})"
-        } else
+        } else {
             return String.format(
                 Locale.US,
                 "(%s:%d) %s.%s()",
@@ -114,6 +114,8 @@ class OCFileLoggingTree(
                 super.createStackElementTag(element)?.replaceFirst(element.fileName.takeWhile { it != '.' }, ""),
                 element.methodName
             )
+        }
+
     }
 
     @SuppressLint("LogNotTimber")

diff --git a/...ary/src/main/java/com/owncloud/android/lib/resources/users/GetRemoteUserQuotaOperation.kt b/...ary/src/main/java/com/owncloud/android/lib/resources/users/GetRemoteUserQuotaOperation.kt
@@ -116,7 +116,9 @@ class GetRemoteUserQuotaOperation : RemoteOperation<RemoteQuota>() {
             val roundedRelativeQuota = if (totalQuota > 0) {
                 val relativeQuota = (quotaUsed * 100).toDouble() / totalQuota
                 (relativeQuota * 100).roundToLong() / 100.0
-            } else 0.0
+            } else {
+                0.0
+            }
 
             RemoteQuota(quotaAvailable, quotaUsed, totalQuota, roundedRelativeQuota)
         }

diff --git a/owncloudData/src/main/java/com/owncloud/android/data/extensions/FileExt.kt b/owncloudData/src/main/java/com/owncloud/android/data/extensions/FileExt.kt
@@ -48,7 +48,8 @@ fun File.moveRecursively(
                 val relPath = src.toRelativeString(this)
                 val dstFile = File(target, relPath)
                 if (dstFile.exists() && !(src.isDirectory && dstFile.isDirectory)) {
-                    val stillExists = if (!overwrite) true else {
+                    val stillExists = if (!overwrite) { true }
+                    else {
                         if (dstFile.isDirectory)
                             !dstFile.deleteRecursively()
                         else

diff --git a/owncloudData/src/main/java/com/owncloud/android/data/files/db/FileDao.kt b/owncloudData/src/main/java/com/owncloud/android/data/files/db/FileDao.kt
@@ -485,7 +485,9 @@ interface FileDao {
             AVAILABLE_OFFLINE_PARENT.ordinal
         } else if (currentFileAvailableOfflineStatus == AVAILABLE_OFFLINE.ordinal) {
             AVAILABLE_OFFLINE.ordinal
-        } else NOT_AVAILABLE_OFFLINE.ordinal
+        } else {
+            NOT_AVAILABLE_OFFLINE.ordinal
+        }
 
     companion object {
 

diff --git a/...in/src/main/java/com/owncloud/android/domain/server/usecases/GetServerInfoAsyncUseCase.kt b/...in/src/main/java/com/owncloud/android/domain/server/usecases/GetServerInfoAsyncUseCase.kt
@@ -58,7 +58,9 @@ class GetServerInfoAsyncUseCase(
             !url.lowercase(Locale.getDefault()).startsWith(HTTPS_PREFIX)
         ) {
             "$HTTPS_PREFIX$url"
-        } else url
+        } else {
+            url
+        }
 
     companion object {
         const val TRAILING_SLASH = '/'