feat(neon_files): Implement sharing file natively

Signed-off-by: jld3103 <[email protected]>
nextcloud · Sep 17, 2023 · 7415278 · 7415278
1 parent 7bc7cc2
commit 7415278
Show file tree

Hide file tree

Showing 10 changed files with 61 additions and 9 deletions.
diff --git a/packages/neon/neon/lib/src/platform/android.dart b/packages/neon/neon/lib/src/platform/android.dart
@@ -28,6 +28,9 @@ class AndroidNeonPlatform implements NeonPlatform {
   @override
   bool get canUseWindowManager => false;
 
+  @override
+  bool get canUseSharing => true;
+
   @override
   Future<String> get userAccessibleAppDataPath async {
     if (!await Permission.storage.request().isGranted) {

diff --git a/packages/neon/neon/lib/src/platform/linux.dart b/packages/neon/neon/lib/src/platform/linux.dart
@@ -28,6 +28,9 @@ class LinuxNeonPlatform implements NeonPlatform {
   @override
   bool get canUsePushNotifications => false;
 
+  @override
+  bool get canUseSharing => false;
+
   @override
   String get userAccessibleAppDataPath => p.join(Platform.environment['HOME']!, 'Neon');
 

diff --git a/packages/neon/neon/lib/src/platform/platform.dart b/packages/neon/neon/lib/src/platform/platform.dart
@@ -53,6 +53,8 @@ abstract interface class NeonPlatform {
 
   abstract final bool canUsePushNotifications;
 
+  abstract final bool canUseSharing;
+
   FutureOr<String> get userAccessibleAppDataPath;
 
   FutureOr<void> init();

diff --git a/packages/neon/neon_files/lib/blocs/files.dart b/packages/neon/neon_files/lib/blocs/files.dart
@@ -7,6 +7,8 @@ abstract interface class FilesBlocEvents {
 
   void openFile(final List<String> path, final String etag, final String? mimeType);
 
+  void shareFileNative(final List<String> path, final String etag);
+
   void delete(final List<String> path);
 
   void rename(final List<String> path, final String name);
@@ -84,15 +86,8 @@ class FilesBloc extends InteractiveBloc implements FilesBlocEvents, FilesBlocSta
   void openFile(final List<String> path, final String etag, final String? mimeType) {
     wrapAction(
       () async {
-        final cacheDir = await getApplicationCacheDirectory();
-        final file = File(p.join(cacheDir.path, 'files', etag.replaceAll('"', ''), path.last));
-        if (!file.existsSync()) {
-          debugPrint('Downloading ${Uri(pathSegments: path)} since it does not exist');
-          if (!file.parent.existsSync()) {
-            await file.parent.create(recursive: true);
-          }
-          await _downloadFile(path, file);
-        }
+        final file = await _cacheFile(path, etag);
+
         final result = await OpenFile.open(file.path, type: mimeType);
         if (result.type != ResultType.done) {
           throw UnableToOpenFileException();
@@ -102,6 +97,18 @@ class FilesBloc extends InteractiveBloc implements FilesBlocEvents, FilesBlocSta
     );
   }
 
+  @override
+  void shareFileNative(final List<String> path, final String etag) {
+    wrapAction(
+      () async {
+        final file = await _cacheFile(path, etag);
+
+        await Share.shareXFiles([XFile(file.path)]);
+      },
+      disableTimeout: true,
+    );
+  }
+
   @override
   Future<void> refresh() async {
     await browser.refresh();
@@ -164,6 +171,21 @@ class FilesBloc extends InteractiveBloc implements FilesBlocEvents, FilesBlocSta
     );
   }
 
+  Future<File> _cacheFile(final List<String> path, final String etag) async {
+    final cacheDir = await getApplicationCacheDirectory();
+    final file = File(p.join(cacheDir.path, 'files', etag.replaceAll('"', ''), path.last));
+
+    if (!file.existsSync()) {
+      debugPrint('Downloading ${Uri(pathSegments: path)} since it does not exist');
+      if (!file.parent.existsSync()) {
+        await file.parent.create(recursive: true);
+      }
+      await _downloadFile(path, file);
+    }
+
+    return file;
+  }
+
   Future<void> _downloadFile(
     final List<String> path,
     final File file,

diff --git a/packages/neon/neon_files/lib/l10n/en.arb b/packages/neon/neon_files/lib/l10n/en.arb
@@ -7,6 +7,7 @@
   "actionMove": "Move",
   "actionCopy": "Copy",
   "actionSync": "Sync",
+  "actionShare": "Share",
   "general": "General",
   "goToPath": "Go to /{path}",
   "@goToPath": {

diff --git a/packages/neon/neon_files/lib/l10n/localizations.dart b/packages/neon/neon_files/lib/l10n/localizations.dart
@@ -131,6 +131,12 @@ abstract class AppLocalizations {
   /// **'Sync'**
   String get actionSync;
 
+  /// No description provided for @actionShare.
+  ///
+  /// In en, this message translates to:
+  /// **'Share'**
+  String get actionShare;
+
   /// No description provided for @general.
   ///
   /// In en, this message translates to:

diff --git a/packages/neon/neon_files/lib/l10n/localizations_en.dart b/packages/neon/neon_files/lib/l10n/localizations_en.dart
@@ -25,6 +25,9 @@ class AppLocalizationsEn extends AppLocalizations {
   @override
   String get actionSync => 'Sync';
 
+  @override
+  String get actionShare => 'Share';
+
   @override
   String get general => 'General';
 

diff --git a/packages/neon/neon_files/lib/neon_files.dart b/packages/neon/neon_files/lib/neon_files.dart
@@ -30,6 +30,7 @@ import 'package:path_provider/path_provider.dart';
 import 'package:provider/provider.dart';
 import 'package:queue/queue.dart';
 import 'package:rxdart/rxdart.dart';
+import 'package:share_plus/share_plus.dart';
 
 part 'blocs/browser.dart';
 part 'blocs/files.dart';

diff --git a/packages/neon/neon_files/lib/widgets/actions.dart b/packages/neon/neon_files/lib/widgets/actions.dart
@@ -1,5 +1,6 @@
 import 'package:filesize/filesize.dart';
 import 'package:flutter/material.dart';
+import 'package:neon/platform.dart';
 import 'package:neon/utils.dart';
 import 'package:neon_files/l10n/localizations.dart';
 import 'package:neon_files/neon_files.dart';
@@ -17,6 +18,8 @@ class FileActions extends StatelessWidget {
     final bloc = Provider.of<FilesBloc>(context, listen: false);
     final browserBloc = bloc.browser;
     switch (action) {
+      case FilesFileAction.share:
+        bloc.shareFileNative(details.path, details.etag!);
       case FilesFileAction.toggleFavorite:
         if (details.isFavorite ?? false) {
           bloc.removeFavorite(details.path);
@@ -118,6 +121,12 @@ class FileActions extends StatelessWidget {
   @override
   Widget build(final BuildContext context) => PopupMenuButton<FilesFileAction>(
         itemBuilder: (final context) => [
+          if (!details.isDirectory && NeonPlatform.instance.canUseSharing) ...[
+            PopupMenuItem(
+              value: FilesFileAction.share,
+              child: Text(AppLocalizations.of(context).actionShare),
+            ),
+          ],
           if (details.isFavorite != null) ...[
             PopupMenuItem(
               value: FilesFileAction.toggleFavorite,
@@ -161,6 +170,7 @@ class FileActions extends StatelessWidget {
 }
 
 enum FilesFileAction {
+  share,
   toggleFavorite,
   details,
   rename,

diff --git a/packages/neon/neon_files/pubspec.yaml b/packages/neon/neon_files/pubspec.yaml
@@ -33,6 +33,7 @@ dependencies:
   provider: ^6.0.5
   queue: ^3.1.0+2
   rxdart: ^0.27.7
+  share_plus: ^4.5.3
 
 dev_dependencies:
   build_runner: ^2.4.6