Bakup warnings

Stephen Paul Weber created 11 months ago

Change summary

src/cheogram/res/values/strings.xml                                                   |  1 
src/main/java/eu/siacs/conversations/ui/fragment/settings/BackupSettingsFragment.java | 20 
src/main/res/xml/preferences_backup.xml                                               |  4 
3 files changed, 24 insertions(+), 1 deletion(-)

Detailed changes

src/cheogram/res/values/strings.xml

@@ -53,4 +53,5 @@
     <string name="pref_custom_tab_summary">Ask your browser to render as if integrated with this app ("custom tab")</string>
     <string name="pref_export">Export Data (experimental)</string>
     <string name="pref_export_summary">Export data useful for importing into another app. Not a full backup.</string>
+    <string name="pref_create_backup_warning">Full backups are meant for restore after a device loss, or for migrating to a new device. You cannot restore a backup to another app while still being connected with this one!</string>
 </resources>

src/main/java/eu/siacs/conversations/ui/fragment/settings/BackupSettingsFragment.java

@@ -11,6 +11,7 @@ import androidx.activity.result.ActivityResultLauncher;
 import androidx.activity.result.contract.ActivityResultContracts;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
+import androidx.appcompat.app.AlertDialog;
 import androidx.core.content.ContextCompat;
 import androidx.preference.ListPreference;
 import androidx.preference.Preference;
@@ -29,6 +30,7 @@ import com.google.common.primitives.Longs;
 
 import eu.siacs.conversations.Config;
 import eu.siacs.conversations.R;
+import eu.siacs.conversations.entities.Account;
 import eu.siacs.conversations.persistance.FileBackend;
 import eu.siacs.conversations.worker.ExportBackupWorker;
 
@@ -131,6 +133,23 @@ public class BackupSettingsFragment extends XmppPreferenceFragment {
     }
 
     private boolean onBackupPreferenceClicked(final Preference preference) {
+        new AlertDialog.Builder(requireActivity())
+            .setTitle("Disable accounts")
+            .setMessage("Do you want to disable your accounts before making a backup (recommended)?")
+            .setPositiveButton(R.string.yes, (dialog, whichButton) -> {
+                for (final var account : requireService().getAccounts()) {
+                    account.setOption(Account.OPTION_DISABLED, true);
+                    if (!requireService().updateAccount(account)) {
+                        Toast.makeText(requireActivity(), R.string.unable_to_update_account, Toast.LENGTH_SHORT).show();
+                    }
+                }
+                aboutToStartOneOffBackup();
+            })
+            .setNegativeButton(R.string.no, (dialog, whichButton) -> aboutToStartOneOffBackup()).show();
+        return true;
+    }
+
+    private void aboutToStartOneOffBackup() {
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.TIRAMISU) {
             if (ContextCompat.checkSelfPermission(
                             requireContext(), Manifest.permission.WRITE_EXTERNAL_STORAGE)
@@ -142,7 +161,6 @@ public class BackupSettingsFragment extends XmppPreferenceFragment {
         } else {
             startOneOffBackup();
         }
-        return true;
     }
 
     private void startOneOffBackup() {

src/main/res/xml/preferences_backup.xml

@@ -1,6 +1,10 @@
 <?xml version="1.0" encoding="utf-8"?>
 <PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android">
 
+    <Preference
+        android:key="backup_warning"
+        android:summary="@string/pref_create_backup_warning" />
+
     <ListPreference
         android:defaultValue="@integer/automatic_message_deletion"
         android:icon="@drawable/ic_calendar_month_24dp"