fix: add warning on download modal

ardriveapp · Oct 25, 2023 · 687b0db · 687b0db
1 parent 0c3f991
commit 687b0db
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 2 deletions.
diff --git a/lib/blocs/file_download/file_download_cubit.dart b/lib/blocs/file_download/file_download_cubit.dart
@@ -5,6 +5,7 @@ import 'package:ardrive/core/arfs/entities/arfs_entities.dart';
 import 'package:ardrive/core/arfs/repository/arfs_repository.dart';
 import 'package:ardrive/core/crypto/crypto.dart';
 import 'package:ardrive/download/ardrive_downloader.dart';
+import 'package:ardrive/download/limits.dart';
 import 'package:ardrive/models/models.dart';
 import 'package:ardrive/services/services.dart';
 import 'package:ardrive/utils/logger/logger.dart';
@@ -13,6 +14,7 @@ import 'package:ardrive_io/ardrive_io.dart' as io;
 import 'package:ardrive_io/ardrive_io.dart';
 import 'package:ardrive_utils/ardrive_utils.dart';
 import 'package:cryptography/cryptography.dart';
+import 'package:drift/drift.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';

diff --git a/lib/blocs/file_download/file_download_state.dart b/lib/blocs/file_download/file_download_state.dart
@@ -81,6 +81,7 @@ class FileDownloadAborted extends FileDownloadState {}
 enum FileDownloadFailureReason {
   unknownError,
   fileAboveLimit,
+  browserDoesNotSupportLargeDownloads,
   networkConnectionError,
   fileNotFound
 }
diff --git a/lib/blocs/file_download/personal_file_download_cubit.dart b/lib/blocs/file_download/personal_file_download_cubit.dart
@@ -42,6 +42,18 @@ class ProfileFileDownloadCubit extends FileDownloadCubit {
         _arfsRepository = arfsRepository,
         super(FileDownloadStarting());
 
+  Future<void> verifyUploadLimitationsAndDownload(SecretKey? cipherKey) async {
+    if (await AppPlatform.isSafari()) {
+      if (_file.size > publicDownloadSafariSizeLimit) {
+        emit(const FileDownloadFailure(
+            FileDownloadFailureReason.browserDoesNotSupportLargeDownloads));
+        return;
+      }
+    }
+
+    download(cipherKey);
+  }
+
   Future<void> download(SecretKey? cipherKey) async {
     try {
       final drive = await _arfsRepository.getDriveById(_file.driveId);

diff --git a/lib/components/file_download_dialog.dart b/lib/components/file_download_dialog.dart
@@ -42,7 +42,7 @@ Future<void> promptToDownloadProfileFile({
     file: arfsFile,
     driveDao: context.read<DriveDao>(),
     arweave: arweave,
-  )..download(cipherKey);
+  )..verifyUploadLimitationsAndDownload(cipherKey);
   return showArDriveDialog(
     context,
     barrierDismissible: false,
@@ -77,7 +77,7 @@ Future<void> promptToDownloadFileRevision({
     file: arfsFile,
     driveDao: context.read<DriveDao>(),
     arweave: arweave,
-  )..download(cipherKey);
+  )..verifyUploadLimitationsAndDownload(cipherKey);
 
   return showArDriveDialog(
     context,
@@ -142,6 +142,9 @@ class FileDownloadDialog extends StatelessWidget {
           } else if (state is FileDownloadFailure) {
             if (state.reason == FileDownloadFailureReason.unknownError) {
               return _fileDownloadFailedDialog(context);
+            } else if (state.reason ==
+                FileDownloadFailureReason.browserDoesNotSupportLargeDownloads) {
+              return _fileDownloadFailedDueToAboveBrowserLimit(context);
             }
 
             return _fileDownloadFailedDueToFileAbovePrivateLimit(context);
@@ -181,6 +184,21 @@ class FileDownloadDialog extends StatelessWidget {
     );
   }
 
+  ArDriveStandardModal _fileDownloadFailedDueToAboveBrowserLimit(
+      BuildContext context) {
+    return _modalWrapper(
+      title: appLocalizationsOf(context).warningEmphasized,
+      description:
+          appLocalizationsOf(context).fileFailedToDownloadFileAbovePublicLimit,
+      actions: [
+        ModalAction(
+          action: () => Navigator.pop(context),
+          title: appLocalizationsOf(context).ok,
+        ),
+      ],
+    );
+  }
+
   ArDriveStandardModal _warningToWaitDownloadFinishes(BuildContext context) {
     return _modalWrapper(
         title: appLocalizationsOf(context).warningEmphasized,

diff --git a/lib/download/limits.dart b/lib/download/limits.dart
@@ -7,6 +7,7 @@ import 'download_utils.dart';
 final publicDownloadUnknownPlatformSizeLimit = const GiB(2).size;
 final publicDownloadWebSizeLimit = const MiB(500).size;
 final publicDownloadFirefoxSizeLimit = const GiB(2).size;
+final publicDownloadSafariSizeLimit = const GiB(1).size;
 final publicDownloadMobileSizeLimit = const MiB(300).size;
 
 final privateDownloadUnknownPlatformSizeLimit = const GiB(2).size;