Merge pull request #56 from Smile-SA/release_1.3

Virtual categories, prevent processing empty values if no product ids
Smile-SA · Mar 2, 2016 · b2a2f0a · b2a2f0a
2 parents c133809 + 5a493d0
commit b2a2f0a
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/app/code/community/Smile/VirtualCategories/Model/Observer.php b/src/app/code/community/Smile/VirtualCategories/Model/Observer.php
@@ -113,7 +113,7 @@ protected function _getVirtualRule($category)
      *
      * @event catalogsearch_query_save_after
      *
-     * @return void Nothing
+     * @return Smile_VirtualCategories_Model_Observer
      */
     public function saveProductsPositions(Varien_Event_Observer $observer)
     {
@@ -132,6 +132,10 @@ public function saveProductsPositions(Varien_Event_Observer $observer)
 
         $productIdsToReindex = array_unique(array_merge($previousProducts, array_keys($filteredPositions)));
 
+        if (empty($productIdsToReindex)) {
+            return $this;
+        }
+
         // If Enterprise version, Mview index will handle editing, otherwise, process reindex
         if (!Mage::helper("smile_elasticsearch")->isEnterpriseSupportEnabled()) {
 
@@ -156,6 +160,7 @@ public function saveProductsPositions(Varien_Event_Observer $observer)
 
             }
         }
+        return $this;
     }
 
     /**