Skip to content

Commit 3cfd6c4

Browse files
jkmasselclaude
andcommitted
Fix detekt violations in extracted files
Suppress SpreadOperator warning in UiHelpers.getTextOfUiString() and MagicNumber warning in adjustDialogSize() — both pre-existing violations newly surfaced by the module split. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 1d37243 commit 3cfd6c4

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

  • WordPress/src/main/java/org/wordpress/android/ui/utils
  • libs/core-ui/src/main/java/org/wordpress/android/ui/utils

WordPress/src/main/java/org/wordpress/android/ui/utils/UiHelpersExt.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ fun adjustDialogSize(dialog: Dialog) {
4949

5050
val maximumWidth = window.context.resources
5151
.getDimension(R.dimen.alert_dialog_max_width).toInt()
52+
@Suppress("MagicNumber")
5253
var proposedWidth = (width * 0.8).toInt()
5354

5455
if (proposedWidth > maximumWidth) {

libs/core-ui/src/main/java/org/wordpress/android/ui/utils/UiHelpers.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@ open class UiHelpers @Inject constructor() {
3232
when (uiString) {
3333
is UiStringRes -> context.getString(uiString.stringRes)
3434
is UiStringText -> uiString.text
35-
is UiStringResWithParams -> context.getString(
35+
is UiStringResWithParams -> @Suppress("SpreadOperator")
36+
context.getString(
3637
uiString.stringRes,
3738
*uiString.params.map { value ->
3839
getTextOfUiString(context, value)

0 commit comments

Comments
 (0)