Refs #4909 Only update the visitEcommerceStatus flag if it's not NONE.

This will prevent some occurrences of this bug, but not the case where visitEcommerceStatus == abandonedCart
matomo-org · Mar 26, 2014 · 5809c35 · 5809c35
1 parent 1c70bff
commit 5809c35
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/core/Tracker/Visit.php b/core/Tracker/Visit.php
@@ -942,7 +942,11 @@ protected function getExistingVisitFieldsToUpdate($action, $visitIsConverted)
         }
 
         // Ecommerce buyer status
-        $valuesToUpdate['visit_goal_buyer'] = $this->goalManager->getBuyerType($this->visitorInfo['visit_goal_buyer']);
+        $visitEcommerceStatus = $this->goalManager->getBuyerType($this->visitorInfo['visit_goal_buyer']);
+
+        if($visitEcommerceStatus != GoalManager::TYPE_BUYER_NONE) {
+            $valuesToUpdate['visit_goal_buyer'] = $visitEcommerceStatus;
+        }
 
         // Custom Variables overwrite previous values on each page view
         $valuesToUpdate = array_merge($valuesToUpdate, $this->visitorCustomVariables);