Merge pull request 'Data import and export' (#6) from development into main

Reviewed-on: #6

.gitea/workflows/android_build.yml

@@ -4,6 +4,9 @@ on:
   push:
     branches:
       - main
+  pull_request:
+    branches:
+      - main
   workflow_dispatch:
 
 

lib/pages/collection_page.dart

@@ -9,7 +9,6 @@ import '../service/notifying.dart';
 import '../service/shared_link_provider.dart';
 import '../service/storage.dart';
 import '../service/url_launcher.dart';
-import '../widgets/collection_page_widgets/list_item_actions_widget.dart';
 import '../widgets/create_bookmark_dialog.dart';
 
 class CollectionPage extends StatefulWidget {
@@ -23,7 +22,6 @@ class CollectionPage extends StatefulWidget {
 
 class _CollectionPageState extends State<CollectionPage> {
   MapsLinkMetadata? selectedMapsLink;
-  int selectedBookmarkId = -1;
 
   @override
   void initState() {
@@ -54,46 +52,29 @@ class _CollectionPageState extends State<CollectionPage> {
         ).whenComplete(() => setState(() {}));
       },
     ),
-  ).whenComplete(deselectBookmark);
+  );
 
   void onBookmarkSaved(Bookmark bookmark) {
     Storage.addOrUpdateBookmark(bookmark);
     setState(() {});
-    Provider.of<SharedLinkProvider>(
-      context,
-      listen: false,
-    ).removeCurrentMapsLink();
+    context.read<SharedLinkProvider>().removeCurrentMapsLink();
   }
 
   Widget bookmarksListItemBuilder(BuildContext context, Bookmark bookmark) {
-    final selected = selectedBookmarkId == bookmark.id;
     return ListTile(
       title: Text(bookmark.name),
-      selected: selected,
-      onTap: () {
-        if (selected) {
-          onCancelSelectionPressed();
-          setState(() {});
-        } else if (selectedBookmarkId != -1 && !selected) {
-          selectedBookmarkId = bookmark.id;
-          setState(() {});
-        } else {
-          launchUrlFromString(bookmark.link).then((errorCode) {
+      onTap: () => launchUrlFromString(bookmark.link).then((errorCode) {
         if (context.mounted) {
           return Notifying.showUrlErrorSnackbar(context, errorCode);
         }
-          });
-        }
-      },
-      onLongPress: () => setState(() {
-        selectedBookmarkId = bookmark.id;
       }),
+      onLongPress: () => editBookmark(bookmark),
     );
   }
 
   @override
   Widget build(BuildContext context) {
-    SharedLinkProvider provider = Provider.of<SharedLinkProvider>(context);
+    SharedLinkProvider provider = context.watch<SharedLinkProvider>();
     selectedMapsLink = provider.currentMapsLinkMetadata;
 
     if (BookmarksProvider.selectedCollectionId == null) {
@@ -106,18 +87,11 @@ class _CollectionPageState extends State<CollectionPage> {
       (c) => c.id == BookmarksProvider.selectedCollectionId,
     );
 
-    return PopScope(
-      canPop: selectedBookmarkId == -1,
-      onPopInvokedWithResult: (didPop, result) {
-        if (didPop == false) deselectBookmark();
-      },
-      child: Scaffold(
+    return Scaffold(
       appBar: AppBar(
         title: selectedMapsLink != null
             ? Text(
-                  AppLocalizations.of(
-                    context,
-                  )!.addToCollection(collection.name),
+                AppLocalizations.of(context)!.addToCollection(collection.name),
               )
             : Text(collection.name),
         actions: [
@@ -128,17 +102,6 @@ class _CollectionPageState extends State<CollectionPage> {
             ),
         ],
       ),
-        bottomNavigationBar: selectedBookmarkId > 0
-            ? ListItemActionsWidget(
-                onDeletePressed: onDeleteBookmarkPressed,
-                onCancelPressed: onCancelSelectionPressed,
-                onEditPressed: () => editBookmark(
-                  bookmarks.firstWhere(
-                    (element) => element.id == selectedBookmarkId,
-                  ),
-                ),
-              )
-            : null,
       body: Center(
         child: SizedBox(
           width: MediaQuery.of(context).size.width * 0.9,
@@ -154,21 +117,6 @@ class _CollectionPageState extends State<CollectionPage> {
         onPressed: onAddButtonPressed,
         child: Icon(selectedMapsLink != null ? Icons.save : Icons.add),
       ),
-      ),
     );
   }
-
-  void onCancelSelectionPressed() => deselectBookmark();
-
-  void onDeleteBookmarkPressed() {
-    Storage.deleteBookmarkById(
-      selectedBookmarkId,
-    ).whenComplete(() => setState(() {}));
-    deselectBookmark();
-  }
-
-  void deselectBookmark() {
-    selectedBookmarkId = -1;
-    setState(() {});
-  }
 }

lib/pages/collections_list_page.dart

@@ -87,8 +87,7 @@ class _CollectionsListPageState extends State<CollectionsListPage> {
     final collections = Storage.loadCollections();
     bookmarkCountMap = Storage.loadPerCollectionBookmarkCount();
     addingNewBookmark =
-        Provider.of<SharedLinkProvider>(context).currentMapsLinkMetadata !=
-        null;
+        context.watch<SharedLinkProvider>().currentMapsLinkMetadata != null;
     return Scaffold(
       appBar: AppBar(
         title: addingNewBookmark
@@ -97,10 +96,8 @@ class _CollectionsListPageState extends State<CollectionsListPage> {
         actions: [
           if (addingNewBookmark)
             TextButton(
-              onPressed: () => Provider.of<SharedLinkProvider>(
-                context,
-                listen: false,
-              ).removeCurrentMapsLink(),
+              onPressed: () =>
+                  context.read<SharedLinkProvider>().removeCurrentMapsLink(),
               child: Text(AppLocalizations.of(context)!.cancel),
             )
           else

lib/pages/settings_page.dart

@@ -17,90 +17,55 @@ class SettingsPage extends StatefulWidget {
 }
 
 class _SettingsPageState extends State<SettingsPage> {
-  bool storagePermissionIsGranted = false;
-  final tileSpacing = 16.0;
-
-  @override
-  void initState() {
-    PermissionService.storagePermissionStatus.then((value) {
-      storagePermissionIsGranted = value.isGranted;
-      if (context.mounted) setState(() {});
-    });
-    super.initState();
-  }
-
   @override
   Widget build(BuildContext context) {
-    final titlePadding = Theme.of(context).listTileTheme.contentPadding!;
-    checkStoragePermission();
     return Scaffold(
       appBar: AppBar(title: Text(AppLocalizations.of(context)!.settings)),
-      body: Center(
-        child: SizedBox(
+      body: SizedBox(
         width: MediaQuery.of(context).size.width * 0.9,
         child: Column(
-            crossAxisAlignment: CrossAxisAlignment.start,
+          crossAxisAlignment: CrossAxisAlignment.center,
           children: [
-              Padding(
-                padding: titlePadding,
-                child: Text(
+            Text(
               AppLocalizations.of(context)!.appData,
               style: Theme.of(context).textTheme.titleLarge,
             ),
+            ElevatedButton(
+              onPressed: () => onActivateJsonImportPressed(),
+              child: Text(AppLocalizations.of(context)!.import),
             ),
-              SizedBox(height: tileSpacing),
-              ListTile(
-                title: Text('Grant storage permisson'),
-                subtitle: Text(
-                  'For app-data settings to work, you need to grant the app permissions to manage internal storage.',
-                ),
-                onTap: () => PermissionService.requestStoragePermission
-                    .whenComplete(() => checkStoragePermission()),
-                trailing: Icon(Icons.arrow_forward_ios_rounded),
-                enabled: !storagePermissionIsGranted,
-              ),
-              SizedBox(height: tileSpacing),
-              ListTile(
-                title: Text(AppLocalizations.of(context)!.import),
-                subtitle: Text('Import app-data from a json file.'),
-                onTap: () => onActivateJsonImportPressed(),
-                trailing: Icon(Icons.arrow_forward_ios_rounded),
-                enabled: storagePermissionIsGranted,
-              ),
-              SizedBox(height: tileSpacing),
-              ListTile(
-                title: Text(AppLocalizations.of(context)!.export),
-                subtitle: Text(
-                  'Export app-data to a json file in the selected directory.',
-                ),
-                onTap: () => onActivateJsonExportPressed(),
-                trailing: Icon(Icons.arrow_forward_ios_rounded),
-                enabled: storagePermissionIsGranted,
+            ElevatedButton(
+              onPressed: () => onActivateJsonExportPressed(),
+              child: Text(AppLocalizations.of(context)!.export),
             ),
           ],
         ),
       ),
-      ),
     );
   }
 
   void onActivateJsonExportPressed() async {
-    if (!await PermissionService.storagePermissionStatus.isGranted) return;
+    if (!await checkStoragePermission) return;
+
     Storage.exportToJsonFile().then(showExportInfo);
   }
 
   void onActivateJsonImportPressed() async {
-    if (!await PermissionService.storagePermissionStatus.isGranted) return;
+    if (!await checkStoragePermission) return;
     Storage.importFromJsonFile().then(showImportInfo);
   }
 
-  Future<void> checkStoragePermission() async {
-    PermissionService.storagePermissionStatus.then((value) {
-      storagePermissionIsGranted = value.isGranted;
-      if (context.mounted && value.isGranted != storagePermissionIsGranted) {
-        setState(() {});
+  Future<bool> get checkStoragePermission async {
+    if (!(await PermissionService.requestStoragePermission).isGranted) {
+      if (mounted) {
+        Notifying.showErrorSnackbar(
+          context,
+          AppLocalizations.of(context)!.errorStoragePermisson,
+        );
+        return false;
       }
-    });
+    }
+    return true;
   }
 
   void showExportInfo(bool success) => Notifying.showSnackbar(
@@ -108,7 +73,7 @@ class _SettingsPageState extends State<SettingsPage> {
     text: success
         ? AppLocalizations.of(context)!.exportSuccess
         : AppLocalizations.of(context)!.exportFailed,
-    isError: !success,
+    isError: success,
   );
 
   void showImportInfo(bool success) => Notifying.showSnackbar(
@@ -116,6 +81,6 @@ class _SettingsPageState extends State<SettingsPage> {
     text: success
         ? AppLocalizations.of(context)!.importSuccess
         : AppLocalizations.of(context)!.importFailed,
-    isError: !success,
+    isError: success,
   );
 }

lib/service/json_file_service.dart

@@ -32,7 +32,7 @@ class JsonFileService {
     } catch (e) {
       return false;
     }
-    return true;
+    return false;
   }
 
   static Future<({List<Collection> collections, List<Bookmark> bookmarks})>

lib/service/notifying.dart

@@ -12,7 +12,7 @@ class Notifying {
     ScaffoldMessenger.of(context).hideCurrentSnackBar();
     ScaffoldMessenger.of(context).showSnackBar(
       SnackBar(
-        backgroundColor: isError ? Theme.of(context).colorScheme.error : null,
+        backgroundColor: Theme.of(context).colorScheme.error,
         content: SizedBox(
           height: 30,
           child: Row(
@@ -29,7 +29,7 @@ class Notifying {
                     ScaffoldMessenger.of(context).hideCurrentSnackBar(),
                 icon: Icon(
                   Icons.close_rounded,
-                  color: isError ? Theme.of(context).colorScheme.onError : null,
+                  color: Theme.of(context).colorScheme.onError,
                 ),
               ),
             ],

lib/theme.dart

@@ -22,9 +22,5 @@ ThemeData _baseTheme(ColorScheme scheme) =>
         shape: RoundedRectangleBorder(
           borderRadius: BorderRadiusGeometry.circular(12),
         ),
-        textColor: scheme.onPrimaryContainer,
-        selectedTileColor: scheme.primaryContainer,
-        selectedColor: scheme.onPrimaryContainer,
-        contentPadding: EdgeInsetsDirectional.only(start: 16.0, end: 24.0),
       ),
     );

lib/widgets/collection_page_widgets/list_item_actions_widget.dart

@@ -1,53 +0,0 @@
-import 'package:flutter/material.dart';
-
-class ListItemActionsWidget extends StatelessWidget {
-  final VoidCallback _onDeletePressed;
-  final VoidCallback _onCancelPressed;
-  final VoidCallback _onEditPressed;
-
-  const ListItemActionsWidget({
-    super.key,
-    required void Function() onDeletePressed,
-    required void Function() onCancelPressed,
-    required void Function() onEditPressed,
-  }) : _onEditPressed = onEditPressed,
-       _onCancelPressed = onCancelPressed,
-       _onDeletePressed = onDeletePressed;
-
-  @override
-  Widget build(BuildContext context) {
-    return Container(
-      padding: EdgeInsetsGeometry.fromLTRB(
-        10,
-        10,
-        10,
-        MediaQuery.of(context).viewPadding.bottom,
-      ),
-      color: Theme.of(context).colorScheme.surfaceContainer,
-      child: Row(
-        mainAxisAlignment: MainAxisAlignment.spaceBetween,
-        crossAxisAlignment: CrossAxisAlignment.start,
-        children: [
-          IconButton(
-            onPressed: _onCancelPressed,
-            icon: const Icon(Icons.close_rounded),
-          ),
-          Row(
-            mainAxisAlignment: MainAxisAlignment.end,
-            mainAxisSize: MainAxisSize.min,
-            children: [
-              IconButton(
-                onPressed: _onDeletePressed,
-                icon: const Icon(Icons.delete_forever_rounded),
-              ),
-              IconButton(
-                onPressed: _onEditPressed,
-                icon: const Icon(Icons.edit_rounded),
-              ),
-            ],
-          ),
-        ],
-      ),
-    );
-  }
-}