Implemented error reporting solution

2025-12-09 12:08:05 -06:00 · 2022-11-14 00:08:07 +01:00
parent b5aac36b2b
commit 511f8ca3f4
33 changed files with 475 additions and 110 deletions
--- a/lib/features/documents/view/pages/document_details_page.dart
+++ b/lib/features/documents/view/pages/document_details_page.dart
@@ -234,8 +234,8 @@ class _DocumentDetailsPageState extends State<DocumentDetailsPage> {
  Future<void> _assignAsn(DocumentModel document) async {
    try {
      await BlocProvider.of<DocumentsCubit>(context).assignAsn(document);
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }

@@ -409,8 +409,8 @@ class _DocumentDetailsPageState extends State<DocumentDetailsPage> {
      try {
        await BlocProvider.of<DocumentsCubit>(context).removeDocument(document);
        showSnackBar(context, S.of(context).documentDeleteSuccessMessage);
-      } on ErrorMessage catch (error) {
-        showError(context, error);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
      } finally {
        Navigator.pop(context);
      }
--- a/lib/features/documents/view/pages/document_edit_page.dart
+++ b/lib/features/documents/view/pages/document_edit_page.dart
@@ -80,8 +80,8 @@ class _DocumentEditPageState extends State<DocumentEditPage> {
            try {
              await getIt<DocumentsCubit>().updateDocument(updatedDocument);
              showSnackBar(context, S.of(context).documentUpdateErrorMessage);
-            } on ErrorMessage catch (error) {
-              showError(context, error);
+            } on ErrorMessage catch (error, stackTrace) {
+              showError(context, error, stackTrace);
            } finally {
              Navigator.pop(context);
            }
--- a/lib/features/documents/view/pages/documents_page.dart
+++ b/lib/features/documents/view/pages/documents_page.dart
@@ -3,6 +3,7 @@ import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:paperless_mobile/core/bloc/connectivity_cubit.dart';
 import 'package:paperless_mobile/core/logic/error_code_localization_mapper.dart';
 import 'package:paperless_mobile/core/model/error_message.dart';
+import 'package:paperless_mobile/core/service/github_issue_service.dart';
 import 'package:paperless_mobile/core/widgets/offline_banner.dart';
 import 'package:paperless_mobile/di_initializer.dart';
 import 'package:paperless_mobile/features/labels/correspondent/bloc/correspondents_cubit.dart';
@@ -53,8 +54,8 @@ class _DocumentsPageState extends State<DocumentsPage> {
  Future<void> _initDocuments() async {
    try {
      BlocProvider.of<DocumentsCubit>(context).loadDocuments();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }

@@ -73,9 +74,20 @@ class _DocumentsPageState extends State<DocumentsPage> {
    }
    try {
      await documentsCubit.loadMore();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
+    Future.delayed(const Duration(seconds: 1), () {
+      try {
+        throw ErrorMessage(ErrorCode.tagLoadFailed);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(
+          context,
+          error,
+          stackTrace,
+        );
+      }
+    });
  }

  void _onSelected(DocumentModel model) {
@@ -87,8 +99,8 @@ class _DocumentsPageState extends State<DocumentsPage> {
      await BlocProvider.of<DocumentsCubit>(context).updateCurrentFilter(
        (filter) => filter.copyWith(page: 1),
      );
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }

--- a/lib/features/documents/view/widgets/search/document_filter_panel.dart
+++ b/lib/features/documents/view/widgets/search/document_filter_panel.dart
@@ -536,8 +536,8 @@ class _DocumentFilterPanelState extends State<DocumentFilterPanel> {
        BlocProvider.of<SavedViewCubit>(context).resetSelection();
        FocusScope.of(context).unfocus();
        widget.panelController.close();
-      } on ErrorMessage catch (error) {
-        showError(context, error);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
      }
    }
  }
--- a/lib/features/documents/view/widgets/selection/documents_page_app_bar.dart
+++ b/lib/features/documents/view/widgets/selection/documents_page_app_bar.dart
@@ -87,8 +87,8 @@ class _DocumentsPageAppBarState extends State<DocumentsPageAppBar> {
          context,
          S.of(context).documentsPageBulkDeleteSuccessfulText,
        );
-      } on ErrorMessage catch (error) {
-        showError(context, error);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
      }
    }
  }
--- a/lib/features/documents/view/widgets/selection/saved_view_selection_widget.dart
+++ b/lib/features/documents/view/widgets/selection/saved_view_selection_widget.dart
@@ -85,8 +85,8 @@ class SavedViewSelectionWidget extends StatelessWidget {
    if (newView != null) {
      try {
        await BlocProvider.of<SavedViewCubit>(context).add(newView);
-      } on ErrorMessage catch (error) {
-        showError(context, error);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
      }
    }
  }
@@ -102,8 +102,8 @@ class SavedViewSelectionWidget extends StatelessWidget {
        BlocProvider.of<DocumentsCubit>(context).updateFilter();
        BlocProvider.of<SavedViewCubit>(context).selectView(null);
      }
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }

@@ -117,8 +117,8 @@ class SavedViewSelectionWidget extends StatelessWidget {
      if (delete) {
        try {
          BlocProvider.of<SavedViewCubit>(context).remove(view);
-        } on ErrorMessage catch (error) {
-          showError(context, error);
+        } on ErrorMessage catch (error, stackTrace) {
+          showError(context, error, stackTrace);
        }
      }
    }
--- a/lib/features/documents/view/widgets/sort_documents_button.dart
+++ b/lib/features/documents/view/widgets/sort_documents_button.dart
@@ -50,8 +50,8 @@ class _SortDocumentsButtonState extends State<SortDocumentsButton> {
                    sortOrder: state.filter.sortOrder.toggle(),
                  ),
                );
