feat: Add restart required note when importing data

Myzel394 · Jul 19, 2023 · 8a854e8 · 8a854e8
1 parent c2bf9d7
commit 8a854e8
Show file tree

Hide file tree

Showing 3 changed files with 70 additions and 4 deletions.
diff --git a/lib/l10n/app_en.arb b/lib/l10n/app_en.arb
@@ -458,6 +458,11 @@
   "settingsScreen_import_file": "Import file",
   "settingsScreen_import_pickerTitle": "Select the export.locus.json file to import your data",
   "settingsScreen_import_transfer": "Transfer from old device",
+  "settingsScreen_import_confirmation_title": "Are you sure you want to import your data?",
+  "settingsScreen_import_confirmation_description": "Do you want to import your data? THIS WILL OVERWRITE YOUR CURRENT DATA (SHARES, SETTINGS, ETC.) WITH THE DATA FROM THE IMPORT!",
+  "settingsScreen_import_confirmation_confirm": "Import",
+  "settingsScreen_import_restart_title": "Restart Locus",
+  "settingsScreen_import_restart_description": "Locus needs to be restart to finish the import. Please restart the app now.",
   "settingsScreen_version": "Version {version} ({flavor, select, gms{GMS} floss{FLOSS} other{Unknown}})",
   "@settingsScreen_version": {
     "placeholders": {

diff --git a/lib/screens/SettingsScreen.dart b/lib/screens/SettingsScreen.dart
@@ -559,7 +559,35 @@ class _SettingsScreenState extends State<SettingsScreen> {
                                   ]);
 
                                   if (context.mounted) {
-                                    Navigator.pop(context, true);
+                                    final shouldClose =
+                                        await showPlatformDialog(
+                                      context: context,
+                                      barrierDismissible: !Platform.isAndroid,
+                                      builder: (context) => PlatformAlertDialog(
+                                        title: Text(l10n
+                                            .settingsScreen_import_restart_title),
+                                        content: Text(l10n
+                                            .settingsScreen_import_restart_description),
+                                        actions: [
+                                          PlatformDialogAction(
+                                            child: Text(l10n.closeApp),
+                                            onPressed: () => Navigator.pop(
+                                                context, Platform.isAndroid),
+                                          ),
+                                        ],
+                                      ),
+                                    );
+
+                                    if (!mounted) {
+                                      return;
+                                    }
+
+                                    if (shouldClose != true) {
+                                      Navigator.pop(context);
+                                      return;
+                                    }
+
+                                    exit(0);
                                   }
                                 },
                               ),

diff --git a/lib/screens/settings_screen_widgets/ImportSheet.dart b/lib/screens/settings_screen_widgets/ImportSheet.dart
@@ -13,6 +13,8 @@ import 'package:locus/services/settings_service.dart';
 import 'package:locus/services/task_service.dart';
 import 'package:locus/services/view_service.dart';
 import 'package:locus/utils/PageRoute.dart';
+import 'package:locus/utils/platform.dart';
+import 'package:locus/widgets/PlatformFlavorWidget.dart';
 
 import '../../constants/spacing.dart';
 import '../../utils/theme.dart';
@@ -38,9 +40,40 @@ class ImportSheet extends StatefulWidget {
 class _ImportSheetState extends State<ImportSheet> {
   String errorMessage = "";
 
-  void parseRawData(final String rawData) async {
+  void importRawData(final String rawData) async {
     final l10n = AppLocalizations.of(context);
 
+    final shouldImport = await showPlatformDialog(
+      context: context,
+      builder: (context) => PlatformAlertDialog(
+        material: (context, __) => MaterialAlertDialogData(
+          icon: PlatformFlavorWidget(
+            material: (context, _) => const Icon(Icons.warning_rounded),
+            cupertino: (context, _) =>
+                const Icon(CupertinoIcons.exclamationmark_triangle_fill),
+          ),
+        ),
+        title: Text(l10n.settingsScreen_import_confirmation_title),
+        content: Text(l10n.settingsScreen_import_confirmation_description),
+        actions: createCancellableDialogActions(
+          context,
+          [
+            PlatformDialogAction(
+              material: (context, _) => MaterialDialogActionData(
+                icon: const Icon(Icons.download_rounded),
+              ),
+              child: Text(l10n.settingsScreen_import_confirmation_confirm),
+              onPressed: () => Navigator.pop(context, true),
+            ),
+          ],
+        ),
+      ),
+    );
+
+    if (shouldImport != true || !mounted) {
+      return;
+    }
+
     try {
       final data = jsonDecode(rawData);
       final tasks = TaskService(
@@ -108,7 +141,7 @@ class _ImportSheetState extends State<ImportSheet> {
                     final content = String.fromCharCodes(
                         List<int>.from(result.files[0].bytes!));
 
-                    parseRawData(content);
+                    importRawData(content);
                   } catch (_) {
                     setState(() {
                       errorMessage = l10n.unknownError;
@@ -131,7 +164,7 @@ class _ImportSheetState extends State<ImportSheet> {
                           NativePageRoute(
                             context: context,
                             builder: (context) => TransferReceiverScreen(
-                                onContentReceived: parseRawData),
+                                onContentReceived: importRawData),
                           ),
                         );
                       },