Merge pull request #43 from Countly/sdk-660

[SDK 660] Event segmentation issue fixed!

Author: ArtursKadikis

CHANGELOG.md

@@ -1,4 +1,5 @@
 21.11.1
+* !! Major breaking change !! Fixed a bug that triggered when providing segmentation to the "RecordEvent" call. Previously, by mistake, every segmentation value was parsed as a JSON and threw an exception when it wasn't a valid JSON string. Now this will not be the case and every String value can be provided. This is marked as a "major breaking change" in case some integrations were adding workarounds to this issue.
 * ! Minor breaking change ! Default automatic session update duration changed to 60 seconds.
 * Added a call to change the automatic session update duration.
 * Fixed a bug where session duration was reported wrong.

include/countly.hpp

@@ -182,7 +182,7 @@ class Countly {
 		Event event(key, count);
 
 		for (auto key_value: segmentation) {
-			event.addSegmentation(key_value.first, json::parse(key_value.second));
+			event.addSegmentation(key_value.first, key_value.second);
 		}
 
 		addEvent(event);
@@ -192,7 +192,7 @@ class Countly {
 		Event event(key, count, sum);
 
 		for (auto key_value: segmentation) {
-			event.addSegmentation(key_value.first, json::parse(key_value.second));
+			event.addSegmentation(key_value.first, key_value.second);
 		}
 
 		addEvent(event);
@@ -202,7 +202,7 @@ class Countly {
 		Event event(key, count, sum, duration);
 
 		for (auto key_value: segmentation) {
-			event.addSegmentation(key_value.first, json::parse(key_value.second));
+			event.addSegmentation(key_value.first, key_value.second);
 		}
 
 		addEvent(event);