Moving ecommerce detection logic to new EcommerceRequestProcessor class from Visit::handle().

2 files changed, 43 insertions, 0 deletions

diff --git a/plugins/Ecommerce/Tracker/EcommerceRequestProcessor.php b/plugins/Ecommerce/Tracker/EcommerceRequestProcessor.php
new file mode 100644
index 0000000000..b5f0064728
--- /dev/null
+++ b/plugins/Ecommerce/Tracker/EcommerceRequestProcessor.php
@@ -0,0 +1,34 @@
+<?php
+/**
+ * Piwik - free/libre analytics platform
+ *
+ * @link http://piwik.org
+ * @license http://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later
+ */
+
+namespace Piwik\Plugins\Ecommerce\Tracker;
+
+use Piwik\Tracker\GoalManager;
+use Piwik\Tracker\Request;
+use Piwik\Tracker\RequestProcessor;
+use Piwik\Tracker\Visit\VisitProperties;
+
+/**
+ * TODO
+ */
+class EcommerceRequestProcessor extends RequestProcessor
+{
+    public function processRequestParams(VisitProperties $visitProperties, Request $request)
+    {
+        $goalManager = new GoalManager($request); // TODO: GoalManager should be stateless and stored in DI.
+
+        if ($goalManager->requestIsEcommerce) {
+            $visitProperties->setRequestMetadata('Goals', 'someGoalsConverted', true);
+
+            // Mark the visit as Converted only if it is an order (not for a Cart update)
+            if ($goalManager->isGoalAnOrder()) {
+                $visitProperties->setRequestMetadata('Goals', 'visitIsConverted', true);
+            }
+        }
+    }
+}
\ No newline at end of file
diff --git a/plugins/Ecommerce/config/config.php b/plugins/Ecommerce/config/config.php
new file mode 100644
index 0000000000..3431748e41
--- /dev/null
+++ b/plugins/Ecommerce/config/config.php
@@ -0,0 +1,9 @@
+<?php
+
+return array(
+
+    'tracker.request.processors' => DI\add(array(
+        DI\get('Piwik\Plugins\Ecommerce\Tracker\EcommerceRequestProcessor'),
+    )),
+
+);