feat: Simplified ImpaktfullAnalyticsUtil and ImpaktfullCrashReporting…

…Util
impaktfull · Feb 20, 2025 · 9812a0d · 9812a0d
1 parent 9a04473
commit 9812a0d
Show file tree

Hide file tree

Showing 5 changed files with 44 additions and 11 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,12 @@
 
 ## BREAKING
 
+- Simplified ImpaktfullAnalyticsUtil and ImpaktfullCrashReportingUtil
+
+# 0.14.0
+
+## BREAKING
+
 - Remove `enabled` from the init function of ImpaktfullAnalyticsUtil and ImpaktfullCrashReportingUtil
 
 # 0.13.0

diff --git a/lib/src/util/analytics/analytics_util.dart b/lib/src/util/analytics/analytics_util.dart
@@ -3,11 +3,14 @@ abstract class ImpaktfullAnalyticsUtil {
 
   Future<void> setEnabled(bool enabled);
 
-  void logEvent(String eventName, {Map<String, dynamic>? data});
+  void setUserId(String userId);
 
-  void setUserProperty(String property, String value);
+  void setUserProperty({
+    required String property,
+    required String value,
+  });
 
-  void setUserId(String userId);
+  void logEvent(String eventName, {Map<String, dynamic>? data});
 
   void setScreenName(String screenName);
 }
diff --git a/lib/src/util/analytics/multi_instance_analytics_util.dart b/lib/src/util/analytics/multi_instance_analytics_util.dart
@@ -29,9 +29,15 @@ class MultiInstanceAnalyticsUtil extends ImpaktfullAnalyticsUtil {
   }
 
   @override
-  void setUserProperty(String property, String value) {
+  void setUserProperty({
+    required String property,
+    required String value,
+  }) {
     for (var util in analyticsUtils) {
-      util.setUserProperty(property, value);
+      util.setUserProperty(
+        property: property,
+        value: value,
+      );
     }
   }
 

diff --git a/lib/src/util/crash_reporting/crash_reporting_util.dart b/lib/src/util/crash_reporting/crash_reporting_util.dart
@@ -5,7 +5,10 @@ abstract class ImpaktfullCrashReportingUtil {
 
   void setUserId(String userId);
 
-  void setUserProperty(String property, String value);
+  void setUserProperty({
+    required String property,
+    required String value,
+  });
 
   void log(String message, {Map<String, dynamic>? data});
 
@@ -15,5 +18,8 @@ abstract class ImpaktfullCrashReportingUtil {
     String? message,
   });
 
-  void logUnhandledError(Object error, StackTrace stackTrace);
+  void logUnhandledError({
+    required Object error,
+    required StackTrace stackTrace,
+  });
 }
diff --git a/lib/src/util/crash_reporting/multi_instance_crashreporting_util.dart b/lib/src/util/crash_reporting/multi_instance_crashreporting_util.dart
@@ -33,9 +33,15 @@ class MultiInstanceCrashReportingUtil extends ImpaktfullCrashReportingUtil {
   }
 
   @override
-  void setUserProperty(String property, String value) {
+  void setUserProperty({
+    required String property,
+    required String value,
+  }) {
     for (final util in crashReportingUtils) {
-      util.setUserProperty(property, value);
+      util.setUserProperty(
+        property: property,
+        value: value,
+      );
     }
   }
 
@@ -58,9 +64,15 @@ class MultiInstanceCrashReportingUtil extends ImpaktfullCrashReportingUtil {
   }
 
   @override
-  void logUnhandledError(Object error, StackTrace stackTrace) {
+  void logUnhandledError({
+    required Object error,
+    required StackTrace stackTrace,
+  }) {
     for (final util in crashReportingUtils) {
-      util.logUnhandledError(error, stackTrace);
+      util.logUnhandledError(
+        error: error,
+        stackTrace: stackTrace,
+      );
     }
   }
 }