alperozturk96 · Oct 26, 2026 · Oct 23, 2026
diff --git a/app/detekt.yml b/app/detekt.yml
@@ -53,9 +53,9 @@ complexity:
     active: false
     threshold: 10
     includeStaticDeclarations: false
-  ComplexMethod:
+  CyclomaticComplexMethod:
     active: true
-    threshold: 10
+    threshold: 15
     ignoreSingleWhenExpression: false
     ignoreSimpleWhenEntries: false
     excludes: ['**/androidTest/**']
@@ -70,7 +70,7 @@ complexity:
     threshold: 60
     excludes: ['**/androidTest/**']
   LongParameterList:
-    active: true
+    active: false
     functionThreshold: 7
     constructorThreshold: 6
     ignoreDefaultParameters: false
@@ -87,7 +87,7 @@ complexity:
     excludeStringsWithLessThan5Characters: true
     ignoreStringsRegex: '$^'
   TooManyFunctions:
-    active: true
+    active: false
     thresholdInFiles: 15
     thresholdInClasses: 15
     thresholdInInterfaces: 15
@@ -158,7 +158,7 @@ exceptions:
   ThrowingNewInstanceOfSameException:
     active: false
   TooGenericExceptionCaught:
-    active: true
+    active: false
     exceptionNames:
      - ArrayIndexOutOfBoundsException
      - Error
@@ -170,7 +170,7 @@ exceptions:
      - Throwable
     allowedExceptionNameRegex: "^(_|(ignore|expected).*)"
   TooGenericExceptionThrown:
-    active: true
+    active: false
     exceptionNames:
      - Error
      - Exception
@@ -226,7 +226,7 @@ naming:
     active: true
     packagePattern: '^[a-z]+(\.[a-z][A-Za-z0-9]*)*$'
   TopLevelPropertyNaming:
-    active: true
+    active: false
     constantPattern: '[A-Z][_A-Z0-9]*'
     propertyPattern: '[A-Za-z][_A-Za-z0-9]*'
     privatePropertyPattern: '(_)?[A-Za-z][A-Za-z0-9]*'
@@ -346,7 +346,7 @@ style:
   NestedClassesVisibility:
     active: false
   NewLineAtEndOfFile:
-    active: true
+    active: false
   NoTabs:
     active: false
   OptionalAbstractKeyword:
@@ -362,7 +362,7 @@ style:
   RedundantVisibilityModifierRule:
     active: false
   ReturnCount:
-    active: true
+    active: false
     max: 2
     excludedFunctions: [equals]
     excludeLabeled: false

diff --git a/app/src/androidTest/java/com/nextcloud/client/migrations/MockSharedPreferences.kt b/app/src/androidTest/java/com/nextcloud/client/migrations/MockSharedPreferences.kt
@@ -16,7 +16,6 @@ import java.util.TreeMap
  *
  * The implementation is not thread-safe.
  */
