fix: make sure requests contain at least 1 event

Author: aawnu

src/Analytics.php

@@ -124,25 +124,23 @@ public function post(): void
         $url = $this->debug ? Facade\Type\AnalyticsType::URL_DEBUG : Facade\Type\AnalyticsType::URL_LIVE;
         $url .= '?' . http_build_query(['measurement_id' => $this->measurement_id, 'api_secret' => $this->api_secret]);
 
-        $body = $this->toArray();
-        array_merge_recursive(
+        $body = array_replace_recursive(
             $this->toArray(),
+            ["user_properties" => $this->user_properties],
             ["consent" => $this->consent->toArray()],
         );
 
         $chunkEvents = array_chunk($this->events, 25);
+
+        if (count($chunkEvents) < 1) {
+            throw Ga4Exception::throwMissingEvents();
+        }
+
+        $this->user_properties = [];
         $this->events = [];
 
         foreach ($chunkEvents as $events) {
-            $body['user_properties'] = $this->user_properties;
-            if (empty($body['user_properties'])) {
-                unset($body['user_properties']);
-            }
-
             $body['events'] = $events;
-            if (empty($body['events'])) {
-                unset($body['events']);
-            }
 
             $kB = 1024;
             if (($size = mb_strlen(json_encode($body))) > ($kB * 130)) {

src/Exception/Ga4Exception.php

@@ -81,4 +81,9 @@ public static function throwRequestInvalidBody(array $msg)
             static::REQUEST_INVALID_BODY
         );
     }
+
+    public static function throwMissingEvents()
+    {
+        return new static("Request must include at least 1 event with a name", static::REQUEST_EMPTY_EVENTLIST);
+    }
 }

src/Facade/Type/Ga4ExceptionType.php

@@ -24,4 +24,5 @@ interface Ga4ExceptionType
     const REQUEST_INVALID_BODY = 104005;
     const REQUEST_MISSING_MEASUREMENT_ID = 104006;
     const REQUEST_MISSING_API_SECRET = 104007;
+    const REQUEST_EMPTY_EVENTLIST = 104008;
 }