Merge pull request ubuntu#1723 from d-loose/error-view

feat: add custom error view

Author: d-loose

packages/app_center/assets/error.svg

@@ -4,6 +4,7 @@ import 'package:app_center/appstream/appstream.dart';
 import 'package:app_center/constants.dart';
 import 'package:app_center/deb/deb_model.dart';
 import 'package:app_center/deb/deb_providers.dart';
+import 'package:app_center/error/error.dart';
 import 'package:app_center/l10n.dart';
 import 'package:app_center/layout.dart';
 import 'package:app_center/packagekit/packagekit.dart';
@@ -59,7 +60,10 @@ class _DebPageState extends ConsumerState<DebPage> {
           debModel: debModel,
         ),
       ),
-      error: (error, stackTrace) => ErrorWidget(error),
+      error: (error, stackTrace) => ErrorView(
+        error: error,
+        onRetry: () => ref.invalidate(debModelProvider(widget.id)),
+      ),
       loading: () => const Center(child: YaruCircularProgressIndicator()),
     );
   }

packages/app_center/lib/deb/deb_page.dart

@@ -1,5 +1,6 @@
 import 'package:app_center/constants.dart';
 import 'package:app_center/deb/local_deb_model.dart';
+import 'package:app_center/error/error.dart';
 import 'package:app_center/l10n.dart';
 import 'package:app_center/layout.dart';
 import 'package:app_center/widgets/widgets.dart';
@@ -25,7 +26,10 @@ class LocalDebPage extends ConsumerWidget {
     return model.when(
       data: (debData) => _LocalDebPage(debData: debData),
       loading: () => const Center(child: YaruCircularProgressIndicator()),
-      error: (error, stackTrace) => ErrorWidget(error),
+      error: (error, stackTrace) => ErrorView(
+        error: error,
+        onRetry: () => ref.invalidate(localDebModelProvider(path: path)),
+      ),
     );
   }
 }

packages/app_center/lib/deb/local_deb_page.dart

@@ -1 +1,3 @@
+export 'error_l10n.dart';
 export 'error_observer.dart';
+export 'error_view.dart';

packages/app_center/lib/error/error.dart

@@ -0,0 +1,82 @@
+import 'package:app_center/l10n.dart';
+import 'package:snapd/snapd.dart';
+
+enum ErrorAction {
+  retry,
+  checkStatus,
+}
+
+sealed class ErrorMessage {
+  const ErrorMessage();
+
+  factory ErrorMessage.fromObject(Object? e) {
+    if (e is! SnapdException) return ErrorMessageUnkown();
+
+    switch (e.kind) {
+      case 'network-timeout':
+        return ErrorMessageNetwork();
+    }
+    for (final patternMap in _patternMaps) {
+      final match = patternMap.pattern.firstMatch(e.message);
+      if (match != null) {
+        return patternMap.message(match);
+      }
+    }
+    return ErrorMessageUnkown();
+  }
+
+  static final _patternMaps =
+      <({RegExp pattern, ErrorMessage Function(Match) message})>[
+    (
+      pattern: RegExp('too many requests'),
+      message: (_) => ErrorMessageTooManyRequests(),
+    ),
+    (
+      pattern: RegExp(
+        r'cannot refresh "(.*?)": snap "\1" has running apps \((.*?)\)',
+      ),
+      message: (match) => ErrorMessageRunningApps(match.group(1)!),
+    ),
+    (
+      pattern: RegExp('persistent network error'),
+      message: (_) => ErrorMessageNetwork(),
+    ),
+  ];
+
+  String body(AppLocalizations l10n) => switch (this) {
+        ErrorMessageNetwork() => l10n.errorViewNetworkErrorDescription,
+        ErrorMessageTooManyRequests() => l10n.errorViewServerErrorDescription,
+        ErrorMessageRunningApps(snap: final snap) =>
+          l10n.snapdExceptionRunningApps(snap),
+        _ => l10n.errorViewUnknownErrorDescription,
+      };
+
+  String title(AppLocalizations l10n) => switch (this) {
+        ErrorMessageNetwork() => l10n.errorViewNetworkErrorTitle,
+        _ => l10n.errorViewUnknownErrorTitle,
+      };
+
+  String actionLabel(AppLocalizations l10n) => switch (this) {
+        ErrorMessageNetwork() => l10n.errorViewNetworkErrorAction,
+        ErrorMessageTooManyRequests() => l10n.errorViewServerErrorAction,
+        _ => l10n.errorViewUnknownErrorAction,
+      };
+
+  List<ErrorAction> get actions => switch (this) {
+        ErrorMessageNetwork() => [ErrorAction.retry],
+        ErrorMessageTooManyRequests() => [ErrorAction.checkStatus],
+        ErrorMessageRunningApps() => [],
+        _ => [ErrorAction.retry, ErrorAction.checkStatus],
+      };
+}
+
+class ErrorMessageNetwork extends ErrorMessage {}
+
+class ErrorMessageTooManyRequests extends ErrorMessage {}
+
+class ErrorMessageRunningApps extends ErrorMessage {
+  const ErrorMessageRunningApps(this.snap);
+  final String snap;
+}
+
+class ErrorMessageUnkown extends ErrorMessage {}