[fix](pnalvarez): using block listener to answer to storage saving errors

Author: pnalvarez

lib/core/designsystem/organisms/show_error_dialog.dart

@@ -7,6 +7,7 @@ Future<void> showErrorDialog(
   String title,
   String content,
   String ctaText,
+  Function()? onCtaPressed,
 ) {
   return showDialog<void>(
     context: context,
@@ -22,8 +23,11 @@ Future<void> showErrorDialog(
       actions: [
         Center(
           child: PrimaryButton(
-            title: title,
-            onPressed: () => Navigator.of(context).pop(),
+            title: ctaText,
+            onPressed: () {
+              onCtaPressed?.call();
+              Navigator.of(context).pop();
+            },
           ),
         ),
       ],

lib/layers/presentation/screens/startreading/start_reading_event.dart

@@ -8,3 +8,5 @@ class DidEditProgressEvent extends StartReadingEvent {
 class DidClickConfirmEvent extends StartReadingEvent {}
 
 class DidClickFinishBookEvent extends StartReadingEvent {}
+
+class DidClickSavingErrorDismissEvent extends StartReadingEvent {}

lib/layers/presentation/screens/startreading/start_reading_page.dart

@@ -7,6 +7,7 @@ import 'package:mibook/core/designsystem/molecules/buttons/secondary_button.dart
 import 'package:mibook/core/designsystem/molecules/indicators/progress_stepper.dart';
 import 'package:mibook/core/designsystem/molecules/inputfields/input_field.dart';
 import 'package:mibook/core/designsystem/organisms/app_nav_bar.dart';
+import 'package:mibook/core/designsystem/organisms/show_error_dialog.dart';
 import 'package:mibook/core/di/di.dart';
 import 'package:mibook/core/utils/strings.dart';
 import 'package:mibook/core/utils/strings.dart' as strings;
@@ -15,6 +16,9 @@ import 'package:mibook/layers/presentation/screens/startreading/start_reading_ev
 import 'package:mibook/layers/presentation/screens/startreading/start_reading_state.dart';
 import 'package:mibook/layers/presentation/screens/startreading/start_reading_view_model.dart';
 
+typedef _BlocListener = BlocListener<StartReadingViewModel, StartReadingState>;
+typedef _BlocBuilder = BlocBuilder<StartReadingViewModel, StartReadingState>;
+
 @RoutePage()
 class StartReadingPage extends StatelessWidget {
   final BookDetailsUI book;
@@ -36,34 +40,57 @@ class StartReadingPage extends StatelessWidget {
 class _StartReadingScaffold extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
-    return Scaffold(
-      appBar: AppNavBar(
-        titleText: startReading,
-        onBack: context.router.maybePop,
-      ),
-      body: BlocBuilder<StartReadingViewModel, StartReadingState>(
-        builder: (context, state) {
-          final viewModel = context.read<StartReadingViewModel>();
-          if (state.shouldNavigateBack) {
-            context.router.maybePop();
-          }
-          return _StartReadingContent(
-            book: viewModel.book,
-            progress: state.progress,
-            errorMessage: state.inputErrorMessage,
-            onChangeProgressText: (progress) {
-              viewModel.add(
-                DidEditProgressEvent(progress: int.tryParse(progress) ?? 0),
-              );
-            },
-            onClickStartReading: () {
-              viewModel.add(DidClickConfirmEvent());
-            },
-            onClickFinishBook: () {
-              viewModel.add(DidClickFinishBookEvent());
+    final viewModel = context.read<StartReadingViewModel>();
+
+    return _BlocListener(
+      bloc: viewModel,
+      listenWhen: (previous, current) =>
+          previous.shouldShowSavingError != current.shouldShowSavingError &&
+          current.shouldShowSavingError,
+      listener: (context, state) {
+        WidgetsBinding.instance.addPostFrameCallback((_) {
+          showErrorDialog(
+            context,
+            'Error',
+            'An error occurred while saving your reading progress. Please try again.',
+            'OK',
+            () {
+              viewModel.add(DidClickSavingErrorDismissEvent());
             },
           );
-        },
+        });
+      },
+      child: Scaffold(
+        appBar: AppNavBar(
+          titleText: startReading,
+          onBack: context.router.maybePop,
+        ),
+        body: _BlocBuilder(
+          builder: (context, state) {
+            final viewModel = context.read<StartReadingViewModel>();
+            if (state.shouldNavigateBack) {
+              context.router.maybePop();
+            }
+            return _StartReadingContent(
+              book: viewModel.book,
+              progress: state.progress,
+              errorMessage: state.inputErrorMessage,
+              onChangeProgressText: (progress) {
+                viewModel.add(
+                  DidEditProgressEvent(
+                    progress: int.tryParse(progress) ?? 0,
+                  ),
+                );
+              },
+              onClickStartReading: () {
+                viewModel.add(DidClickConfirmEvent());
+              },
+              onClickFinishBook: () {
+                viewModel.add(DidClickFinishBookEvent());
+              },
+            );
+          },
+        ),
       ),
     );
   }

lib/layers/presentation/screens/startreading/start_reading_view_model.dart

@@ -30,6 +30,10 @@ class StartReadingViewModel extends Bloc<StartReadingEvent, StartReadingState> {
       final response = await _didClickFinishBook();
       emit(response);
     });
+    on<DidClickSavingErrorDismissEvent>((event, emit) {
+      final response = _handleDidClickSavingErrorDismiss();
+      emit(response);
+    });
   }
 
   // Event Handler to DidEditProgressEvent
@@ -62,7 +66,15 @@ class StartReadingViewModel extends Bloc<StartReadingEvent, StartReadingState> {
       bookThumb: book.thumbnail,
       progress: progress,
     );
-    await _startReading(reading: reading);
-    return state.copyWith(shouldNavigateBack: true);
+    try {
+      await _startReading(reading: reading);
+      return state.copyWith(shouldNavigateBack: true);
+    } catch (_) {
+      print('Error saving reading for bookId: ${book.id}');
+      return state.copyWith(shouldShowSavingError: true);
+    }
   }
+
+  StartReadingState _handleDidClickSavingErrorDismiss() =>
+      state.copyWith(shouldShowSavingError: false);
 }

pubspec.lock

@@ -49,6 +49,14 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "9.2.2"
+  auto_route_generator:
+    dependency: "direct main"
+    description:
+      name: auto_route_generator
+      sha256: c9086eb07271e51b44071ad5cff34e889f3156710b964a308c2ab590769e79e6
+      url: "https://pub.dev"
+    source: hosted
+    version: "9.0.0"
   auto_size_text:
     dependency: "direct main"
     description:
@@ -672,6 +680,14 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "2.3.0"
+  petitparser:
+    dependency: transitive
+    description:
+      name: petitparser
+      sha256: "1a97266a94f7350d30ae522c0af07890c70b8e62c71e8e3920d1db4d23c057d1"
+      url: "https://pub.dev"
+    source: hosted
+    version: "7.0.1"
   platform:
     dependency: transitive
     description:
@@ -1093,6 +1109,14 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "1.1.0"
+  xml:
+    dependency: transitive
+    description:
+      name: xml
+      sha256: "971043b3a0d3da28727e40ed3e0b5d18b742fa5a68665cca88e74b7876d5e025"
+      url: "https://pub.dev"
+    source: hosted
+    version: "6.6.1"
   yaml:
     dependency: transitive
     description:

pubspec.yaml

@@ -32,6 +32,7 @@ dependencies:
   shared_preferences: 2.5.3
   auto_size_text: 3.0.0
   path_provider: ^2.1.5
+  auto_route_generator: ^9.0.0
 
 dev_dependencies:
   flutter_test: