From 51d8c08ea02ef3e7cf374f2430a171d598822200 Mon Sep 17 00:00:00 2001
From: Nikolas Rimikis <rimikis.nikolas@gmail.com>
Date: Tue, 29 Aug 2023 17:13:35 +0200
Subject: [PATCH] fix(neon): SettingsExportHelper

Signed-off-by: Nikolas Rimikis <rimikis.nikolas@gmail.com>
---
 .../neon/neon/lib/src/pages/settings.dart     |  41 ++---
 .../lib/src/settings/models/exportable.dart   |  40 +++++
 .../models/nextcloud_app_options.dart         |  23 ---
 .../neon/lib/src/settings/models/option.dart  |  10 ++
 .../settings/models/options_collection.dart   |  21 ++-
 .../neon/lib/src/settings/models/storage.dart |   2 +
 .../utils/settings_export_helper.dart         | 147 ++++++++++++++++++
 .../lib/src/utils/settings_export_helper.dart |  98 ------------
 8 files changed, 233 insertions(+), 149 deletions(-)
 create mode 100644 packages/neon/neon/lib/src/settings/models/exportable.dart
 delete mode 100644 packages/neon/neon/lib/src/settings/models/nextcloud_app_options.dart
 create mode 100644 packages/neon/neon/lib/src/settings/utils/settings_export_helper.dart
 delete mode 100644 packages/neon/neon/lib/src/utils/settings_export_helper.dart

diff --git a/packages/neon/neon/lib/src/pages/settings.dart b/packages/neon/neon/lib/src/pages/settings.dart
index 92a2fa58..a1e2bd70 100644
--- a/packages/neon/neon/lib/src/pages/settings.dart
+++ b/packages/neon/neon/lib/src/pages/settings.dart
@@ -1,6 +1,3 @@
-import 'dart:convert';
-import 'dart:typed_data';
-
 import 'package:file_picker/file_picker.dart';
 import 'package:flutter/material.dart';
 import 'package:material_design_icons_flutter/material_design_icons_flutter.dart';
@@ -10,6 +7,7 @@ import 'package:neon/src/models/account.dart';
 import 'package:neon/src/models/app_implementation.dart';
 import 'package:neon/src/platform/platform.dart';
 import 'package:neon/src/router.dart';
+import 'package:neon/src/settings/utils/settings_export_helper.dart';
 import 'package:neon/src/settings/widgets/account_settings_tile.dart';
 import 'package:neon/src/settings/widgets/checkbox_settings_tile.dart';
 import 'package:neon/src/settings/widgets/custom_settings_tile.dart';
@@ -23,7 +21,6 @@ import 'package:neon/src/theme/dialog.dart';
 import 'package:neon/src/utils/confirmation_dialog.dart';
 import 'package:neon/src/utils/global_options.dart';
 import 'package:neon/src/utils/save_file.dart';
-import 'package:neon/src/utils/settings_export_helper.dart';
 import 'package:neon/src/widgets/exception.dart';
 import 'package:package_info_plus/package_info_plus.dart';
 import 'package:provider/provider.dart';
