feat: make event interface return void

Author: arifBurakDemiray

lib/src/countly_flutter.dart

@@ -411,7 +411,8 @@ class Countly {
       return message;
     }
 
-    return _instance.events.recordEvent(key, segmentation, count, sum, duration);
+    await _instance.events.recordEvent(key, segmentation, count, sum, duration);
+    return 'recordEvent called';
   }
 
   static T _parseValue<T>(Map<String, Object> map, String key, T fallback) {
@@ -1680,7 +1681,8 @@ class Countly {
   /// returns error or success message
   @Deprecated('This function is deprecated, please use "startEvent" of events instead')
   static Future<String?> startEvent(String key) async {
-    return _instance.events.startEvent(key);
+    await _instance.events.startEvent(key);
+    return 'startEvent called';
   }
 
   /// ends a timed event
@@ -1698,7 +1700,8 @@ class Countly {
       return message;
     }
 
-    return _instance.events.endEvent(key, segmentation, count, sum);
+    await _instance.events.endEvent(key, segmentation, count, sum);
+    return 'endEvent called';
   }
 
   /// Call used for testing error handling
@@ -2211,12 +2214,13 @@ class Countly {
       }
 
       /// Experimental END ---------------------------
-      
+
       /// Content ---------------------------
       if (config.content.zoneTimerInterval != null) {
         log('"_configToJson", value provided for zoneTimerInterval: [${config.content.zoneTimerInterval}]', logLevel: LogLevel.INFO);
         countlyConfig['zoneTimerInterval'] = config.content.zoneTimerInterval;
       }
+
       /// Content END ---------------------------
 
       /// APM ---------------------------

lib/src/events.dart

@@ -6,31 +6,23 @@ abstract class Events {
   /// [count]: Count to associate with the event, should be more than zero. (optional)
   /// [sum]: Sum to associate with the event. (optional)
   /// [duration]: Duration of the event. (optional)
-  ///
-  /// Returns a future that resolves to the result of the operation whether success or not.
-  Future<String?> recordEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum, int? duration]);
+  Future<void> recordEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum, int? duration]);
 
   /// Start timed event with a specified key
   ///
   /// [key]: Name of the custom event, required, must not be an empty string.
-  ///
-  /// Returns a future that resolves to the result of the operation whether success or not.
-  Future<String?> startEvent(String key);
+  Future<void> startEvent(String key);
 
   /// Ends a timed event with a specified key.
   ///
   /// [key]: Name of the custom event, required, must not be an empty string.
   /// [segmentation]: Segmentation map to associate with the event, can be null. (optional)
   /// [count]: Count to associate with the event, should be more than zero. Default value is 1. (optional)
   /// [sum]: Sum to associate with the event. Default value is 0. (optional)
-  ///
-  /// Returns a future that resolves to the result of the operation whether success or not.
-  Future<String?> endEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum]);
+  Future<void> endEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum]);
 
   /// Cancel timed event with a specified key
   ///
   /// [key]: Name of the custom event, required, must not be an empty string.
-  ///
-  /// Returns a future that resolves to the result of the operation whether success or not.
-  Future<String?> cancelEvent(String key);
+  Future<void> cancelEvent(String key);
 }

lib/src/events_internal.dart

@@ -10,40 +10,36 @@ class EventsInternal implements Events {
   final CountlyState _countlyState;
 
   @override
-  Future<String?> recordEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum, int? duration]) async {
-    return _internalEventMethodCall(key, 'recordEvent', segmentation, count ?? 1, sum ?? 0, duration ?? 0);
+  Future<void> recordEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum, int? duration]) async {
+    await _internalEventMethodCall(key, 'recordEvent', segmentation, count ?? 1, sum ?? 0, duration ?? 0);
   }
 
   @override
-  Future<String?> startEvent(String key) async {
-    return _internalEventMethodCall(key, 'startEvent');
+  Future<void> startEvent(String key) async {
+    await _internalEventMethodCall(key, 'startEvent');
   }
 
   @override
-  Future<String?> endEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum]) async {
-    return _internalEventMethodCall(key, 'endEvent', segmentation, count ?? 1, sum ?? 0);
+  Future<void> endEvent(String key, [Map<String, Object>? segmentation, int? count, double? sum]) async {
+    await _internalEventMethodCall(key, 'endEvent', segmentation, count ?? 1, sum ?? 0);
   }
 
   @override
-  Future<String?> cancelEvent(String key) async {
-    return _internalEventMethodCall(key, 'cancelEvent');
+  Future<void> cancelEvent(String key) async {
+    await _internalEventMethodCall(key, 'cancelEvent');
   }
 
-  Future<String?> _internalEventMethodCall(String key, String method, [Map<String, Object>? segmentation, int? count, double? sum, int? duration]) async {
+  Future<void> _internalEventMethodCall(String key, String method, [Map<String, Object>? segmentation, int? count, double? sum, int? duration]) async {
     List<Object?> args = [];
 
     if (!_countlyState.isInitialized) {
-      String message = '"initWithConfig" must be called before "$method"';
-      Countly.log('[EventsInternal] $method, $message', logLevel: LogLevel.ERROR);
-      return message;
+      Countly.log('[EventsInternal] $method, "initWithConfig" must be called', logLevel: LogLevel.ERROR);
     }
 
     Countly.log('[EventsInternal] $method, key:[$key] segmentation:[$segmentation] count:[$count] sum:[$sum] duration:[$duration]');
 
     if (key.isEmpty) {
-      String error = '$method, key name is required';
-      Countly.log('[EventsInternal] $method, $error');
-      return 'Error : $error';
+      Countly.log('[EventsInternal] $method, key name is required');
     }
 
     args.add(key);
@@ -53,7 +49,6 @@ class EventsInternal implements Events {
     args.add(segmentation);
 
     final String? result = await _countlyState.channel.invokeMethod(method, <String, dynamic>{'data': json.encode(args.where((item) => item != null).toList())});
-
-    return result;
+    Countly.log('[EventsInternal] $method, result:[$result]');
   }
 }