Reduce needless saves by avoiding setting _hasDataChanges flag (#2066)

Author: colinmollenhour

README.md

@@ -86,6 +86,7 @@ Do not use 20.x.x if you need IE support.
 - fixed incorrect datetime in customer block (`$useTimezone` parameter) #1525
 - add redis as a valid option for `global/session_save` #1513
 - possibility to disable global search in backend #1532
+- reduce needless saves by avoiding setting `_hasDataChanges` flag #2066
 
 For full list of changes, you can [compare tags](https://github.com/OpenMage/magento-lts/compare/1.9.4.x...20.0).
 

app/code/core/Mage/Catalog/Model/Product.php

@@ -223,9 +223,6 @@
  * @method $this setStoreId(int $store)
  * @method bool hasStoreIds()
  * @method $this setStoreIds(array $storeIds)
- * @method Mage_CatalogInventory_Model_Stock_Item getStockItem()
- * @method bool hasStockItem()
- * @method $this setStockItem(Mage_CatalogInventory_Model_Stock_Item $value)
  * @method array getSwatchPrices()
  *
  * @method int getTaxClassId()
@@ -344,6 +341,11 @@ class Mage_Catalog_Model_Product extends Mage_Catalog_Model_Abstract
      */
     protected $_calculatePrice = true;
 
+    /**
+     * @var Mage_CatalogInventory_Model_Stock_Item
+     */
+    protected $_stockItem;
+
     /**
      * Initialize resources
      */
@@ -694,6 +696,32 @@ public function getAttributes($groupId = null, $skipSuper = false)
         return $attributes;
     }
 
+    /**
+     * @return Mage_CatalogInventory_Model_Stock_Item
+     */
+    public function getStockItem()
+    {
+        return $this->_stockItem;
+    }
+
+    /**
+     * @return bool
+     */
+    public function hasStockItem()
+    {
+        return !!$this->_stockItem;
+    }
+
+    /**
+     * @param Mage_CatalogInventory_Model_Stock_Item $stockItem
+     * @return $this
+     */
+    public function setStockItem(Mage_CatalogInventory_Model_Stock_Item $stockItem)
+    {
+        $this->_stockItem = $stockItem;
+        return $this;
+    }
+
     /**
      * Check product options and type options and save them, too
      *

app/code/core/Mage/Customer/Model/Address.php

@@ -97,7 +97,9 @@ public function getCustomer()
     public function setCustomer(Mage_Customer_Model_Customer $customer)
     {
         $this->_customer = $customer;
-        $this->setCustomerId($customer->getId());
+        if ($this->getCustomerId() != $customer->getId()) {
+            $this->setCustomerId($customer->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Order/Creditmemo/Item.php

@@ -176,7 +176,9 @@ public function getCreditmemo()
     public function setOrderItem(Mage_Sales_Model_Order_Item $item)
     {
         $this->_orderItem = $item;
-        $this->setOrderItemId($item->getId());
+        if ($this->getOrderItemId() != $item->getId()) {
+            $this->setOrderItemId($item->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Order/Invoice/Item.php

@@ -159,7 +159,9 @@ public function getInvoice()
     public function setOrderItem(Mage_Sales_Model_Order_Item $item)
     {
         $this->_orderItem = $item;
-        $this->setOrderItemId($item->getId());
+        if ($this->getOrderItemId() != $item->getId()) {
+            $this->setOrderItemId($item->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Order/Item.php

@@ -440,7 +440,9 @@ public function getQtyToCancelBundleItem()
     public function setOrder(Mage_Sales_Model_Order $order)
     {
         $this->_order = $order;
-        $this->setOrderId($order->getId());
+        if ($this->getOrderId() != $order->getId()) {
+            $this->setOrderId($order->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Order/Shipment/Item.php

@@ -104,7 +104,9 @@ public function getShipment()
     public function setOrderItem(Mage_Sales_Model_Order_Item $item)
     {
         $this->_orderItem = $item;
-        $this->setOrderItemId($item->getId());
+        if ($this->getOrderItemId() != $item->getId()) {
+            $this->setOrderItemId($item->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Quote.php

@@ -297,7 +297,9 @@ public function getStore()
      */
     public function setStore(Mage_Core_Model_Store $store)
     {
-        $this->setStoreId($store->getId());
+        if ($this->getStoreId() != $store->getId()) {
+            $this->setStoreId($store->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Quote/Address.php

@@ -427,7 +427,9 @@ protected function _afterSave()
     public function setQuote(Mage_Sales_Model_Quote $quote)
     {
         $this->_quote = $quote;
-        $this->setQuoteId($quote->getId());
+        if ($this->getQuoteId() != $quote->getId()) {
+            $this->setQuoteId($quote->getId());
+        }
         return $this;
     }
 

app/code/core/Mage/Sales/Model/Quote/Item.php

@@ -275,7 +275,9 @@ protected function _beforeSave()
     public function setQuote(Mage_Sales_Model_Quote $quote)
     {
         $this->_quote = $quote;
-        $this->setQuoteId($quote->getId());
+        if ($this->getQuoteId() != $quote->getId()) {
+            $this->setQuoteId($quote->getId());
+        }
         return $this;
     }