Skip to content

Commit 014a615

Browse files
jkmasselclaude
andcommitted
Suppress detekt LongMethod for application password error handling
These methods are long due to exhaustive error handling across multiple WpRequestResult variants, which is reasonable to suppress. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 90471b8 commit 014a615

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

WordPress/src/main/java/org/wordpress/android/ui/accounts/login/applicationpassword/ApplicationPasswordAutoAuthDialogViewModel.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ class ApplicationPasswordAutoAuthDialogViewModel @Inject constructor(
4747
private val _isLoading = MutableStateFlow(false)
4848
val isLoading: StateFlow<Boolean> = _isLoading.asStateFlow()
4949

50-
@Suppress("TooGenericExceptionCaught")
50+
@Suppress("TooGenericExceptionCaught", "LongMethod")
5151
fun createApplicationPassword(site: SiteModel) {
5252
viewModelScope.launch {
5353
try {

WordPress/src/main/java/org/wordpress/android/ui/mysite/cards/applicationpassword/ApplicationPasswordViewModelSlice.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ class ApplicationPasswordViewModelSlice @Inject constructor(
6363
}
6464
}
6565

66-
@Suppress("TooGenericExceptionCaught")
66+
@Suppress("TooGenericExceptionCaught", "LongMethod")
6767
private fun validateCredentialsAndBuildCard(site: SiteModel) {
6868
appLogWrapper.d(AppLog.T.MAIN, "A_P: Validating credentials for ${site.url}")
6969
appLogWrapper.d(AppLog.T.MAIN, "A_P: isUsingSelfHostedRestApi: ${site.isUsingSelfHostedRestApi}")

0 commit comments

Comments
 (0)