Display popup error for OIDC4VP transaction

hawkbee1 · hawkbee1 · commit ffa7e4212192 · 2025-11-10T07:11:42.000Z
diff --git a/lib/app/shared/alert_message/alert_message.dart b/lib/app/shared/alert_message/alert_message.dart
@@ -1,4 +1,5 @@
 import 'package:altme/app/app.dart';
+import 'package:altme/app/shared/alert_message/exception_message.dart';
 import 'package:altme/dashboard/dashboard.dart';
 import 'package:altme/l10n/l10n.dart';
 import 'package:flutter/material.dart';
@@ -102,6 +103,19 @@ class AlertMessage {
       );
     }
   }
+
+  static void displayMessage({
+    required BuildContext context,
+    required ExceptionMessage exception,
+  }) {
+    showDialog<bool>(
+      context: context,
+      builder: (context) => ErrorDialog(
+        title: exception.error,
+        erroDescription: exception.errorDescription,
+      ),
+    );
+  }
 }
 
 class SnackBarContent extends StatelessWidget {
diff --git a/lib/app/shared/alert_message/exception_message.dart b/lib/app/shared/alert_message/exception_message.dart
@@ -0,0 +1,5 @@
+class ExceptionMessage implements Exception {
+  ExceptionMessage({required this.error, required this.errorDescription});
+  final String error;
+  final String errorDescription;
+}
diff --git a/lib/app/shared/alert_message/message_cubit.dart b/lib/app/shared/alert_message/message_cubit.dart
@@ -0,0 +1,10 @@
+import 'package:altme/app/shared/alert_message/exception_message.dart';
+import 'package:bloc/bloc.dart';
+
+class MessageCubit extends Cubit<ExceptionMessage> {
+  MessageCubit() : super(ExceptionMessage(error: '', errorDescription: ''));
+  
+  void error(ExceptionMessage exception) {
+    emit(exception);
+  }
+}
diff --git a/lib/app/shared/helper_functions/helper_functions.dart b/lib/app/shared/helper_functions/helper_functions.dart
@@ -989,6 +989,8 @@ MessageHandler getMessageHandler(dynamic e) {
     final error = NetworkException.getDioException(error: e);
 
     return NetworkException(data: error.data);
+  } else if (e is ResponseMessage) {
+    return e;
   } else if (e is FormatException) {
     return ResponseMessage(
       data: {
diff --git a/lib/app/shared/message_handler/message_handler/message_handler.dart b/lib/app/shared/message_handler/message_handler/message_handler.dart
@@ -1,6 +1,7 @@
 import 'package:flutter/material.dart';
 
 mixin MessageHandler implements Exception {
+  
   String getMessage(
     BuildContext context,
     MessageHandler messageHandler, {
diff --git a/lib/app/shared/message_handler/response_message.dart b/lib/app/shared/message_handler/response_message.dart
@@ -775,3 +775,7 @@ class ResponseMessage with MessageHandler {
     return '';
   }
 }
+
+class ErrorMessage extends ResponseMessage {
+  ErrorMessage({required super.message, super.data});
+}
diff --git a/lib/app/view/app.dart b/lib/app/view/app.dart
@@ -7,6 +7,7 @@
 
 import 'package:altme/activity_log/activity_log.dart';
 import 'package:altme/app/app.dart';
+import 'package:altme/app/shared/alert_message/message_cubit.dart';
 import 'package:altme/chat_room/chat_room.dart';
 import 'package:altme/connection_bridge/connection_bridge.dart';
 import 'package:altme/credentials/credentials.dart';
@@ -292,6 +293,7 @@ class App extends StatelessWidget {
             ),
           ),
           BlocProvider(create: (context) => HomeTabbarCubit()),
+          BlocProvider(create: (context) => MessageCubit()),
         ],
         child: const MaterialAppDefinition(),
       ),
diff --git a/lib/credentials/cubit/credentials_cubit.dart b/lib/credentials/cubit/credentials_cubit.dart
@@ -1045,6 +1045,10 @@ class CredentialsCubit extends Cubit<CredentialsState> {
       await addCryptoProofsPerProfile(cryptoAccountData: cryptoAccountData);
     }
   }
+
+  void emitError(MessageHandler error) {
+    emit(state.error(messageHandler: error));
+  }
 }
 
 List<DiscoverDummyCredential> getDummiesFromExternalIssuerList(
diff --git a/lib/oidc4vp_transaction/widget/attestation_list.dart b/lib/oidc4vp_transaction/widget/attestation_list.dart
@@ -1,6 +1,6 @@
-import 'package:altme/app/shared/issuer/models/issuer.dart';
-import 'package:altme/app/shared/loading/loading_view.dart';
-import 'package:altme/app/shared/message_handler/response_message.dart';
+import 'package:altme/app/app.dart';
+import 'package:altme/app/shared/alert_message/exception_message.dart';
+import 'package:altme/app/shared/alert_message/message_cubit.dart';
 import 'package:altme/credentials/cubit/credentials_cubit.dart';
 import 'package:altme/dashboard/dashboard.dart';
 import 'package:altme/dashboard/home/tab_bar/credentials/models/blockchain/blockchain_credential_subject_model/blockchain_credential_subject_model.dart';
@@ -26,7 +26,10 @@ class AttestationList extends StatelessWidget {
             if (snapshot.connectionState == ConnectionState.waiting) {
               return const Center(child: CircularProgressIndicator());
             } else if (snapshot.hasError) {
-              return Center(child: Text('Error: ${snapshot.error}'));
+              LoadingView().hide();
+              final error = snapshot.error! as ExceptionMessage;
+              context.read<MessageCubit>().error(error);
+              return Center(child: Text(error.error));
             } else {
               return ListView.builder(
                 shrinkWrap: true,
@@ -82,6 +85,12 @@ class AttestationList extends StatelessWidget {
             [],
         profileType: profileModel.profileType,
       );
+      if (credentialManifestPickCubit.state.filteredCredentialList.isEmpty) {
+        throw ExceptionMessage(
+          error: 'Credential not found',
+          errorDescription: 'No suitable credential found in the wallet.',
+        );
+      }
 
       /// for sd-jwt we only support single credentials right now
       /// skip is not considered for sd-jwt right now
@@ -108,8 +117,13 @@ class AttestationList extends StatelessWidget {
                 }
                 return false;
               },
-              orElse: () =>
-                  credentialManifestPickCubit.state.filteredCredentialList[0],
+              orElse: () {
+                throw ExceptionMessage(
+                  error: 'No valid crypto account found',
+                  errorDescription:
+                      'No suitable crypto account found in the wallet.',
+                );
+              },
             );
       } else {
         firstOne = credentialManifestPickCubit.state.filteredCredentialList[0];
@@ -175,11 +189,9 @@ class AttestationList extends StatelessWidget {
 
         credentialsToBePresented.addAll(credToBePresented);
       } else {
-        throw ResponseMessage(
-          data: {
-            'error': 'invalid_request',
-            'error_description': 'Issue with the disclosure encryption of jwt.',
-          },
+        throw ExceptionMessage(
+          error: 'invalid_request',
+          errorDescription: 'Issue with the disclosure encryption of jwt.',
         );
       }
 
diff --git a/lib/splash/bloclisteners/blocklisteners.dart b/lib/splash/bloclisteners/blocklisteners.dart
@@ -2,6 +2,8 @@ import 'dart:async';
 import 'dart:convert';
 
 import 'package:altme/app/app.dart';
+import 'package:altme/app/shared/alert_message/exception_message.dart';
+import 'package:altme/app/shared/alert_message/message_cubit.dart';
 import 'package:altme/connection_bridge/connection_bridge.dart';
 import 'package:altme/credentials/cubit/credentials_cubit.dart';
 import 'package:altme/dashboard/dashboard.dart';
@@ -49,6 +51,12 @@ final splashBlocListener = BlocListener<SplashCubit, SplashState>(
   },
 );
 
+final messageCubitListener = BlocListener<MessageCubit, ExceptionMessage>(
+  listener: (BuildContext context, exceptionMessage) {
+    AlertMessage.displayMessage(context: context, exception: exceptionMessage);
+  },
+);
+
 final ProfileCubitListener = BlocListener<ProfileCubit, ProfileState>(
   listener: (BuildContext context, ProfileState state) {
     if (state.status == AppStatus.addEuropeanProfile) {
diff --git a/lib/splash/view/splash_page.dart b/lib/splash/view/splash_page.dart
@@ -156,6 +156,7 @@ class _SplashViewState extends State<SplashView> {
         walletConnectBlocListener,
         enterpriseBlocListener,
         ProfileCubitListener,
+        messageCubitListener,
       ],
       child: BlocBuilder<ProfileCubit, ProfileState>(
         builder: (context, state) {
diff --git a/test/scan/cubit/scan_cubit_test.mocks.dart b/test/scan/cubit/scan_cubit_test.mocks.dart
@@ -751,6 +751,15 @@ class MockCredentialsCubit extends _i1.Mock implements _i12.CredentialsCubit {
         returnValueForMissingStub: _i16.Future<void>.value(),
       ) as _i16.Future<void>);
 
+  @override
+  void emitError(_i15.MessageHandler? error) => super.noSuchMethod(
+        Invocation.method(
+          #emitError,
+          [error],
+        ),
+        returnValueForMissingStub: null,
+      );
+
   @override
   void emit(_i12.CredentialsState? state) => super.noSuchMethod(
         Invocation.method(

Original file line number	Diff line number	Diff line change
`@@ -775,3 +775,7 @@ class ResponseMessage with MessageHandler {`
`775`	`775`	`return '';`
`776`	`776`	`}`
`777`	`777`	`}`
	`778`	`+`
	`779`	`+class ErrorMessage extends ResponseMessage {`
	`780`	`+ ErrorMessage({required super.message, super.data});`
	`781`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -1045,6 +1045,10 @@ class CredentialsCubit extends Cubit<CredentialsState> {`
`1045`	`1045`	`await addCryptoProofsPerProfile(cryptoAccountData: cryptoAccountData);`
`1046`	`1046`	`}`
`1047`	`1047`	`}`
	`1048`	`+`
	`1049`	`+ void emitError(MessageHandler error) {`
	`1050`	`+ emit(state.error(messageHandler: error));`
	`1051`	`+ }`
`1048`	`1052`	`}`
`1049`	`1053`
`1050`	`1054`	`List<DiscoverDummyCredential> getDummiesFromExternalIssuerList(`