-              } on ErrorMessage catch (error) {
-                showError(context, error);
+              } on ErrorMessage catch (error, stackTrace) {
+                showError(context, error, stackTrace);
              } finally {
                setState(() => _isLoading = false);
              }
--- a/lib/features/home/view/home_page.dart
+++ b/lib/features/home/view/home_page.dart
@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:paperless_mobile/core/bloc/connectivity_cubit.dart';
-import 'package:paperless_mobile/core/logic/error_code_localization_mapper.dart';
+import 'package:paperless_mobile/core/bloc/paperless_server_information_cubit.dart';
 import 'package:paperless_mobile/core/model/error_message.dart';
 import 'package:paperless_mobile/core/widgets/offline_banner.dart';
 import 'package:paperless_mobile/di_initializer.dart';
@@ -32,7 +32,7 @@ class _HomePageState extends State<HomePage> {
  @override
  void initState() {
    super.initState();
-    initializeLabelData(context);
+    _initializeData(context);
  }

  @override
@@ -41,7 +41,7 @@ class _HomePageState extends State<HomePage> {
      //Only re-initialize data if the connectivity changed from not connected to connected
      listenWhen: (previous, current) => current == ConnectivityState.connected,
      listener: (context, state) {
-        initializeLabelData(context);
+        _initializeData(context);
      },
      builder: (context, connectivityState) {
        return Scaffold(
@@ -73,15 +73,17 @@ class _HomePageState extends State<HomePage> {
    );
  }

-  initializeLabelData(BuildContext context) {
+  _initializeData(BuildContext context) async {
    try {
+      await BlocProvider.of<PaperlessServerInformationCubit>(context)
+          .updateStatus();
      BlocProvider.of<DocumentTypeCubit>(context).initialize();
      BlocProvider.of<CorrespondentCubit>(context).initialize();
      BlocProvider.of<TagCubit>(context).initialize();
      BlocProvider.of<StoragePathCubit>(context).initialize();
      BlocProvider.of<SavedViewCubit>(context).initialize();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/home/view/widget/bottom_navigation_bar.dart
+++ b/lib/features/home/view/widget/bottom_navigation_bar.dart
@@ -19,15 +19,29 @@ class BottomNavBar extends StatelessWidget {
      selectedIndex: selectedIndex,
      destinations: [
        NavigationDestination(
-          icon: const Icon(Icons.description),
+          icon: const Icon(Icons.description_outlined),
+          selectedIcon: Icon(
+            Icons.description,
+            color: Theme.of(context).colorScheme.primary,
+          ),
          label: S.of(context).bottomNavDocumentsPageLabel,
        ),
        NavigationDestination(
          icon: const Icon(Icons.document_scanner),
+          selectedIcon: Icon(
+            Icons.document_scanner,
+            color: Theme.of(context).colorScheme.primary,
+          ),
          label: S.of(context).bottomNavScannerPageLabel,
        ),
        NavigationDestination(
-          icon: const Icon(Icons.sell),
+          icon: const Icon(
+            Icons.sell,
+          ),
+          selectedIcon: Icon(
+            Icons.sell,
+            color: Theme.of(context).colorScheme.primary,
+          ),
          label: S.of(context).bottomNavLabelsPageLabel,
        ),
      ],
--- a/lib/features/home/view/widget/info_drawer.dart
+++ b/lib/features/home/view/widget/info_drawer.dart
@@ -1,6 +1,8 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:paperless_mobile/core/bloc/paperless_server_information_cubit.dart';
 import 'package:paperless_mobile/core/model/error_message.dart';
+import 'package:paperless_mobile/core/model/paperless_server_information.dart';
 import 'package:paperless_mobile/features/settings/bloc/application_settings_cubit.dart';
 import 'package:paperless_mobile/di_initializer.dart';
 import 'package:paperless_mobile/features/labels/correspondent/bloc/correspondents_cubit.dart';
@@ -39,24 +41,46 @@ class InfoDrawer extends StatelessWidget {
                    Text(
                      S.of(context).appTitleText,
                      style: Theme.of(context).textTheme.headline5!.copyWith(
-                          color:
-                              Theme.of(context).colorScheme.onPrimaryContainer),
+                            color: Theme.of(context)
+                                .colorScheme
+                                .onPrimaryContainer,
+                          ),
                    ),
                  ],
                ),
                Align(
                  alignment: Alignment.bottomRight,
-                  child: BlocBuilder<AuthenticationCubit, AuthenticationState>(
+                  child: BlocBuilder<PaperlessServerInformationCubit,
+                      PaperlessServerInformation>(
                    builder: (context, state) {
-                      return Text(
-                        state.authentication?.serverUrl
-                                .replaceAll(RegExp(r'https?://'), "") ??
-                            "",
-                        textAlign: TextAlign.end,
-                        style: TextStyle(
-                            color: Theme.of(context)
-                                .colorScheme
-                                .onPrimaryContainer),
+                      return Column(
+                        crossAxisAlignment: CrossAxisAlignment.end,
+                        children: [
+                          ListTile(
+                            contentPadding: EdgeInsets.zero,
+                            dense: true,
+                            title: Text(
+                              'example.paperless.myinstance.com.de',
+                              overflow: TextOverflow.ellipsis,
+                              textAlign: TextAlign.end,
+                              maxLines: 1,
+                            ),
+                            isThreeLine: true,
+                            subtitle: Column(
+                              crossAxisAlignment: CrossAxisAlignment.end,
+                              children: [
+                                Text(
+                                  'Logged in as anton',
+                                  textAlign: TextAlign.end,
+                                ),
+                                Text(
+                                  '${S.of(context).serverInformationPaperlessVersionText} ${state.version} (API v${state.apiVersion})',
+                                  style: Theme.of(context).textTheme.caption,
+                                ),
+                              ],
+                            ),
+                          ),
+                        ],
                      );
                    },
                  ),
@@ -134,8 +158,8 @@ class InfoDrawer extends StatelessWidget {
                getIt<DocumentTypeCubit>().reset();
                getIt<TagCubit>().reset();
                getIt<DocumentScannerCubit>().reset();
-              } on ErrorMessage catch (error) {
-                showError(context, error);
+              } on ErrorMessage catch (error, stackTrace) {
+                showError(context, error, stackTrace);
              }
            },
          ),
--- a/lib/features/labels/correspondent/view/pages/edit_correspondent_page.dart
+++ b/lib/features/labels/correspondent/view/pages/edit_correspondent_page.dart
@@ -38,8 +38,8 @@ class EditCorrespondentPage extends StatelessWidget {
        );
      }
      Navigator.pop(context);
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/correspondent/view/widgets/correspondent_widget.dart
+++ b/lib/features/labels/correspondent/view/widgets/correspondent_widget.dart
@@ -59,8 +59,8 @@ class CorrespondentWidget extends StatelessWidget {
        );
      }
      afterSelected?.call();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/document_type/view/widgets/document_type_widget.dart
+++ b/lib/features/labels/document_type/view/widgets/document_type_widget.dart
@@ -54,8 +54,8 @@ class DocumentTypeWidget extends StatelessWidget {
        );
      }
      afterSelected?.call();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/storage_path/view/pages/edit_storage_path_page.dart
+++ b/lib/features/labels/storage_path/view/pages/edit_storage_path_page.dart
@@ -43,8 +43,8 @@ class EditStoragePathPage extends StatelessWidget {
        );
      }
      Navigator.pop(context);
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/storage_path/view/widgets/storage_path_widget.dart
+++ b/lib/features/labels/storage_path/view/widgets/storage_path_widget.dart
@@ -58,8 +58,8 @@ class StoragePathWidget extends StatelessWidget {
        );
      }
      afterSelected?.call();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/tags/view/pages/edit_tag_page.dart
+++ b/lib/features/labels/tags/view/pages/edit_tag_page.dart
@@ -57,8 +57,8 @@ class EditTagPage extends StatelessWidget {
      }
      cubit.updateFilter(filter: updatedFilter);
      Navigator.pop(context);
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/tags/view/widgets/tag_widget.dart
+++ b/lib/features/labels/tags/view/widgets/tag_widget.dart
@@ -28,6 +28,7 @@ class TagWidget extends StatelessWidget {
              tag.name,
              style: TextStyle(color: tag.textColor),
            ),
+            checkmarkColor: tag.textColor,
            backgroundColor: tag.color,
            side: BorderSide.none,
          );
