Merge pull request #53 from elgentos/WouterSteen-patch-5

Loop over $data items instead of hardcoded param set
elgentos · Mar 25, 2024 · 647410d · 647410d
2 parents 61ab6dc + e34551b
commit 647410d
Showing 1 changed file with 3 additions and 15 deletions.
diff --git a/Model/GAClient.php b/Model/GAClient.php
@@ -140,23 +140,11 @@ public function setTrackingData(\Magento\Framework\DataObject $data)
      */
     public function setTransactionData($data)
     {
-        $this->getPurchaseEvent()
-            ->setTransactionId($data->getTransactionId())
-            ->setCurrency($data->getCurrency())
-            ->setValue($data->getRevenue())
-            ->setTax($data->getTax())
-            ->setShipping($data->getShipping());
-
-        $this->getPurchaseEvent()->setParamValue('session_id', $data->getSessionId());
-        $this->getPurchaseEvent()->setParamValue('timestamp_micros', $this->getMicroTime());
-
-        if ($data->getAffiliation()) {
-            $this->getPurchaseEvent()->setAffiliation($data->getAffiliation());
+        foreach ($data->getData() as $key => $param) {
+            $this->getPurchaseEvent()->setParamValue($key, $param);
         }
 
-        if ($data->getCouponCode()) {
-            $this->getPurchaseEvent()->setCouponCode($data->getCouponCode());
-        }
+        $this->getPurchaseEvent()->setParamValue('timestamp_micros', $this->getMicroTime());
     }
 
     /**