From cfceb38ee72896dad3701dcf3f12d8480166e009 Mon Sep 17 00:00:00 2001 From: Harsh Shandilya Date: Mon, 10 Jan 2022 00:31:03 +0530 Subject: [PATCH] Fix a bunch of Lint warnings and errors (#1655) * autofill-parser: add missing `@RequiresApi` annotations * app: remove unused strings and sync with Crowdin * app: remove unused chip_text_color selector * app: remove unused capitalization_type_values array * app: add `PendingIntent.FLAG_IMMUTABLE` flag where required --- .../msfjarvis/aps/ui/autofill/AutofillDecryptActivity.kt | 2 +- .../aps/ui/autofill/AutofillDecryptActivityV2.kt | 2 +- .../dev/msfjarvis/aps/ui/autofill/AutofillFilterView.kt | 2 +- .../aps/ui/autofill/AutofillPublisherChangedActivity.kt | 2 +- .../dev/msfjarvis/aps/util/autofill/AutofillViewUtils.kt | 7 ++++++- .../dev/msfjarvis/aps/util/services/ClipboardService.kt | 7 ++++++- app/src/main/res/color/chip_text_color.xml | 9 --------- app/src/main/res/values-de/strings.xml | 1 - app/src/main/res/values-fr/strings.xml | 2 -- app/src/main/res/values-gl/strings.xml | 7 ++----- app/src/main/res/values-it/strings.xml | 3 --- app/src/main/res/values-pt-rBR/strings.xml | 3 --- app/src/main/res/values-ru/strings.xml | 9 +++------ app/src/main/res/values/arrays.xml | 6 ------ app/src/main/res/values/strings.xml | 3 --- .../autofillparser/AutofillScenario.kt | 2 ++ .../autofillparser/AutofillStrategyDsl.kt | 1 + 17 files changed, 24 insertions(+), 44 deletions(-) delete mode 100644 app/src/main/res/color/chip_text_color.xml diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivity.kt b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivity.kt index 282864e1..3ea11982 100644 --- a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivity.kt +++ b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivity.kt @@ -79,7 +79,7 @@ class AutofillDecryptActivity : AppCompatActivity() { context, decryptFileRequestCode++, intent, - PendingIntent.FLAG_CANCEL_CURRENT + PendingIntent.FLAG_CANCEL_CURRENT or PendingIntent.FLAG_IMMUTABLE, ) .intentSender } diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivityV2.kt b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivityV2.kt index 697d628d..4401308a 100644 --- a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivityV2.kt +++ b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillDecryptActivityV2.kt @@ -67,7 +67,7 @@ class AutofillDecryptActivityV2 : AppCompatActivity() { context, decryptFileRequestCode++, intent, - PendingIntent.FLAG_CANCEL_CURRENT + PendingIntent.FLAG_CANCEL_CURRENT or PendingIntent.FLAG_IMMUTABLE, ) .intentSender } diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillFilterView.kt b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillFilterView.kt index 55f3fc11..7b3477cd 100644 --- a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillFilterView.kt +++ b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillFilterView.kt @@ -74,7 +74,7 @@ class AutofillFilterView : AppCompatActivity() { context, matchAndDecryptFileRequestCode++, intent, - PendingIntent.FLAG_CANCEL_CURRENT + PendingIntent.FLAG_CANCEL_CURRENT or PendingIntent.FLAG_IMMUTABLE, ) .intentSender } diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillPublisherChangedActivity.kt b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillPublisherChangedActivity.kt index e37ebf5e..00a39dc0 100644 --- a/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillPublisherChangedActivity.kt +++ b/app/src/main/java/dev/msfjarvis/aps/ui/autofill/AutofillPublisherChangedActivity.kt @@ -54,7 +54,7 @@ class AutofillPublisherChangedActivity : AppCompatActivity() { context, publisherChangedRequestCode++, intent, - PendingIntent.FLAG_CANCEL_CURRENT + PendingIntent.FLAG_CANCEL_CURRENT or PendingIntent.FLAG_IMMUTABLE, ) .intentSender } diff --git a/app/src/main/java/dev/msfjarvis/aps/util/autofill/AutofillViewUtils.kt b/app/src/main/java/dev/msfjarvis/aps/util/autofill/AutofillViewUtils.kt index 1d09d001..5a224f39 100644 --- a/app/src/main/java/dev/msfjarvis/aps/util/autofill/AutofillViewUtils.kt +++ b/app/src/main/java/dev/msfjarvis/aps/util/autofill/AutofillViewUtils.kt @@ -52,7 +52,12 @@ fun makeInlinePresentation( if (UiVersions.INLINE_UI_VERSION_1 !in UiVersions.getVersions(imeSpec.style)) return null val launchIntent = - PendingIntent.getActivity(context, 0, Intent(context, PasswordStore::class.java), 0) + PendingIntent.getActivity( + context, + 0, + Intent(context, PasswordStore::class.java), + PendingIntent.FLAG_IMMUTABLE + ) val slice = InlineSuggestionUi.newContentBuilder(launchIntent).run { setTitle(metadata.title) diff --git a/app/src/main/java/dev/msfjarvis/aps/util/services/ClipboardService.kt b/app/src/main/java/dev/msfjarvis/aps/util/services/ClipboardService.kt index 84de5c91..808089ab 100644 --- a/app/src/main/java/dev/msfjarvis/aps/util/services/ClipboardService.kt +++ b/app/src/main/java/dev/msfjarvis/aps/util/services/ClipboardService.kt @@ -115,7 +115,12 @@ class ClipboardService : Service() { if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) { PendingIntent.getForegroundService(this, 0, clearIntent, PendingIntent.FLAG_UPDATE_CURRENT) } else { - PendingIntent.getService(this, 0, clearIntent, PendingIntent.FLAG_UPDATE_CURRENT) + PendingIntent.getService( + this, + 0, + clearIntent, + PendingIntent.FLAG_UPDATE_CURRENT or PendingIntent.FLAG_IMMUTABLE + ) } val notification = if (Build.VERSION.SDK_INT <= Build.VERSION_CODES.M) { diff --git a/app/src/main/res/color/chip_text_color.xml b/app/src/main/res/color/chip_text_color.xml deleted file mode 100644 index 38d9a0ea..00000000 --- a/app/src/main/res/color/chip_text_color.xml +++ /dev/null @@ -1,9 +0,0 @@ - - - - - - diff --git a/app/src/main/res/values-de/strings.xml b/app/src/main/res/values-de/strings.xml index 3d3279ca..9c73d68a 100644 --- a/app/src/main/res/values-de/strings.xml +++ b/app/src/main/res/values-de/strings.xml @@ -227,7 +227,6 @@ Alles löschen Abbrechen und hochladen Biometrische Abfrage - Authentifizierungsfehler Authentifizierungsfehler: %s Biometrische Authentifizierung aktivieren Wenn aktiviert, werden Sie beim Starten der App nach Ihrem Fingerabdruck gefragt diff --git a/app/src/main/res/values-fr/strings.xml b/app/src/main/res/values-fr/strings.xml index 86ad9793..732bfe94 100644 --- a/app/src/main/res/values-fr/strings.xml +++ b/app/src/main/res/values-fr/strings.xml @@ -231,7 +231,6 @@ Tout supprimer Abandonner et pousser Identification biométrique - Échec de l\'authentification Échec de l\'authentification : %s Activer l\'authentification biométrique Lorsque cette option est activée, Password Store vous demandera votre empreinte digitale au lancement @@ -318,7 +317,6 @@ Un fichier portant ce nom existe déjà Un dossier portant ce nom existe déjà - Chiffres(d)/Symboles(s) Emplacement du dépôt \n Allons-y diff --git a/app/src/main/res/values-gl/strings.xml b/app/src/main/res/values-gl/strings.xml index 2c18beaf..832f083f 100644 --- a/app/src/main/res/values-gl/strings.xml +++ b/app/src/main/res/values-gl/strings.xml @@ -182,7 +182,7 @@ Controlar a visibilidade do contrasinal unha vez descifrado, esto non desactiva o copiado do contrasinal Crear Actualizar lista - Enviar contrasinal como texto plano usando... + Enviar contrasinal como texto plano usando… Icona da app Elixe o elemento a completar @@ -233,7 +233,6 @@ a app desde unha fonte de confianza, como a Play Store, Amazon Appstore, F-Droid Eliminar todo Abortar e Push Petición biométrica - Fallo de autenticación Fallou a autenticación: %s Activar autenticación biométrica Ao activala, Password Store vaiche pedir a túa pegada dactilar ao iniciar a app @@ -304,7 +303,7 @@ a app desde unha fonte de confianza, como a Play Store, Amazon Appstore, F-Droid Fallou a acción pull, estás nun head diferente. Utiliza \"axustes > utilidades git\", garda os cambios no remoto nunha nova rama e resolve o conflicto nun ordenador. Fallou o merge, hai un conflito nos datos. TAREFA: engadir método de recuperación. O push foi rexeitado polo remoto, executa pull antes de voltar a subilos de novo. Podes usar Sincronizar mellor que push/pull xa que inclú ambos. - O push foi rexeitado polo remoto, razón: + O push foi rexeitado polo remoto, razón: %1$s O remoto rexeitou o push non-fast-forward. Comproba a variable receive.denyNonFastForwards no ficheiro de configuración do repositorio de destino. Host descoñecido: %1$s Realizando operación git… @@ -320,8 +319,6 @@ a app desde unha fonte de confianza, como a Play Store, Amazon Appstore, F-Droid Xa existe un ficheiro con ese nome Xa existe un cartafol con ese nome - Díxitos/Símbolos (d/s) - ds Repositorio \nLocalización Imos alá diff --git a/app/src/main/res/values-it/strings.xml b/app/src/main/res/values-it/strings.xml index 512061e8..2ec66e24 100644 --- a/app/src/main/res/values-it/strings.xml +++ b/app/src/main/res/values-it/strings.xml @@ -226,7 +226,6 @@ Hai selezionato il root della tua sdcard per il negozio. Questo è estremamente pericoloso e perderai i tuoi dati poiché i suoi contenuti saranno, infine, eliminati Interrompi e Spingi Richiesta Biometrica - Autenticazione non riuscita Autenticazione non riuscita: %s Abilita autenticazione biometrica Quando abilitata, il Password Store ti chiederà la tua impronta digitale al lancio dell\'app @@ -312,8 +311,6 @@ Un file con quel nome esiste già Una cartella con quel nome esiste già - Cifre/Simboli (c/s) - cs Repository \nLocation Andiamo diff --git a/app/src/main/res/values-pt-rBR/strings.xml b/app/src/main/res/values-pt-rBR/strings.xml index e1bf643b..e3fddf98 100644 --- a/app/src/main/res/values-pt-rBR/strings.xml +++ b/app/src/main/res/values-pt-rBR/strings.xml @@ -232,7 +232,6 @@ Remover tudo Abortar e executar Push Confirmação Biométrica - Falha de autenticação Falha de autenticação: %s Ativar autenticação biométrica Quando ativado, o Password Store irá pedir a sua impressão digital ao iniciar o aplicativo @@ -319,8 +318,6 @@ Já existe um arquivo com esse nome Já existe uma pasta com esse nome - Dígitos/Símbolos (d/s) - ds Local \ndo repositório Vamos lá diff --git a/app/src/main/res/values-ru/strings.xml b/app/src/main/res/values-ru/strings.xml index 847aa204..a12e92d9 100644 --- a/app/src/main/res/values-ru/strings.xml +++ b/app/src/main/res/values-ru/strings.xml @@ -188,11 +188,11 @@ Совпадает с %1$s Очистить существующие совпадения Не найдено. - Искать в хранилище... + Искать в хранилище Сохранение не удалось из-за внутренней ошибки Это приложение в настоящее время не поддерживается Пароли не совпадают - Сгенерировать пароль... + Сгенерировать пароль Извлечение кодов из SMS… Достигнуто максимальное количество совпадений (%1$d); очистите совпадения перед тем как добавите новые. Издатель приложения изменился с тех пор как вы первый раз связали с ним запись хранилища паролей: @@ -202,7 +202,7 @@ Оставить автозаполнение отключенным Включить автозаполнение снова Предупреждение - Нажмите для получения подробностей... + Нажмите для получения подробностей Возможная попытка фишинга Заполнить и сохранить учетные данные Заполнить учетные данные @@ -229,7 +229,6 @@ Вы выбрали корень вашей sd-карты для хранения. Это очень опасно и вы потеряете ваши данные, поскольку они будут в конечном итоге удалены Прервать и записать изменения Запрос биометрии - Ошибка авторизации Ошибка аутентификации: %s Включить биометрическую аутентификацию Когда ключено, Password Store будет запрашивать ваш опечаток пальца при каждом запуске приложения @@ -315,8 +314,6 @@ Файл с таким именем уже существует Папка с таким именем уже существует - Цыфры / Символы (d/s) - ds Расположение \nрепозитория Поехали diff --git a/app/src/main/res/values/arrays.xml b/app/src/main/res/values/arrays.xml index 3c1568d6..495e880e 100644 --- a/app/src/main/res/values/arrays.xml +++ b/app/src/main/res/values/arrays.xml @@ -16,12 +16,6 @@ INDEPENDENT RECENTLY_USED - - lowercase - UPPERCASE - TitleCase - Sentence - Classic Diceware diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 432ee6ea..7b476587 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -258,7 +258,6 @@ Remove everything Abort and Push Biometric Prompt - Authentication failure Authentication failure: %s Enable biometric authentication When enabled, Password Store will prompt you for your fingerprint when launching the app @@ -352,8 +351,6 @@ A file by that name already exists A folder by that name already exists - Digits/Symbols (d/s) - ds Repository \nLocation diff --git a/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillScenario.kt b/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillScenario.kt index fec33285..66ef6d0e 100644 --- a/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillScenario.kt +++ b/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillScenario.kt @@ -204,6 +204,7 @@ internal data class GenericAutofillScenario( get() = genericPassword } +@RequiresApi(Build.VERSION_CODES.O) internal fun AutofillScenario.passesOriginCheck(singleOriginMode: Boolean): Boolean { return if (singleOriginMode) { // In single origin mode, only the browsers URL bar (which is never filled) should have @@ -235,6 +236,7 @@ public fun Dataset.Builder.fillWith( } } +@RequiresApi(Build.VERSION_CODES.O) internal inline fun AutofillScenario.map( transform: (T) -> S ): AutofillScenario { diff --git a/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillStrategyDsl.kt b/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillStrategyDsl.kt index 6d2414d2..9348fab8 100644 --- a/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillStrategyDsl.kt +++ b/autofill-parser/src/main/java/com/github/androidpasswordstore/autofillparser/AutofillStrategyDsl.kt @@ -432,5 +432,6 @@ internal class AutofillStrategy private constructor(private val rules: List Unit) = AutofillStrategy.Builder().apply(block).build()