fix: requests with just queryString could not be handled

Author: tinect

composer.json

@@ -1,6 +1,6 @@
 {
     "name": "tinect/matomo",
-    "version": "5.0.1",
+    "version": "5.0.2",
     "description": "Matomo plugin for shopware 6",
     "type": "shopware-platform-plugin",
     "license": "MIT",

src/MessageQueue/TrackHandler.php

@@ -51,18 +51,19 @@ public function __invoke(TrackMessage $message): void
             // Bulk tracking payload: enrich each request; do NOT also put params on URL
             try {
                 $parameter = $this->enrichRequests($parameter, $commonParameters);
-            } catch (\JsonException $exception) {
-                $this->logger->error('Error enriching request.', [
-                    'parameter' => $parameter,
-                    'commonParameters' => $commonParameters,
-                    'message' => $exception->getMessage(),
-                ]);
+            } catch (\JsonException) {
+                parse_str($parameter, $parsedQuery);
+                if ($parsedQuery !== []) {
+                    $parameter = $parsedQuery;
+                }
             }
 
             $data = [
                 'body' => $parameter,
             ];
-        } else {
+        }
+
+        if (\is_array($parameter)) {
             $merged = array_merge($parameter, $commonParameters);
             // Remove empty-string values to avoid invalid parameters like _id=""
             $merged = array_filter($merged, static function ($v) {
@@ -74,6 +75,14 @@ public function __invoke(TrackMessage $message): void
             ];
         }
 
+        if (!isset($data)) {
+            $this->logger->error('Parameters for Matomo tracking could not be handled.', [
+                'parameters' => $message->parameters,
+            ]);
+
+            throw new \RuntimeException('Parameters for Matomo tracking could not be handled.');
+        }
+
         $client = new Client();
 
         // Build headers and set JSON content type when sending bulk payload