-@Suppress("TooManyFunctions")
 class MockSharedPreferences : SharedPreferences {
 
     class MockEditor(val store: MutableMap<String?, Any?>) : SharedPreferences.Editor {

diff --git a/app/src/androidTest/java/com/nextcloud/extensions/StringExtensionTests.kt b/app/src/androidTest/java/com/nextcloud/extensions/StringExtensionTests.kt
@@ -11,7 +11,6 @@ import junit.framework.TestCase.assertFalse
 import junit.framework.TestCase.assertTrue
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class StringExtensionTests {
     @Test
     fun testIsNotBlankAndEqualsWhenGivenBothStringsAreNull() {

diff --git a/app/src/androidTest/java/com/nextcloud/test/FileDeletionTests.kt b/app/src/androidTest/java/com/nextcloud/test/FileDeletionTests.kt
@@ -22,7 +22,7 @@ import org.junit.Test
 import java.io.File
 import kotlin.random.Random
 
-@Suppress("TooManyFunctions", "MagicNumber")
+@Suppress("MagicNumber")
 class FileDeletionTests : AbstractIT() {
 
     private val createdFilePaths = mutableListOf<String>()

diff --git a/app/src/androidTest/java/com/nextcloud/test/NextcloudViewMatchers.kt b/app/src/androidTest/java/com/nextcloud/test/NextcloudViewMatchers.kt
@@ -18,7 +18,6 @@ fun withSelectedText(expected: String): Matcher<View> = object : TypeSafeMatcher
         description.appendText("with selected text \"$expected\"")
     }
 
-    @Suppress("ReturnCount")
     override fun matchesSafely(view: View): Boolean {
         if (view !is TextView) return false
         val text = view.text?.toString() ?: ""

diff --git a/app/src/androidTest/java/com/nextcloud/test/RetryTestRule.kt b/app/src/androidTest/java/com/nextcloud/test/RetryTestRule.kt
@@ -27,7 +27,6 @@ class RetryTestRule(val retryCount: Int = defaultRetryValue) : TestRule {
 
     override fun apply(base: Statement, description: Description): Statement = statement(base, description)
 
-    @Suppress("TooGenericExceptionCaught") // and this exactly what we want here
     private fun statement(base: Statement, description: Description): Statement {
         return object : Statement() {
 

diff --git a/app/src/androidTest/java/com/nextcloud/utils/AutoRenameTests.kt b/app/src/androidTest/java/com/nextcloud/utils/AutoRenameTests.kt
@@ -18,7 +18,6 @@ import com.owncloud.android.lib.resources.status.OCCapability
 import org.junit.Before
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 @SinceServer(majorVersion = 30)
 class AutoRenameTests : AbstractOnServerIT() {
 

diff --git a/app/src/androidTest/java/com/nextcloud/utils/FileExtensionTest.kt b/app/src/androidTest/java/com/nextcloud/utils/FileExtensionTest.kt
@@ -12,7 +12,6 @@ import junit.framework.TestCase.assertFalse
 import junit.framework.TestCase.assertTrue
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class FileExtensionTest {
 
     @Test

diff --git a/app/src/androidTest/java/com/nextcloud/utils/FileHelperTest.kt b/app/src/androidTest/java/com/nextcloud/utils/FileHelperTest.kt
@@ -15,7 +15,6 @@ import org.junit.Test
 import java.io.File
 import java.nio.file.Files
 
-@Suppress("TooManyFunctions")
 class FileHelperTest {
 
     private lateinit var testDirectory: File

diff --git a/app/src/androidTest/java/com/nextcloud/utils/FileNameValidatorTests.kt b/app/src/androidTest/java/com/nextcloud/utils/FileNameValidatorTests.kt
@@ -21,7 +21,6 @@ import org.junit.Assert.assertTrue
 import org.junit.Before
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class FileNameValidatorTests : AbstractOnServerIT() {
     companion object {
         private const val MAX_FILENAME_SIZE = 250

diff --git a/app/src/androidTest/java/com/nextcloud/utils/OCCapabilityJsonToListTests.kt b/app/src/androidTest/java/com/nextcloud/utils/OCCapabilityJsonToListTests.kt
@@ -17,7 +17,7 @@ import junit.framework.TestCase.assertEquals
 import junit.framework.TestCase.assertTrue
 import org.junit.Test
 
-@Suppress("MagicNumber", "TooManyFunctions")
+@Suppress("MagicNumber")
 class OCCapabilityJsonToListTests : AbstractIT() {
     private var capability: OCCapability = fileDataStorageManager.getCapability(account.name)
 

diff --git a/app/src/androidTest/java/com/nextcloud/utils/SharePermissionManagerTest.kt b/app/src/androidTest/java/com/nextcloud/utils/SharePermissionManagerTest.kt
@@ -18,7 +18,6 @@ import junit.framework.TestCase.assertFalse
 import junit.framework.TestCase.assertTrue
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class SharePermissionManagerTest {
 
     private fun createShare(sharePermission: Int, isFolder: Boolean = false, attributesJson: String? = null): OCShare =

diff --git a/app/src/androidTest/java/com/nextcloud/utils/WebDavParentPathTests.kt b/app/src/androidTest/java/com/nextcloud/utils/WebDavParentPathTests.kt
@@ -11,7 +11,6 @@ import com.nextcloud.utils.extensions.webDavParentPath
 import org.junit.Assert.assertEquals
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class WebDavParentPathTests {
 
     @Test

diff --git a/app/src/androidTest/java/com/owncloud/android/UploadIT.kt b/app/src/androidTest/java/com/owncloud/android/UploadIT.kt
@@ -40,7 +40,7 @@ import java.nio.file.attribute.BasicFileAttributes
 import java.util.concurrent.TimeUnit
 import kotlin.math.abs
 
-@Suppress("TooManyFunctions", "DEPRECATION")
+@Suppress("DEPRECATION")
 class UploadIT : AbstractOnServerIT() {
     private val uploadsStorageManager = UploadsStorageManager(
         UserAccountManagerImpl.fromContext(targetContext),

diff --git a/app/src/androidTest/java/com/owncloud/android/authentication/AuthenticatorActivityIT.kt b/app/src/androidTest/java/com/owncloud/android/authentication/AuthenticatorActivityIT.kt
@@ -18,7 +18,6 @@ class AuthenticatorActivityIT {
     }
 
     @Test
-    @Suppress("TooGenericExceptionCaught")
     fun tryCatch() {
         val color = try {
             Color.parseColor("1")
@@ -30,7 +29,6 @@ class AuthenticatorActivityIT {
     }
 
     @Test
-    @Suppress("TooGenericExceptionCaught")
     fun tryCatch2() {
         val color = try {
             Color.parseColor("")
@@ -42,7 +40,6 @@ class AuthenticatorActivityIT {
     }
 
     @Test
-    @Suppress("TooGenericExceptionCaught")
     fun tryCatch3() {
         val color = try {
             Color.parseColor(null)
@@ -54,7 +51,6 @@ class AuthenticatorActivityIT {
     }
 
     @Test
-    @Suppress("TooGenericExceptionCaught")
     fun tryCatch4() {
         val color = try {
             Color.parseColor("abc")

diff --git a/app/src/androidTest/java/com/owncloud/android/operations/InternalTwoWaySyncIT.kt b/app/src/androidTest/java/com/owncloud/android/operations/InternalTwoWaySyncIT.kt
@@ -18,7 +18,7 @@ import org.junit.Assert.assertTrue
 import org.junit.Test
 import java.io.File
 
-@Suppress("TooManyFunctions", "DEPRECATION")
+@Suppress("DEPRECATION")
 class InternalTwoWaySyncIT : AbstractInternalTwoWaySyncIT() {
 
     companion object {

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/LoginIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/LoginIT.kt
@@ -52,7 +52,7 @@ class LoginIT : AbstractIT() {
      */
     @Test
     @Throws(InterruptedException::class)
-    @Suppress("MagicNumber", "SwallowedException")
+    @Suppress("MagicNumber")
     @SdkSuppress(minSdkVersion = Build.VERSION_CODES.Q)
     @Flaky("Login no longer uses an in-app WebView, so espresso-web (Espresso.onWebView) cannot drive it. ")
     fun login() {

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/adapter/UnifiedSearchListAdapterIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/adapter/UnifiedSearchListAdapterIT.kt
@@ -31,7 +31,6 @@ import javax.inject.Provider
 import org.junit.Before
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class UnifiedSearchListAdapterIT : AbstractIT() {
 
     private val testClassName = "com.owncloud.android.ui.adapter.UnifiedSearchListAdapterIT"

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/dialog/DialogFragmentIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/dialog/DialogFragmentIT.kt
@@ -78,7 +78,6 @@ import org.junit.Test
 import java.net.URI
 import java.util.function.Supplier
 
-@Suppress("TooManyFunctions")
 class DialogFragmentIT : AbstractIT() {
     private val testClassName = "com.owncloud.android.ui.dialog.DialogFragmentIT"
     private val serverUrl = "https://nextcloud.localhost"

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/fragment/FileDetailSharingFragmentIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/fragment/FileDetailSharingFragmentIT.kt
@@ -58,7 +58,6 @@ import org.junit.Before
 import org.junit.Rule
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class FileDetailSharingFragmentIT : AbstractIT() {
     private val testClassName = "com.owncloud.android.ui.fragment.FileDetailSharingFragmentIT"
 

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/fragment/GalleryFragmentIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/fragment/GalleryFragmentIT.kt
@@ -151,7 +151,6 @@ class GalleryFragmentIT : AbstractIT() {
     }
 
     /** Recursively walk view tree to find the first RecyclerView. Runs on the same thread that calls it. */
-    @Suppress("ReturnCount")
     private fun findRecyclerViewRecursively(root: View?): RecyclerView? {
         if (root == null) return null
         if (root is RecyclerView) return root

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/fragment/NotificationsFragmentIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/fragment/NotificationsFragmentIT.kt
@@ -115,7 +115,6 @@ class NotificationsFragmentIT : AbstractIT() {
         add(buildNotificationManyActions())
     }
 
-    @Suppress("ReturnCount")
     private fun findFragment(sut: NavigatorActivity): NotificationsFragment? {
         val allFragments = sut.supportFragmentManager.fragments
         for (f in allFragments) {

diff --git a/app/src/androidTest/java/com/owncloud/android/ui/trashbin/TrashbinFragmentIT.kt b/app/src/androidTest/java/com/owncloud/android/ui/trashbin/TrashbinFragmentIT.kt
@@ -32,7 +32,6 @@ class TrashbinFragmentIT : AbstractIT() {
         FILES
     }
 
-    @Suppress("ReturnCount")
     private fun findFragment(sut: NavigatorActivity): TrashbinFragment? {
         val allFragments = sut.supportFragmentManager.fragments
         for (f in allFragments) {

diff --git a/app/src/androidTest/java/com/owncloud/android/utils/EncryptionUtilsV2IT.kt b/app/src/androidTest/java/com/owncloud/android/utils/EncryptionUtilsV2IT.kt
@@ -32,7 +32,7 @@ import org.junit.Assert.assertFalse
 import org.junit.Assert.assertNotEquals
 import org.junit.Test
 
-@Suppress("TooManyFunctions", "LargeClass")
+@Suppress("LargeClass")
 class EncryptionUtilsV2IT : EncryptionIT() {
     private val encryptionTestUtils = EncryptionTestUtils()
     private val encryptionUtilsV2 = EncryptionUtilsV2()

diff --git a/app/src/androidTest/java/com/owncloud/android/utils/FileUploadHelperTest.kt b/app/src/androidTest/java/com/owncloud/android/utils/FileUploadHelperTest.kt
@@ -30,7 +30,6 @@ import org.junit.Assert.assertNull
 import org.junit.Before
 import org.junit.Test
 
-@Suppress("TooManyFunctions")
 class FileUploadHelperTest {
 
     private lateinit var uploadDao: UploadDao
@@ -49,7 +48,6 @@ class FileUploadHelperTest {
         fileUploadHelper.fileStorageManager = mockFileStorageManager
     }
 
-    @Suppress("LongParameterList")
     private fun buildEntity(
         id: Int = 1,
         localPath: String = this.localPath,

diff --git a/app/src/main/java/com/nextcloud/client/assistant/AssistantScreen.kt b/app/src/main/java/com/nextcloud/client/assistant/AssistantScreen.kt
@@ -88,7 +88,7 @@ import kotlinx.coroutines.withContext
 private const val CHAT_INPUT_DELAY = 100L
 private const val PULL_TO_REFRESH_DELAY = 1500L
 
-@Suppress("LongMethod")
+@Suppress("LongMethod", "CyclomaticComplexMethod")
 @OptIn(ExperimentalMaterial3Api::class)
 @Composable
 fun AssistantScreen(

diff --git a/app/src/main/java/com/nextcloud/client/assistant/AssistantViewModel.kt b/app/src/main/java/com/nextcloud/client/assistant/AssistantViewModel.kt
@@ -28,7 +28,6 @@ import kotlinx.coroutines.flow.update
 import kotlinx.coroutines.isActive
 import kotlinx.coroutines.launch
 
-@Suppress("TooManyFunctions")
 class AssistantViewModel(
     private val accountName: String,
     private val remoteRepository: AssistantRemoteRepository,

diff --git a/app/src/main/java/com/nextcloud/client/assistant/chat/ChatContent.kt b/app/src/main/java/com/nextcloud/client/assistant/chat/ChatContent.kt
@@ -5,7 +5,7 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
-@file:Suppress("TopLevelPropertyNaming", "MagicNumber")
+@file:Suppress("MagicNumber")
 
 package com.nextcloud.client.assistant.chat
 

diff --git a/app/src/main/java/com/nextcloud/client/assistant/conversation/ConversationScreen.kt b/app/src/main/java/com/nextcloud/client/assistant/conversation/ConversationScreen.kt
@@ -5,7 +5,7 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
-@file:Suppress("TopLevelPropertyNaming", "MagicNumber")
+@file:Suppress("MagicNumber")
 
 package com.nextcloud.client.assistant.conversation
 

diff --git a/app/src/main/java/com/nextcloud/client/assistant/translate/TranslationScreen.kt b/app/src/main/java/com/nextcloud/client/assistant/translate/TranslationScreen.kt
@@ -146,7 +146,7 @@ fun TranslationScreen(viewModel: TranslationViewModel, assistantViewModel: Assis
     }
 }
 
-@Suppress("LongMethod", "LongParameterList")
+@Suppress("LongMethod")
 @Composable
 private fun TranslationSection(
     labelId: Int,

diff --git a/app/src/main/java/com/nextcloud/client/assistant/translate/TranslationScreenState.kt b/app/src/main/java/com/nextcloud/client/assistant/translate/TranslationScreenState.kt
@@ -11,7 +11,6 @@ import com.owncloud.android.R
 import com.owncloud.android.lib.resources.assistant.v2.model.TaskTypeData
 import com.owncloud.android.lib.resources.assistant.v2.model.toTranslationLanguages
 
-@Suppress("LongParameterList")
 sealed class TranslationScreenState(
     open val taskTypeData: TaskTypeData,
     open val source: TranslationSideState,

diff --git a/app/src/main/java/com/nextcloud/client/assistant/translate/TranslationViewModel.kt b/app/src/main/java/com/nextcloud/client/assistant/translate/TranslationViewModel.kt
@@ -84,7 +84,6 @@ class TranslationViewModel(private val remoteRepository: AssistantRemoteReposito
         }
     }
 
-    @Suppress("ReturnCount")
     private suspend fun pollTranslationResult() {
         val screenStateValue = _screenState.value
         if (screenStateValue is Uninitialized) {

diff --git a/app/src/main/java/com/nextcloud/client/core/Task.kt b/app/src/main/java/com/nextcloud/client/core/Task.kt
@@ -12,7 +12,6 @@ import java.util.concurrent.atomic.AtomicBoolean
  * This is a wrapper for a task function running in background.
  * It executes task function and handles result or error delivery.
  */
-@Suppress("LongParameterList")
 internal class Task<T, P>(
     private val postResult: (Runnable) -> Boolean,
     private val removeFromQueue: (Runnable) -> Boolean,
@@ -32,7 +31,6 @@ internal class Task<T, P>(
         postResult(Runnable { onProgress?.invoke(p) })
     }
 
-    @Suppress("TooGenericExceptionCaught") // this is exactly what we want here
     override fun run() {
         try {
             val result = taskBody.invoke({ postProgress(it) }, this::isCancelled)

diff --git a/app/src/main/java/com/nextcloud/client/database/NextcloudDatabase.kt b/app/src/main/java/com/nextcloud/client/database/NextcloudDatabase.kt
@@ -109,7 +109,6 @@ import com.owncloud.android.db.ProviderMeta
     ],
     exportSchema = true
 )
-@Suppress("Detekt.UnnecessaryAbstractClass") // needed by Room
 @TypeConverters(OfflineOperationTypeConverter::class)
 abstract class NextcloudDatabase : RoomDatabase() {
 

diff --git a/app/src/main/java/com/nextcloud/client/database/dao/FileDao.kt b/app/src/main/java/com/nextcloud/client/database/dao/FileDao.kt
@@ -14,7 +14,6 @@ import com.nextcloud.client.database.entity.FileEntity
 import com.owncloud.android.db.ProviderMeta.ProviderTableMeta
 import com.owncloud.android.utils.MimeType
 
-@Suppress("TooManyFunctions")
 @Dao
 interface FileDao {
     @Update