@@ -250,16 +247,10 @@ class _SettingsPageState extends State<SettingsPage> {
                     final settingsExportHelper = _buildSettingsExportHelper(context);
 
                     try {
-                      final fileName =
-                          'nextcloud-neon-settings-${DateTime.now().millisecondsSinceEpoch ~/ 1000}.json.base64';
-                      final data = base64.encode(
-                        utf8.encode(
-                          json.encode(
-                            settingsExportHelper.toJsonExport(),
-                          ),
-                        ),
-                      );
-                      await saveFileWithPickDialog(fileName, utf8.encode(data) as Uint8List);
+                      final fileName = 'nextcloud-neon-settings-${DateTime.now().millisecondsSinceEpoch ~/ 1000}.json';
+
+                      final data = settingsExportHelper.exportToFile();
+                      await saveFileWithPickDialog(fileName, data);
                     } catch (e, s) {
                       debugPrint(e.toString());
                       debugPrint(s.toString());
@@ -280,14 +271,14 @@ class _SettingsPageState extends State<SettingsPage> {
 
                     try {
                       final result = await FilePicker.platform.pickFiles(
-                        withData: true,
+                        withReadStream: true,
                       );
 
                       if (result == null) {
                         return;
                       }
 
-                      if (!result.files.single.path!.endsWith('.json.base64')) {
+                      if (!result.files.single.path!.endsWith('.json')) {
                         if (mounted) {
                           NeonException.showSnackbar(
                             context,
@@ -297,9 +288,7 @@ class _SettingsPageState extends State<SettingsPage> {
                         return;
                       }
 
-                      final data = json.decode(utf8.decode(base64.decode(utf8.decode(result.files.single.bytes!))));
-
-                      await settingsExportHelper.applyFromJson(data as Map<String, dynamic>);
+                      await settingsExportHelper.applyFromFile(result.files.single.readStream);
                     } catch (e, s) {
                       debugPrint(e.toString());
                       debugPrint(s.toString());
@@ -329,16 +318,16 @@ class _SettingsPageState extends State<SettingsPage> {
   }
 
   SettingsExportHelper _buildSettingsExportHelper(final BuildContext context) {
-    final globalOptions = Provider.of<GlobalOptions>(context);
+    final globalOptions = Provider.of<GlobalOptions>(context, listen: false);
     final accountsBloc = Provider.of<AccountsBloc>(context, listen: false);
-    final appImplementations = Provider.of<Iterable<AppImplementation>>(context);
+    final appImplementations = Provider.of<Iterable<AppImplementation>>(context, listen: false);
 
     return SettingsExportHelper(
-      globalOptions: globalOptions,
-      appImplementations: appImplementations,
-      accountSpecificOptions: accountsBloc.accounts.value.asMap().map(
-            (final _, final account) => MapEntry(account, accountsBloc.getOptionsFor(account).options),
-          ),
+      exportables: {
+        globalOptions,
+        AccountsBlocExporter(accountsBloc),
+        AppImplementationsExporter(appImplementations),
+      },
     );
   }
 }
diff --git a/packages/neon/neon/lib/src/settings/models/exportable.dart b/packages/neon/neon/lib/src/settings/models/exportable.dart
new file mode 100644
index 00000000..d2aa8152
--- /dev/null
+++ b/packages/neon/neon/lib/src/settings/models/exportable.dart
@@ -0,0 +1,40 @@
+import 'dart:async';
+
+import 'package:collection/collection.dart';
+import 'package:neon/src/settings/models/option.dart';
+
+/// Exportable data interface.
+abstract interface class Exportable {
+  /// Exports into a json map entry.
+  MapEntry<String, Object?> export();
+
+  /// Imports [data] from an export.
+  FutureOr<void> import(final Map<String, Object?> data);
+}
+
+/// Serialization helpers for a collection of [Option]s.
+extension SerializeOptions on Iterable<Option<dynamic>> {
+  /// Serializes into an [Iterable<JsonEntry>].
+  ///
+  /// Use [Map.fromEntries] to get a json Map.
+  Iterable<MapEntry<String, Object?>> serialize() sync* {
+    for (final option in this) {
+      if (option.enabled) {
+        yield MapEntry(option.key.value, option.serialize());
+      }
+    }
+  }
+
+  /// Deserializes [data] and updates the [Option]s.
+  void deserialize(final Map<String, Object?> data) {
+    for (final entry in data.entries) {
+      final option = firstWhereOrNull((final option) => option.key.value == entry.key);
+
+      if (entry.value != null) {
+        option?.load(entry.value);
+      } else {
+        option?.reset();
+      }
+    }
+  }
+}
diff --git a/packages/neon/neon/lib/src/settings/models/nextcloud_app_options.dart b/packages/neon/neon/lib/src/settings/models/nextcloud_app_options.dart
deleted file mode 100644
index 53b32f10..00000000
--- a/packages/neon/neon/lib/src/settings/models/nextcloud_app_options.dart
+++ /dev/null
@@ -1,23 +0,0 @@
-import 'package:neon/src/settings/models/option.dart';
-import 'package:neon/src/settings/models/options_category.dart';
-import 'package:neon/src/settings/models/storage.dart';
-
-abstract class NextcloudAppOptions {
-  NextcloudAppOptions(this.storage);
-
-  final AppStorage storage;
-  late final List<OptionsCategory> categories;
-  late final List<Option> options;
-
-  void reset() {
-    for (final option in options) {
-      option.reset();
-    }
-  }
-
-  void dispose() {
-    for (final option in options) {
-      option.dispose();
-    }
-  }
-}
diff --git a/packages/neon/neon/lib/src/settings/models/option.dart b/packages/neon/neon/lib/src/settings/models/option.dart
index ac954b4d..d6433c25 100644
--- a/packages/neon/neon/lib/src/settings/models/option.dart
+++ b/packages/neon/neon/lib/src/settings/models/option.dart
@@ -79,10 +79,20 @@ abstract class Option<T> extends ChangeNotifier implements ValueListenable<T> {
   }
 
   /// Resets the option to its [default] value.
+  @mustBeOverridden
   void reset() {
     value = defaultValue;
   }
 
+  /// Loads [data] into [value] by calling [deserialize] on it.
+  void load(final Object? data) {
+    final value = deserialize(data);
+
+    if (value != null) {
+      this.value = value;
+    }
+  }
+
   /// Deserializes the data.
   T? deserialize(final Object? data);
 
diff --git a/packages/neon/neon/lib/src/settings/models/options_collection.dart b/packages/neon/neon/lib/src/settings/models/options_collection.dart
index a981ad31..d81acc2c 100644
--- a/packages/neon/neon/lib/src/settings/models/options_collection.dart
+++ b/packages/neon/neon/lib/src/settings/models/options_collection.dart
@@ -1,15 +1,16 @@
 import 'package:meta/meta.dart';
+import 'package:neon/src/settings/models/exportable.dart';
 import 'package:neon/src/settings/models/option.dart';
 import 'package:neon/src/settings/models/options_category.dart';
 import 'package:neon/src/settings/models/storage.dart';
 
 /// Collection of [Option]s.
-abstract class OptionsCollection {
+abstract class OptionsCollection implements Exportable {
   OptionsCollection(this.storage);
 
   /// Storage backend to use.
   @protected
-  final SettingsStorage storage;
+  final AppStorage storage;
 
   /// Collection of options.
   @protected
@@ -34,6 +35,22 @@ abstract class OptionsCollection {
       option.dispose();
     }
   }
+
+  @override
+  MapEntry<String, Object?> export() {
+    final data = Map.fromEntries(options.serialize());
+
+    return MapEntry(storage.id, data);
+  }
+
+  @override
+  void import(final Map<String, Object?> data) {
+    final values = data[storage.id] as Map<String, Object?>?;
+
+    if (values != null) {
+      options.deserialize(values);
+    }
+  }
 }
 
 /// OpptionsCollection for a neon app.
diff --git a/packages/neon/neon/lib/src/settings/models/storage.dart b/packages/neon/neon/lib/src/settings/models/storage.dart
index ebdd58ee..df865c61 100644
--- a/packages/neon/neon/lib/src/settings/models/storage.dart
+++ b/packages/neon/neon/lib/src/settings/models/storage.dart
@@ -108,6 +108,8 @@ final class AppStorage implements SettingsStorage {
 
   final String? suffix;
 
+  String get id => suffix ?? key.value;
+
   @visibleForTesting
   String formatKey(final String key) {
     if (suffix != null) {
diff --git a/packages/neon/neon/lib/src/settings/utils/settings_export_helper.dart b/packages/neon/neon/lib/src/settings/utils/settings_export_helper.dart
new file mode 100644
index 00000000..a5e34a5c
--- /dev/null
+++ b/packages/neon/neon/lib/src/settings/utils/settings_export_helper.dart
@@ -0,0 +1,147 @@
+import 'dart:async';
+import 'dart:convert';
+import 'dart:typed_data';
+
+import 'package:meta/meta.dart';
+import 'package:neon/src/blocs/accounts.dart';
+import 'package:neon/src/models/account.dart' show Account, AccountFind;
+import 'package:neon/src/models/app_implementation.dart';
+import 'package:neon/src/settings/models/exportable.dart';
+import 'package:neon/src/settings/models/option.dart';
+import 'package:neon/src/settings/models/storage.dart';
+
+/// Helper class to export all [Option]s.
+///
+/// Json based operations:
+/// * [exportToJson]
+/// * [applyFromJson]
+///
+/// [Uint8List] based operations:
+/// * [exportToFile]
+/// * [applyFromFile]
+@internal
+@immutable
+class SettingsExportHelper {
+  const SettingsExportHelper({
+    required this.exportables,
+  });
+
+  /// Collections of elements to export.
+  final Set<Exportable> exportables;
+
+  /// Imports [file] and applies the stored [Option]s.
+  ///
+  /// See:
+  /// * [applyFromJson] to import a json map.
+  Future<void> applyFromFile(final Stream<List<int>>? file) async {
+    final transformer = const Utf8Decoder().fuse(const JsonDecoder());
+
+    final data = await file?.transform(transformer).single;
+
+    if (data == null) {
+      return;
+    }
+
+    await applyFromJson(data as Map<String, Object?>);
+  }
+
+  /// Imports the [json] data and applies the stored [Option]s.
+  ///
+  /// See:
+  /// * [applyFromFile] to import data from a [Stream<Uint8List>].
+  Future<void> applyFromJson(final Map<String, Object?> json) async {
+    for (final exportable in exportables) {
+      await exportable.import(json);
+    }
+  }
+
+  /// Exports the stored [Option]s into a [Uint8List].
+  ///
+  /// See:
+  /// * [exportToJson] to export to a json map.
+  Uint8List exportToFile() {
+    final transformer = JsonUtf8Encoder();
+
+    final json = exportToJson();
+    return transformer.convert(json) as Uint8List;
+  }
+
+  /// Exports the stored [Option]s into a json map.
+  ///
+  /// See:
+  /// * [exportToFile] to export data to a [Uint8List].
+  Map<String, Object?> exportToJson() => Map.fromEntries(exportables.map((final e) => e.export()));
+}
+
+/// Helper class to export [AppImplementation]s implementing the [Exportable] interface.
+@immutable
+class AppImplementationsExporter implements Exportable {
+  const AppImplementationsExporter(this.appImplementations);
+
+  /// List of apps to export.
+  final Iterable<AppImplementation> appImplementations;
+
+  /// Key the exported value will be stored at.
+  static final _key = StorageKeys.apps.value;
+
+  @override
+  MapEntry<String, Object?> export() => MapEntry(
+        _key,
+        Map.fromEntries(appImplementations.map((final app) => app.options.export())),
+      );
+
+  @override
+  void import(final Map<String, Object?> data) {
+    final values = data[_key] as Map<String, Object?>?;
+
+    if (values == null) {
+      return;
+    }
+
+    for (final element in values.entries) {
+      final app = appImplementations.tryFind(element.key);
+
+      if (app != null) {
+        app.options.import(values);
+      }
+    }
+  }
+}
+
+/// Helper class to export [Account]s implementing the [Exportable] interface.
+@immutable
+class AccountsBlocExporter implements Exportable {
+  const AccountsBlocExporter(this.accountsBloc);
+
+  /// AccountsBloc containing the accounts to export.
+  final AccountsBloc accountsBloc;
+
+  /// Key the exported value will be stored at.
+  static final _key = StorageKeys.accounts.value;
+
+  @override
+  MapEntry<String, Object?> export() => MapEntry(_key, Map.fromEntries(_serialize()));
+
+  Iterable<MapEntry<String, Object?>> _serialize() sync* {
+    for (final account in accountsBloc.accounts.value) {
+      yield accountsBloc.getOptionsFor(account).export();
+    }
+  }
+
+  @override
+  void import(final Map<String, Object?> data) {
+    final values = data[_key] as Map<String, Object?>?;
+
+    if (values == null) {
+      return;
+    }
+
+    for (final element in values.entries) {
+      final account = accountsBloc.accounts.value.tryFind(element.key);
+
+      if (account != null) {
+        accountsBloc.getOptionsFor(account).import(values);
+      }
+    }
+  }
+}
diff --git a/packages/neon/neon/lib/src/utils/settings_export_helper.dart b/packages/neon/neon/lib/src/utils/settings_export_helper.dart
deleted file mode 100644
index 77db0670..00000000
--- a/packages/neon/neon/lib/src/utils/settings_export_helper.dart
+++ /dev/null
@@ -1,98 +0,0 @@
-import 'package:meta/meta.dart';
-import 'package:neon/src/models/account.dart';
-import 'package:neon/src/models/app_implementation.dart';
-import 'package:neon/src/settings/models/option.dart';
-import 'package:neon/src/utils/global_options.dart';
-
-@internal
-class SettingsExportHelper {
-  SettingsExportHelper({
-    required this.globalOptions,
-    required this.appImplementations,
-    required this.accountSpecificOptions,
-  });
-
-  final GlobalOptions globalOptions;
-  final Iterable<AppImplementation> appImplementations;
-  final Map<Account, List<Option>> accountSpecificOptions;
-
-  Future applyFromJson(final Map<String, dynamic> data) async {
-    final globalOptionsData = data['global'] as Map<String, dynamic>;
-    await _applyOptionsMapToOptions(
-      globalOptions.options,
-      globalOptionsData,
-    );
-
-    final appImplementationsData = data['apps'] as Map<String, dynamic>;
-    for (final appId in appImplementationsData.keys) {
-      final app = appImplementations.tryFind(appId);
-      if (app == null) {
-        return;
-      }
-      final appImplementationData = appImplementationsData[appId]! as Map<String, dynamic>;
-      await _applyOptionsMapToOptions(
-        app.options.options,
-        appImplementationData,
-      );
-    }
-
-    final accountsData = data['accounts'] as Map<String, dynamic>;
-    for (final accountId in accountsData.keys) {
-      final account = accountSpecificOptions.keys.tryFind(accountId);
-      if (account == null) {
-        return;
-      }
-      final accountData = accountsData[accountId]! as Map<String, dynamic>;
-      await _applyOptionsMapToOptions(
-        accountSpecificOptions[account]!,
-        accountData,
-      );
-    }
-  }
-
-  Future _applyOptionsMapToOptions(final Iterable<Option> options, final Map<String, dynamic> data) async {
-    for (final optionKey in data.keys) {
-      for (final option in options) {
-        if (option.key.value == optionKey) {
-          final Object? value = data[optionKey];
-
-          if (value != null) {
-            option.value = await option.deserialize(value);
-          }
-        }
-      }
-    }
-  }
-
-  Map<String, dynamic> toJsonExport() => {
-        'global': {
-          for (final option in globalOptions.options) ...{
-            if (option.enabled) ...{
-              option.key: option.serialize(),
-            },
-          },
-        },
-        'apps': {
-          for (final appImplementation in appImplementations) ...{
-            appImplementation.id: {
-              for (final option in appImplementation.options.options) ...{
-                if (option.enabled) ...{
-                  option.key: option.serialize(),
-                },
-              },
-            },
-          },
-        },
-        'accounts': {
-          for (final account in accountSpecificOptions.keys) ...{
-            account.id: {
-              for (final option in accountSpecificOptions[account]!) ...{
-                if (option.enabled) ...{
-                  option.key: option.serialize(),
-                },
-              },
-            },
-          },
-        },
-      };
-}