MAGETWO-86143: Disable statistic collecting for Reports module

Author: mslabko

app/code/Magento/Customer/Model/Visitor.php

@@ -40,8 +40,6 @@ class Visitor extends \Magento\Framework\Model\AbstractModel
     protected $httpHeader;
 
     /**
-     * Skip logging during request. Do not use this property directly, but use self::isSkipRequestLogging instead
-     *
      * @var bool
      */
     protected $skipRequestLogging = false;
@@ -145,19 +143,6 @@ public function setSkipRequestLogging($skipRequestLogging)
         return $this;
     }
 
-    /**
-     * Check if we can log request fot event. Additionally prevent logging for safe methods (e.g. GET request)
-     *
-     * @return bool
-     */
-    private function isSkipRequestLogging()
-    {
-        if ($this->requestSafety->isSafeMethod()) {
-            return true;
-        }
-        return $this->skipRequestLogging;
-    }
-
     /**
      * Initialization visitor by request
      *
@@ -168,7 +153,7 @@ private function isSkipRequestLogging()
      */
     public function initByRequest($observer)
     {
-        if ($this->isSkipRequestLogging() || $this->isModuleIgnored($observer)) {
+        if ($this->skipRequestLogging || $this->isModuleIgnored($observer)) {
             return $this;
         }
 
@@ -181,6 +166,10 @@ public function initByRequest($observer)
 
         $this->setLastVisitAt((new \DateTime())->format(\Magento\Framework\Stdlib\DateTime::DATETIME_PHP_FORMAT));
 
+        // prevent saving Visitor for safe methods, e.g. GET request
+        if ($this->requestSafety->isSafeMethod()) {
+            return $this;
+        }
         if (!$this->getId()) {
             $this->setSessionId($this->session->getSessionId());
             $this->save();
@@ -200,7 +189,8 @@ public function initByRequest($observer)
      */
     public function saveByRequest($observer)
     {
-        if ($this->isSkipRequestLogging() || $this->isModuleIgnored($observer)) {
+        // prevent saving Visitor for safe methods, e.g. GET request
+        if ($this->skipRequestLogging || $this->requestSafety->isSafeMethod() || $this->isModuleIgnored($observer)) {
             return $this;
         }