@@ -57,8 +58,8 @@ class TagWidget extends StatelessWidget {
      if (afterTagTapped != null) {
        afterTagTapped!();
      }
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }
 }
--- a/lib/features/labels/view/pages/edit_label_page.dart
+++ b/lib/features/labels/view/pages/edit_label_page.dart
@@ -146,8 +146,8 @@ class _EditLabelPageState<T extends Label> extends State<EditLabelPage<T>> {
        Navigator.pop(context);
      } on PaperlessValidationErrors catch (errorMessages) {
        setState(() => _errors = errorMessages);
-      } on ErrorMessage catch (error) {
-        showError(context, error);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
      }
    }
  }
--- a/lib/features/login/view/login_page.dart
+++ b/lib/features/login/view/login_page.dart
@@ -97,10 +97,10 @@ class _LoginPageState extends State<LoginPage> {
          clientCertificate:
              form[ClientCertificateFormField.fkClientCertificate],
        );
-      } on ErrorMessage catch (error) {
-        showError(context, error);
-      } catch (unknownError) {
-        showSnackBar(context, unknownError.toString());
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
+      } catch (unknownError, stackTrace) {
+        showError(context, ErrorMessage.unknown(), stackTrace);
      } finally {
        setState(() => _isLoginLoading = false);
      }
