Migrate to file_selector, related to #558

Author: CodeDoctorDE

app/lib/actions/export.dart

@@ -1,4 +1,4 @@
-import 'package:butterfly/api/save_data.dart';
+import 'package:butterfly/api/save.dart';
 import 'package:butterfly/bloc/document_bloc.dart';
 import 'package:butterfly_api/butterfly_api.dart';
 import 'package:flutter/material.dart';

app/lib/api/open.dart

@@ -1,8 +1,8 @@
 import 'dart:async';
-import 'dart:io';
 
+import 'package:butterfly/visualizer/asset.dart';
 import 'package:butterfly_api/butterfly_api.dart';
-import 'package:file_picker/file_picker.dart';
+import 'package:file_selector/file_selector.dart' as fs;
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:go_router/go_router.dart';
@@ -18,21 +18,19 @@ Future<bool> openHelp(List<String> pageLocation, [String? fragment]) {
       mode: LaunchMode.externalApplication);
 }
 
-Future<(Uint8List?, String?)> openSupported(
-    [List<String>? fileExtension]) async {
-  final isMobile = !kIsWeb && (Platform.isAndroid || Platform.isIOS);
-  final files = await FilePicker.platform.pickFiles(
-    type: isMobile ? FileType.any : FileType.custom,
-    allowedExtensions: isMobile
-        ? null
-        : (fileExtension ??
-            AssetFileType.values.expand((e) => e.getFileExtensions()).toList()),
-    allowMultiple: false,
-    withData: true,
+Future<(Uint8List?, String?)> importFile(BuildContext context,
+    [List<AssetFileType>? types]) async {
+  final result = await fs.openFile(
+    acceptedTypeGroups: (types ?? AssetFileType.values)
+        .map((e) => fs.XTypeGroup(
+              label: e.getLocalizedName(context),
+              extensions: e.getFileExtensions(),
+              mimeTypes: [e.getMime()],
+            ))
+        .toList(),
   );
-  if (files?.files.isEmpty ?? true) return (null, null);
-  var e = files!.files.first;
-  return (e.bytes, e.extension);
+  final data = await result?.readAsBytes();
+  return (data, result?.name.split('.').lastOrNull);
 }
 
 Future<void> openFile(

app/lib/api/save_data.dart app/lib/api/save.dartapp/lib/api/save_data.dart renamed to app/lib/api/save.dart

@@ -1,7 +1,4 @@
-import 'package:butterfly/api/save_data_stub.dart'
-    if (dart.library.io) 'package:butterfly/api/save_data_io.dart'
-    if (dart.library.js) 'package:butterfly/api/save_data_html.dart'
-    as save_data;
+import 'package:file_selector/file_selector.dart' as fs;
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_gen/gen_l10n/app_localizations.dart';
@@ -11,8 +8,21 @@ Future<void> exportFile(
   List<int> bytes,
   String fileExtension,
   String mimeType,
-) async =>
-    save_data.exportFile(context, bytes, fileExtension, mimeType);
+) async {
+  final result = await fs.getSaveLocation(
+    acceptedTypeGroups: [
+      fs.XTypeGroup(
+        label: AppLocalizations.of(context).export,
+        extensions: [fileExtension],
+        mimeTypes: [mimeType],
+      ),
+    ],
+  );
+  if (result == null) return;
+  final file = fs.XFile.fromData(Uint8List.fromList(bytes),
+      mimeType: mimeType, name: 'output.$fileExtension');
+  await file.saveTo(result.path);
+}
 
 Future<void> exportSvg(BuildContext context, String data) =>
     exportFile(context, data.codeUnits, 'svg', 'image/svg');

app/lib/api/save_data_html.dart

@@ -1,13 +1,10 @@
-import 'dart:io';
 import 'dart:ui' as ui;
 
 import 'package:butterfly/api/open.dart';
 import 'package:butterfly/bloc/document_bloc.dart';
 import 'package:butterfly/dialogs/texture.dart';
 import 'package:butterfly/visualizer/preset.dart';
 import 'package:butterfly_api/butterfly_api.dart';
-import 'package:file_picker/file_picker.dart';
-import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_gen/gen_l10n/app_localizations.dart';

app/lib/api/save_data_io.dart

@@ -49,16 +49,13 @@ class _GeneralBackgroundPropertiesView extends StatelessWidget {
               onTap: () async {
                 final state = context.read<DocumentBloc>().state;
                 if (state is! DocumentLoaded) return;
-                final files = await FilePicker.platform.pickFiles(
-                    type: FileType.image, allowMultiple: false, withData: true);
-                if (files?.files.isEmpty ?? true) return;
-                final e = files!.files.first;
-                var content = e.bytes ?? Uint8List(0);
-                if (!kIsWeb) {
-                  content = await File(e.path ?? '').readAsBytes();
-                }
-                final dataPath = Uri.dataFromBytes(content).toString();
-                final codec = await ui.instantiateImageCodec(content);
+                final (result, _) = await importFile(
+                  context,
+                  [AssetFileType.image],
+                );
+                if (result == null) return;
+                final dataPath = Uri.dataFromBytes(result).toString();
+                final codec = await ui.instantiateImageCodec(result);
                 final frame = await codec.getNextFrame();
                 final image = frame.image;
                 final width = image.width.toDouble(),

app/lib/api/save_data_stub.dart

@@ -1,4 +1,4 @@
-import 'package:butterfly/api/save_data.dart';
+import 'package:butterfly/api/save.dart';
 import 'package:butterfly/bloc/document_bloc.dart';
 import 'package:butterfly/cubits/current_index.dart';
 import 'package:butterfly/services/network.dart';

app/lib/dialogs/background/dialog.dart

@@ -1,6 +1,6 @@
 import 'dart:ui' as ui;
 
-import 'package:butterfly/api/save_data.dart';
+import 'package:butterfly/api/save.dart';
 import 'package:butterfly/bloc/document_bloc.dart';
 import 'package:butterfly_api/butterfly_api.dart';
 import 'package:flutter/foundation.dart';