From e7463ec24c929860f0a7311b558f496919d54d20 Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <msfjarvis@gmail.com>
Date: Thu, 28 May 2020 22:42:13 +0530
Subject: Remove HOTP/TOTP support (#806)

---
 app/src/main/res/layout/decrypt_layout.xml     | 43 ++------------------------
 app/src/main/res/layout/otp_confirm_layout.xml | 23 --------------
 app/src/main/res/values-ar/strings.xml         |  1 -
 app/src/main/res/values-cs/strings.xml         |  2 --
 app/src/main/res/values-es/strings.xml         | 10 ------
 app/src/main/res/values-fr/strings.xml         | 10 ------
 app/src/main/res/values-ru/strings.xml         | 10 ------
 app/src/main/res/values/strings.xml            | 11 -------
 app/src/main/res/xml/preference.xml            |  3 --
 9 files changed, 2 insertions(+), 111 deletions(-)
 delete mode 100644 app/src/main/res/layout/otp_confirm_layout.xml

(limited to 'app/src/main/res')

diff --git a/app/src/main/res/layout/decrypt_layout.xml b/app/src/main/res/layout/decrypt_layout.xml
index c9f3f922..932eefd6 100644
--- a/app/src/main/res/layout/decrypt_layout.xml
+++ b/app/src/main/res/layout/decrypt_layout.xml
@@ -160,56 +160,17 @@
                 app:layout_constraintTop_toBottomOf="@id/crypto_username_show_label"
                 tools:visibility="visible" />
 
-            <androidx.appcompat.widget.AppCompatImageView
-                android:id="@+id/crypto_copy_otp"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:layout_below="@id/crypto_username_show"
-                android:layout_alignParentEnd="true"
-                android:background="?android:attr/windowBackground"
-                android:contentDescription="@string/copy_otp"
-                android:src="@drawable/ic_content_copy"
-                android:visibility="invisible"
-                app:layout_constraintEnd_toEndOf="parent"
-                app:layout_constraintTop_toTopOf="@id/crypto_otp_show_label"
-                tools:visibility="visible" />
-
-            <androidx.appcompat.widget.AppCompatTextView
-                android:id="@+id/crypto_otp_show_label"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:layout_below="@id/crypto_username_show"
-                android:layout_alignParentStart="true"
-                android:layout_toStartOf="@id/crypto_copy_otp"
-                android:text="@string/otp"
-                android:textColor="?android:attr/textColor"
-                android:textStyle="bold"
-                app:layout_constraintStart_toStartOf="parent"
-                app:layout_constraintTop_toBottomOf="@id/crypto_username_show" />
-
-            <androidx.appcompat.widget.AppCompatTextView
-                android:id="@+id/crypto_otp_show"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layout_below="@id/crypto_otp_show_label"
-                android:layout_alignParentStart="true"
-                android:layout_toStartOf="@id/crypto_copy_otp"
-                android:textColor="?android:attr/textColor"
-                android:textIsSelectable="true"
-                android:typeface="monospace"
-                app:layout_constraintTop_toBottomOf="@id/crypto_otp_show_label" />
-
             <androidx.appcompat.widget.AppCompatTextView
                 android:id="@+id/crypto_extra_show_label"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:layout_below="@id/crypto_otp_show"
+                android:layout_below="@id/crypto_username_show"
                 android:layout_alignParentStart="true"
                 android:text="@string/extra_content"
                 android:textColor="?android:attr/textColor"
                 android:textStyle="bold"
                 app:layout_constraintStart_toStartOf="parent"
-                app:layout_constraintTop_toBottomOf="@id/crypto_otp_show" />
+                app:layout_constraintTop_toBottomOf="@id/crypto_username_show" />
 
             <androidx.appcompat.widget.AppCompatTextView
                 android:id="@+id/crypto_extra_show"
diff --git a/app/src/main/res/layout/otp_confirm_layout.xml b/app/src/main/res/layout/otp_confirm_layout.xml
deleted file mode 100644
index 7db7b105..00000000
--- a/app/src/main/res/layout/otp_confirm_layout.xml
+++ /dev/null
@@ -1,23 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?><!--
-  ~ Copyright © 2014-2020 The Android Password Store Authors. All Rights Reserved.
-  ~ SPDX-License-Identifier: GPL-3.0-only
-  -->
-
-<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    android:layout_width="match_parent"
-    android:layout_height="match_parent">
-
-    <CheckBox
-        android:id="@+id/hotp_remember_checkbox"
-        android:layout_width="0dp"
-        android:layout_height="wrap_content"
-        android:layout_marginStart="16dp"
-        android:layout_marginTop="8dp"
-        android:layout_marginEnd="16dp"
-        android:text="@string/dialog_update_check"
-        app:layout_constraintEnd_toEndOf="parent"
-        app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent" />
-
-</androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/values-ar/strings.xml b/app/src/main/res/values-ar/strings.xml
index 380d58b7..ecb03154 100644
--- a/app/src/main/res/values-ar/strings.xml
+++ b/app/src/main/res/values-ar/strings.xml
@@ -63,7 +63,6 @@
     <string name="edit_password">تعديل كلمة السر</string>
     <string name="copy_password">نسخ كلمة السر</string>
     <string name="copy_username">نسخ إسم المستخدم</string>
-    <string name="copy_otp">نسخ رمز الـ OTP</string>
     <string name="share_as_plaintext">شارك كنص مجرد</string>
     <string name="last_changed">آخِر تعديل %s</string>
     <!-- Preferences -->
diff --git a/app/src/main/res/values-cs/strings.xml b/app/src/main/res/values-cs/strings.xml
index 5b75d8d5..420c0ce4 100644
--- a/app/src/main/res/values-cs/strings.xml
+++ b/app/src/main/res/values-cs/strings.xml
@@ -34,7 +34,6 @@
     <string name="clipboard_password_toast_text">Heslo zkopírováno do schránky, máte %d sekund na jeho zkopírování.</string>
     <string name="clipboard_password_no_clear_toast_text">Heslo zkopírováno do schránky</string>
     <string name="clipboard_username_toast_text">Jméno zkopírováno do schránky</string>
-    <string name="clipboard_otp_toast_text">OTP kód zkopírován do schránky</string>
     <string name="file_toast_text">Zadejte prosím jméno souboru</string>
     <string name="path_toast_text">Prosím zadejte cestu k souboru</string>
     <string name="empty_toast_text">Nelze zadat prázdné heslo nebo další obsah</string>
@@ -102,7 +101,6 @@
     <string name="edit_password">Editovat heslo</string>
     <string name="copy_password">Kopírovat heslo</string>
     <string name="copy_username">Kopírovat jméno</string>
-    <string name="copy_otp">Kopírovat OTP kód</string>
     <string name="share_as_plaintext">Sdílet v nešifrované podobě</string>
     <string name="last_changed">Naposled změněno %s</string>
     <!-- Preferences -->
diff --git a/app/src/main/res/values-es/strings.xml b/app/src/main/res/values-es/strings.xml
index 8922fbdf..975f088f 100644
--- a/app/src/main/res/values-es/strings.xml
+++ b/app/src/main/res/values-es/strings.xml
@@ -24,14 +24,12 @@
     <string name="git_commit_edit_text">"%1$s editada usando Android Password Store."</string>
     <string name="git_commit_remove_text">"%1$s eliminada del almacén usando Android Password Store."</string>
     <string name="git_commit_move_text">"Renombrado %1$s a %2$s usando Android Password Store.."</string>
-    <string name="git_commit_increment_text">"Contador de HOTP incrementado para %1$s."</string>
 
     <!-- PGPHandler -->
     <string name="provider_toast_text">¡No se ha seleccionado ningún proveedor OpenGPG!</string>
     <string name="clipboard_password_toast_text">Contraseña copiada al portapapeles, tienes %d segundos para pegarla.</string>
     <string name="clipboard_password_no_clear_toast_text">Contraseña copiada al portapapeles</string>
     <string name="clipboard_username_toast_text">Nombre de usuario copiado al portapapeles</string>
-    <string name="clipboard_otp_toast_text">Código OTP copiado al portapapeles</string>
     <string name="file_toast_text">Por favor selecciona un nombre de archivo</string>
     <string name="empty_toast_text">No puedes dejar la contraseña y el contenido extra ambos vacíos</string>
 
@@ -95,14 +93,8 @@
     <string name="edit_password">Editar contraseña</string>
     <string name="copy_password">Copiar contraseña</string>
     <string name="copy_username">Copiar nombre de usuario</string>
-    <string name="copy_otp">Copiar código OTP</string>
     <string name="share_as_plaintext">Compartir como texto plano</string>
     <string name="last_changed">Última modificación %s</string>
-    <string name="dialog_update_title">Atención</string>
-    <string name="dialog_update_positive">Actualizar registro</string>
-    <string name="dialog_update_negative">Dejar sin cambios</string>
-    <string name="dialog_update_body">El contador HOTP será incrementado. Este cambio será confirmado. Si presionas "Dejar sin cambios", el código HOTP será mostrado, pero el contador no se cambiará.</string>
-    <string name="dialog_update_check">Recordar elección</string>
 
     <!-- Preferences -->
     <string name="pref_edit_server_info">Editar ajustes del servidor Git</string>
@@ -216,12 +208,10 @@
     <string name="git_push_nff_error">La subida fue rechazada por el servidor, Ejecuta \'Descargar desde servidor\' antes de subir o pulsa \'Sincronizar con servidor\' para realizar ambas acciones.</string>
     <string name="git_push_generic_error">El envío fue rechazado por el servidor, la razón:</string>
     <string name="jgit_error_push_dialog_text">Ocurrió un error durante el envío:</string>
-    <string name="hotp_remember_clear_choice">Limpiar preferencia para incremento HOTP</string>
     <string name="remember_the_passphrase">Recordar contraseñagit  (inseguro)</string>
     <string name="hackish_tools">Hackish tools</string>
     <string name="abort_rebase">Abortar rebase</string>
     <string name="commit_hash">Hash del commit</string>
     <string name="crypto_extra_edit_hint">Username: Nombre de usuario\n… o algún contenido extra</string>
     <string name="get_last_changed_failed">Error al obtener la fecha de último cambio</string>
-    <string name="hotp_pending">Presiona copiar para calcular el HOTP</string>
 </resources>
diff --git a/app/src/main/res/values-fr/strings.xml b/app/src/main/res/values-fr/strings.xml
index ad5ad866..eb52acf9 100644
--- a/app/src/main/res/values-fr/strings.xml
+++ b/app/src/main/res/values-fr/strings.xml
@@ -31,13 +31,11 @@
     <string name="git_commit_edit_text">Editer %1$s depuis le dépôt.</string>
     <string name="git_commit_remove_text">Retirer %1$s de la mémoire.</string>
     <string name="git_commit_move_text">Renommer %1$sà %2$s. </string>
-    <string name="git_commit_increment_text">Incrémenter le compteur HOTP pour %1$s.</string>
 
     <!-- PGPHandler -->
     <string name="provider_toast_text">Aucun prestataire OpenPGP sélectionné !</string>
     <string name="clipboard_password_toast_text">Mot de passe copié dans le presse papier, vous avez %d secondes pour coller celui-ci.</string>
     <string name="clipboard_username_toast_text">Nom d\'utilisateur copié</string>
-    <string name="clipboard_otp_toast_text">Code OTP copié dans le presse-papier</string>
     <string name="file_toast_text">Renseignez un nom de fichier</string>
     <string name="empty_toast_text">Vous ne pouvez pas utiliser un mot de passe vide ou des données supplémentaires vide</string>
 
@@ -103,14 +101,8 @@
     <string name="edit_password">Éditer le mot de passe</string>
     <string name="copy_password">Copier le mot de passe</string>
     <string name="copy_username">Copier le nom d\'utilisateur</string>
-    <string name="copy_otp">Copier le code OTP</string>
     <string name="share_as_plaintext">Partager en clair</string>
     <string name="last_changed">Dernière modification le %s</string>
-    <string name="dialog_update_title">Attention</string>
-    <string name="dialog_update_positive">Mettre à jour l\'entrée</string>
-    <string name="dialog_update_negative">Laisser inchangé</string>
-    <string name="dialog_update_body">Le compteur HOTP est sur le point d’être incrémenté. Ce changement sera engagé. Si vous appuyez sur \"Laisser inchangé\", le code HOTP sera affiché mais le compteur ne sera pas modifié</string>
-    <string name="dialog_update_check">Se souvenir de mon choix</string>
 
     <!-- Preferences -->
     <string name="pref_edit_server_info">Editer les paramètres du serveur Git</string>
@@ -217,11 +209,9 @@
     <string name="git_push_generic_error">Poussée rejetée par le dépôt distant, raison:</string>
     <string name="git_push_other_error">Pousser au dépôt distant sans avance rapide rejetée. Vérifiez la variable receive.denyNonFastForwards dans le fichier de configuration du répertoire de destination.</string>
     <string name="jgit_error_push_dialog_text">Une erreur s\'est produite lors de l\'opération de poussée:</string>
-    <string name="hotp_remember_clear_choice">Effacer les préférences enregistrées pour l’incrémentation HOTP</string>
     <string name="remember_the_passphrase">Se rappeler de la phrase secrète dans la configuration de l\'application (peu sûr)</string>
     <string name="hackish_tools">Outils de hack</string>
     <string name="commit_hash">Commettre la clé</string>
     <string name="crypto_extra_edit_hint">nom d\'utilisateur: quelque chose d\'autre contenu supplémentaire</string>
     <string name="get_last_changed_failed">Failed to get last changed date</string>
-    <string name="hotp_pending">Appuyez sur copie pour calculer le HOTP</string>
 </resources>
diff --git a/app/src/main/res/values-ru/strings.xml b/app/src/main/res/values-ru/strings.xml
index ca7ffaa0..d286bbb4 100644
--- a/app/src/main/res/values-ru/strings.xml
+++ b/app/src/main/res/values-ru/strings.xml
@@ -31,14 +31,12 @@
     <string name="git_commit_edit_text">Отредактирован %1$s из хранилища.</string>
     <string name="git_commit_remove_text">Удалить %1$sиз хранилища.</string>
     <string name="git_commit_move_text">Переименовать %1$sв%2$s.</string>
-    <string name="git_commit_increment_text">Увеличить значение HOTP счетчика для %1$s</string>
 
     <!-- PGPHandler -->
     <string name="provider_toast_text">Не выбран провайдер OpenPGP!</string>
     <string name="clipboard_password_toast_text">Пароль скопирован в буфер обмена, у вас есть %d секунд чтобы вставить его.</string>
     <string name="clipboard_password_no_clear_toast_text">Пароль скопирован в буфер обмена</string>
     <string name="clipboard_username_toast_text">Имя пользователя скопировано в буфер обмена</string>
-    <string name="clipboard_otp_toast_text">OTP код скопирован в буфер обмена</string>
     <string name="file_toast_text">Пожалуйста, укажите имя файла</string>
     <string name="path_toast_text">Пожалуйста, задайте путь к файлу</string>
     <string name="empty_toast_text">Вы не можете использовать пустой пароль или пустое поле информации</string>
@@ -108,14 +106,8 @@
     <string name="edit_password">Редактировать пароль</string>
     <string name="copy_password">Скопировать пароль</string>
     <string name="copy_username">Скопировать имя пользователя</string>
-    <string name="copy_otp">Скопировать OTP код</string>
     <string name="share_as_plaintext">Поделиться в открытом виде</string>
     <string name="last_changed">Последние изменение %s</string>
-    <string name="dialog_update_title">Внимание</string>
-    <string name="dialog_update_positive">Обновить запись</string>
-    <string name="dialog_update_negative">Оставить без изменений</string>
-    <string name="dialog_update_body">HOTP счетчик будет увеличен. Это изменение будет сохранено. Если вы нажмете \"Оставить без изменений\", тогда HOTP код будет показан, но счетчик не изменится.</string>
-    <string name="dialog_update_check">Запомнить мой выбор</string>
 
     <!-- Preferences -->
     <string name="pref_edit_server_info">Изменить настройки сервера git</string>
@@ -278,7 +270,6 @@
     <string name="git_push_generic_error">Запись изменений была отклонена удаленным репозиторием, причина:</string>
     <string name="git_push_other_error">Удаленный репозиторий отклонил запись изменений без быстрой перемотки вперед. Проверьте переменную receive.denyNonFastForwards в файле конфигурации репозитория назначения.</string>
     <string name="jgit_error_push_dialog_text">В хоте операции записи изменений возникла ошибка:</string>
-    <string name="hotp_remember_clear_choice">Очистить сохраненные настройки для увеличения HOTP</string>
     <string name="remember_the_passphrase">Заполнить парольную фразу в конфигурации приложнеия (небезопасно)</string>
     <string name="hackish_tools">Костыльные инструменты</string>
     <string name="abort_rebase">Прервать перебазирование и записать изменения в новую ветку</string>
@@ -286,7 +277,6 @@
     <string name="commit_hash">Хэш-сумма изменений</string>
     <string name="crypto_extra_edit_hint">имя пользователя: какой-то другой дополнительный контент</string>
     <string name="get_last_changed_failed">Failed to get last changed date</string>
-    <string name="hotp_pending">Нажмите скопировать чтобы расчитать HOTP</string>
     <string name="openkeychain_ssh_api_connect_fail">Ошибка при подключении к сервису OpenKeychain SSH API</string>
     <string name="no_ssh_api_provider">Не найдено SSH API провайдеров. OpenKeychain установлен?</string>
     <string name="ssh_api_pending_intent_failed">Ожидаемое намерение SSH API не удалось</string>
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 1b2f9c52..9b76e05f 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -42,14 +42,12 @@
     <string name="git_commit_edit_text">Edit password for %1$s using android password store.</string>
     <string name="git_commit_remove_text">Remove %1$s from store.</string>
     <string name="git_commit_move_text">Rename %1$s to %2$s.</string>
-    <string name="git_commit_increment_text">Increment HOTP counter for %1$s.</string>
 
     <!-- PGPHandler -->
     <string name="provider_toast_text">No OpenPGP provider selected!</string>
     <string name="clipboard_password_toast_text">Password copied to clipboard, you have %d seconds to paste it somewhere.</string>
     <string name="clipboard_password_no_clear_toast_text">Password copied to clipboard</string>
     <string name="clipboard_username_toast_text">Username copied to clipboard</string>
-    <string name="clipboard_otp_toast_text">OTP code copied to clipboard</string>
     <string name="file_toast_text">Please provide a file name</string>
     <string name="path_toast_text">Please provide a file path</string>
     <string name="empty_toast_text">You cannot use an empty password or empty extra content</string>
@@ -121,18 +119,11 @@
     <string name="password">Password:</string>
     <string name="extra_content">Extra content:</string>
     <string name="username">Username:</string>
-    <string name="otp" translatable="false">OTP:</string>
     <string name="edit_password">Edit password</string>
     <string name="copy_password">Copy password</string>
     <string name="copy_username">Copy username</string>
-    <string name="copy_otp">Copy OTP code</string>
     <string name="share_as_plaintext">Share as plaintext</string>
     <string name="last_changed">Last changed %s</string>
-    <string name="dialog_update_title">Attention</string>
-    <string name="dialog_update_positive">Update entry</string>
-    <string name="dialog_update_negative">Leave unchanged</string>
-    <string name="dialog_update_body">The HOTP counter is about to be incremented. This change will be committed. If you press "Leave unchanged", the HOTP code will be shown, but the counter will not be changed.</string>
-    <string name="dialog_update_check">Remember my choice</string>
 
     <!-- Preferences -->
     <string name="pref_repository_title">Repository</string>
@@ -316,7 +307,6 @@
     <string name="jgit_error_push_dialog_text">Error occurred during the push operation:</string>
     <string name="clear_saved_passphrase_ssh">Clear saved passphrase for local SSH key</string>
     <string name="clear_saved_passphrase_https">Clear saved HTTPS password</string>
-    <string name="hotp_remember_clear_choice">Clear saved preference for HOTP incrementing</string>
     <string name="remember_the_passphrase">Remember key passphrase</string>
     <string name="hackish_tools">Hackish tools</string>
     <string name="abort_rebase">Abort rebase and push new branch</string>
@@ -324,7 +314,6 @@
     <string name="commit_hash">Commit hash</string>
     <string name="crypto_extra_edit_hint">username: something other extra content</string>
     <string name="get_last_changed_failed">Failed to get last changed date</string>
-    <string name="hotp_pending">Tap copy to calculate HOTP</string>
     <string name="openkeychain_ssh_api_connect_fail">Failed to connect to OpenKeychain SSH API service.</string>
     <string name="no_ssh_api_provider">No SSH API provider found. Is OpenKeychain installed?</string>
     <string name="ssh_api_pending_intent_failed">SSH API pending intent failed</string>
diff --git a/app/src/main/res/xml/preference.xml b/app/src/main/res/xml/preference.xml
index c823bc6a..9e56ce8f 100644
--- a/app/src/main/res/xml/preference.xml
+++ b/app/src/main/res/xml/preference.xml
@@ -54,9 +54,6 @@
             app:key="ssh_keygen"
             app:title="@string/pref_ssh_keygen_title" />
         <Preference app:key="clear_saved_pass" />
-        <Preference
-            app:key="hotp_remember_clear_choice"
-            app:title="@string/hotp_remember_clear_choice" />
         <Preference
             app:key="ssh_openkeystore_clear_keyid"
             app:title="@string/ssh_openkeystore_clear_keyid" />
-- 
cgit v1.2.3