--- a/lib/features/scan/view/document_upload_page.dart
+++ b/lib/features/scan/view/document_upload_page.dart
@@ -212,12 +212,12 @@ class _DocumentUploadPageState extends State<DocumentUploadPage> {
        showSnackBar(context, S.of(context).documentUploadSuccessText);
        Navigator.pop(context);
        widget.afterUpload?.call();
-      } on ErrorMessage catch (error) {
-        showError(context, error);
+      } on ErrorMessage catch (error, stackTrace) {
+        showError(context, error, stackTrace);
      } on PaperlessValidationErrors catch (errorMessages) {
        setState(() => _errors = errorMessages);
-      } catch (other) {
-        showSnackBar(context, other.toString());
+      } catch (unknownError, stackTrace) {
+        showError(context, ErrorMessage.unknown(), stackTrace);
      } finally {
        setState(() {
          _isUploadLoading = false;
@@ -233,8 +233,8 @@ class _DocumentUploadPageState extends State<DocumentUploadPage> {
          onPressed: () async {
            try {
              getIt<DocumentsCubit>().reloadDocuments();
-            } on ErrorMessage catch (error) {
-              showError(context, error);
+            } on ErrorMessage catch (error, stackTrace) {
+              showError(context, error, stackTrace);
            }
          },
          label:
--- a/lib/features/scan/view/scanner_page.dart
+++ b/lib/features/scan/view/scanner_page.dart
@@ -201,8 +201,8 @@ class _ScannerPageState extends State<ScannerPage>
              try {
                BlocProvider.of<DocumentScannerCubit>(context)
                    .removeScan(index);
-              } on ErrorMessage catch (error) {
-                showError(context, error);
+              } on ErrorMessage catch (error, stackTrace) {
+                showError(context, error, stackTrace);
              }
            },
            index: index,
@@ -214,8 +214,8 @@ class _ScannerPageState extends State<ScannerPage>
  void _reset(BuildContext context) {
    try {
      BlocProvider.of<DocumentScannerCubit>(context).reset();
-    } on ErrorMessage catch (error) {
-      showError(context, error);
+    } on ErrorMessage catch (error, stackTrace) {
+      showError(context, error, stackTrace);
    }
  }