fixed: error, stacktrace, data, hint, & message

webreinvent · May 27, 2024 · c506c50 · c506c50
1 parent c997d36
commit c506c50
Show file tree

Hide file tree

Showing 10 changed files with 171 additions and 98 deletions.
diff --git a/lib/vaahextendflutter/env.dart b/lib/vaahextendflutter/env.dart
@@ -63,7 +63,11 @@ class EnvController extends GetxController {
     try {
       _config = getSpecificConfig(environment).copyWith(openCount: _storage.read('open_count'));
     } catch (error, stackTrace) {
-      Log.exception(error, stackTrace: stackTrace);
+      Log.exception(
+        "Error occured while initializing EnvController",
+        throwable: error,
+        stackTrace: stackTrace,
+      );
       exit(0);
     }
   }

diff --git a/lib/vaahextendflutter/services/api.dart b/lib/vaahextendflutter/services/api.dart
@@ -348,7 +348,11 @@ abstract class Api {
     bool showAlert,
     String alertType,
   ) async {
-    Log.exception(error, stackTrace: error.stackTrace);
+    Log.exception(
+      "Handle Timeout Error",
+      throwable: error,
+      stackTrace: error.stackTrace,
+    );
     if (showAlert) {
       if (alertType == 'dialog') {
         if (Alerts.showErrorDialog != null) {
@@ -416,7 +420,12 @@ abstract class Api {
 
       if (error.response?.data != null) {
         try {
-          Log.exception(catchErr, data: error.response, stackTrace: stackTrace);
+          Log.exception(
+            "Handle Response Error",
+            throwable: catchErr,
+            hint: error.response,
+            stackTrace: stackTrace,
+          );
 
           final Map<String, dynamic> response = error.response?.data as Map<String, dynamic>;
           if (response['errors'] != null) {

diff --git a/lib/vaahextendflutter/services/dynamic_links.dart b/lib/vaahextendflutter/services/dynamic_links.dart
@@ -46,7 +46,7 @@ abstract class DynamicLinks {
         shortLinkType: ShortDynamicLinkType.unguessable,
       );
     } catch (error, stackTrace) {
-      Log.exception(error, stackTrace: stackTrace, hint: "Error creating dynamic link!");
+      Log.exception("Error creating dynamic link!", throwable: error, stackTrace: stackTrace);
       return null;
     }
   }
@@ -61,10 +61,13 @@ abstract class DynamicLinks {
           decoded: "${linkData.link.host}${linkData.link.path}?payload=$payload",
         ),
       );
-      Log.success({
-        "encoded": linkData.link.toString(),
-        "decoded": "${linkData.link.host}${linkData.link.path}?payload=$payload",
-      });
+      Log.success(
+        "Handle Deeplink",
+        data: {
+          "encoded": linkData.link.toString(),
+          "decoded": "${linkData.link.host}${linkData.link.path}?payload=$payload",
+        },
+      );
       if (payload != null && payload['path'] != null) {
         Get.to(
           payload['path'],
@@ -76,9 +79,9 @@ abstract class DynamicLinks {
       }
     } catch (error, stackTrace) {
       Log.exception(
-        error,
+        "Error handling dynamic link! ${linkData.asMap()}",
+        throwable: error,
         stackTrace: stackTrace,
-        hint: "Error handling dynamic link! ${linkData.asMap()}",
       );
     }
   }
@@ -88,9 +91,9 @@ abstract class DynamicLinks {
       return jsonDecode(link.queryParameters['payload'].toString());
     } catch (error, stackTrace) {
       Log.exception(
-        error,
+        "Error decoding payload! $link",
+        throwable: error,
         stackTrace: stackTrace,
-        hint: "Error decoding payload! $link",
       );
       return null;
     }

diff --git a/lib/vaahextendflutter/services/logging_library/_cloud/firebase_logging_service.dart b/lib/vaahextendflutter/services/logging_library/_cloud/firebase_logging_service.dart
@@ -2,16 +2,17 @@ import '../models/log.dart';
 import 'logging_service.dart';
 
 abstract class FirebaseLoggingService implements LoggingService {
-  static logEvent({
-    required String message,
-    EventType? type,
+  static logEvent(
+    String message, {
     Object? data,
+    EventType? type,
   }) =>
       throw UnimplementedError();
 
   static logException(
-    dynamic throwable, {
-    dynamic stackTrace,
+    String message, {
+    Object? throwable,
+    StackTrace? stackTrace,
     dynamic hint,
   }) =>
       throw UnimplementedError();

diff --git a/lib/vaahextendflutter/services/logging_library/_cloud/sentry_logging_service.dart b/lib/vaahextendflutter/services/logging_library/_cloud/sentry_logging_service.dart
@@ -4,10 +4,10 @@ import '../models/log.dart';
 import 'logging_service.dart';
 
 abstract class SentryLoggingService implements LoggingService {
-  static logEvent({
-    required String message,
-    SentryLevel? level,
+  static logEvent(
+    String message, {
     Object? data,
+    SentryLevel? level,
   }) {
     final SentryEvent event = SentryEvent(message: SentryMessage(message), level: level);
     Sentry.captureEvent(
@@ -17,11 +17,19 @@ abstract class SentryLoggingService implements LoggingService {
   }
 
   static logException(
-    dynamic throwable, {
-    dynamic stackTrace,
+    String message, {
+    Object? throwable,
+    StackTrace? stackTrace,
     dynamic hint,
   }) {
-    Sentry.captureException(throwable, stackTrace: stackTrace, hint: hint);
+    Sentry.captureException(
+      throwable,
+      stackTrace: stackTrace,
+      hint: Hint.withMap({
+        "message": message,
+        "hint": hint,
+      }),
+    );
   }
 
   static logTransaction({

diff --git a/lib/vaahextendflutter/services/logging_library/_local/console_service.dart b/lib/vaahextendflutter/services/logging_library/_local/console_service.dart
@@ -32,61 +32,101 @@ class Console {
     }
   }
 
-  static void log(String text, [Object? data]) {
-    _printLog(text);
+  static void log(
+    String message, {
+    Object? data,
+  }) {
+    _printLog(message);
 
     if (data != null) {
-      _printLog(_parseData(data), {
-        Styles.WHITE,
-      });
+      _printLog(
+        _parseData(data),
+        {Styles.WHITE},
+      );
     }
   }
 
-  static void info(String text, [Object? data]) {
-    _printLog(text, {
-      Styles.BLUE,
-    });
+  static void info(
+    String message, {
+    Object? data,
+  }) {
+    _printLog(
+      message,
+      {Styles.BLUE},
+    );
 
     if (data != null) {
-      _printLog(_parseData(data), {
-        Styles.BLUE,
-      });
+      _printLog(
+        _parseData(data),
+        {Styles.BLUE},
+      );
     }
   }
 
-  static void success(String text, [Object? data]) {
-    _printLog(text, {
-      Styles.GREEN,
-    });
+  static void success(
+    String message, {
+    Object? data,
+  }) {
+    _printLog(
+      message,
+      {Styles.GREEN},
+    );
 
     if (data != null) {
-      _printLog(_parseData(data), {
-        Styles.GREEN,
-      });
+      _printLog(
+        _parseData(data),
+        {Styles.GREEN},
+      );
     }
   }
 
-  static void warning(String text, [Object? data]) {
-    _printLog(text, {
-      Styles.YELLOW,
-    });
+  static void warning(
+    String message, {
+    Object? data,
+  }) {
+    _printLog(
+      message,
+      {Styles.YELLOW},
+    );
 
     if (data != null) {
-      _printLog(_parseData(data), {
-        Styles.YELLOW,
-      });
+      _printLog(
+        _parseData(data),
+        {Styles.YELLOW},
+      );
     }
   }
 
-  static void danger(String text, [Object? data]) {
-    _printLog(text, {
-      Styles.RED,
-    });
+  static void danger(
+    String message, {
+    Object? throwable,
+    StackTrace? stackTrace,
+    dynamic hint,
+  }) {
+    _printLog(
+      message,
+      {Styles.RED},
+    );
+
+    if (throwable != null) {
+      _printLog(
+        _parseData(throwable),
+        {Styles.RED},
+      );
+    }
 
-    if (data != null) {
-      _printLog(_parseData(data), {
-        Styles.RED,
-      });
+    if (stackTrace != null) {
+      _printLog(
+        stackTrace.toString(),
+        {Styles.RED},
+      );
+    }
+
+    if (hint != null) {
+      _printLog(
+        _parseData(hint),
+        {Styles.RED},
+      );
     }
   }