algolia · maxiloc · Feb 10, 2016 · Jan 8, 2016 · Jan 8, 2016 · Jan 14, 2016
diff --git a/Algolia_Algoliasearch.xml b/Algolia_Algoliasearch.xml
@@ -4,7 +4,7 @@
         <Algolia_Algoliasearch>
             <active>true</active>
             <codePool>community</codePool>
-            <version>1.5.2</version>
+            <version>1.5.3</version>
         </Algolia_Algoliasearch>
     </modules>
 </config>
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,15 @@
 ## Change Log
 
+### 1.5.3
+
+- UPDATED: added a config to disable logging
+- UPDATED: better instant search UI
+- FIX: various design improvements
+- FIX: take into account "Include in Navigation" for categories
+- FIX: sorting in instant search page
+- FIX: wrong price for configurable products
+- FIX: mass action delete
+
 ### 1.5.2
 
 ==== BREAKING CHANGES ====

diff --git a/README.md b/README.md
@@ -7,7 +7,7 @@ This extension replaces the default search of Magento with a typo-tolerant, fast
 
 See features and benefits of [Algolia Search Extension for Magento](https://community.algolia.com/magento).
 
-![Latest version](https://img.shields.io/badge/latest-1.5.2-green.svg)
+![Latest version](https://img.shields.io/badge/latest-1.5.3-green.svg)
 ![Magento 1.6.2](https://img.shields.io/badge/magento-1.6.2-blue.svg)
 ![Magento 1.7.1](https://img.shields.io/badge/magento-1.7.1-blue.svg)
 ![Magento 1.8.1](https://img.shields.io/badge/magento-1.8.1-blue.svg)

diff --git a/code/Helper/Algoliahelper.php b/code/Helper/Algoliahelper.php
@@ -12,7 +12,7 @@
 class Algolia_Algoliasearch_Helper_Algoliahelper extends Mage_Core_Helper_Abstract
 {
     protected $client;
-    private $config;
+    protected $config;
 
     public function __construct()
     {

diff --git a/code/Helper/Config.php b/code/Helper/Config.php
@@ -56,7 +56,7 @@ class Algolia_Algoliasearch_Helper_Config extends Mage_Core_Helper_Abstract
     const AUTOCOMPLETE_SELECTOR                = 'algoliasearch/advanced/autocomplete_selector';
 
     const SHOW_OUT_OF_STOCK                    = 'cataloginventory/options/show_out_of_stock';
-    const LOGGING_ENABLED                      = 'dev/log/active';
+    const LOGGING_ENABLED                      = 'algoliasearch/credentials/debug';
 
     protected $_productTypeMap = array();
 

diff --git a/code/Helper/Data.php b/code/Helper/Data.php
@@ -25,7 +25,7 @@ class Algolia_Algoliasearch_Helper_Data extends Mage_Core_Helper_Abstract
 
     public function __construct()
     {
-        \AlgoliaSearch\Version::$custom_value = " Magento (1.5.2)";
+        \AlgoliaSearch\Version::$custom_value = " Magento (1.5.3)";
 
         $this->algolia_helper               = Mage::helper('algoliasearch/algoliahelper');
 
@@ -427,7 +427,7 @@ public function rebuildStoreCategoryIndexPage($storeId, $collectionDefault, $pag
             $this->stopEmulation($emulationInfoPage);
     }
 
-    private function getProductsRecords($storeId, $collection)
+    protected function getProductsRecords($storeId, $collection)
     {
         $indexData = array();
 

diff --git a/code/Helper/Entity/Categoryhelper.php b/code/Helper/Entity/Categoryhelper.php
@@ -57,8 +57,6 @@ public function getCategoryCollectionQuery($storeId, $categoryIds = null)
     {
         $storeRootCategoryPath = sprintf('%d/%d', $this->getRootCategoryId(), Mage::app()->getStore($storeId)->getRootCategoryId());
 
-        $index_name = $this->getIndexName($storeId);
-
         $categories = Mage::getResourceModel('catalog/category_collection'); /** @var $categories Mage_Catalog_Model_Resource_Eav_Mysql4_Category_Collection */
 
         $unserializedCategorysAttrs = $this->config->getCategoryAdditionalAttributes($storeId);
@@ -74,6 +72,7 @@ public function getCategoryCollectionQuery($storeId, $categoryIds = null)
             ->addUrlRewriteToResult()
             ->addIsActiveFilter()
             ->setStoreId($storeId)
+            ->addAttributeToFilter('include_in_menu', '1')
             ->addAttributeToSelect(array_merge(array('name'), $additionalAttr))
             ->addFieldToFilter('level', array('gt' => 1));
 

diff --git a/code/Helper/Entity/Producthelper.php b/code/Helper/Entity/Producthelper.php
@@ -234,7 +234,7 @@ public function setSettings($storeId)
 
                             $suffix_index_name = 'group_' . $group_id;
 
-                            $sort_attribute = strpos($values['attribute'], 'price') !== false ? $values['attribute'].'.'.$suffix_index_name : $values['attribute'];
+                            $sort_attribute = strpos($values['attribute'], 'price') !== false ? $values['attribute'].'.'.$currencies[0].'.'.$suffix_index_name : $values['attribute'];
 
                             $mergeSettings['ranking'] = array($values['sort'].'('.$sort_attribute.')', 'typo', 'geo', 'words', 'proximity', 'attribute', 'exact', 'custom');
 
@@ -244,7 +244,7 @@ public function setSettings($storeId)
                 }
                 else
                 {
-                    $sort_attribute = strpos($values['attribute'], 'price') !== false ? $values['attribute'].'.'.'default' : $values['attribute'];
+                    $sort_attribute = strpos($values['attribute'], 'price') !== false ? $values['attribute'].'.'.$currencies[0].'.'.'default' : $values['attribute'];
 
                     $mergeSettings['ranking'] = array($values['sort'].'('.$sort_attribute.')', 'typo', 'geo', 'words', 'proximity', 'attribute', 'exact', 'custom');
 
@@ -258,7 +258,7 @@ public function setSettings($storeId)
         }
     }
 
-    private function getFields($store)
+    protected function getFields($store)
     {
         $tax_helper = Mage::helper('tax');
 
@@ -271,7 +271,7 @@ private function getFields($store)
         return array('price' => false, 'price_with_tax' => true);
     }
 
-    private function formatPrice($price, $includeContainer, $currency_code)
+    protected function formatPrice($price, $includeContainer, $currency_code)
     {
         if (!isset(static::$_currencies[$currency_code]))
         {
@@ -286,7 +286,7 @@ private function formatPrice($price, $includeContainer, $currency_code)
         return $price;
     }
 
-    private function handlePrice(&$product, $sub_products, &$customData)
+    protected function handlePrice(&$product, $sub_products, &$customData)
     {
         $fields                     = $this->getFields($product->getStore());
         $customer_groups_enabled    = $this->config->isCustomerGroupsEnabled($product->getStoreId());
@@ -371,7 +371,7 @@ private function handlePrice(&$product, $sub_products, &$customData)
                     }
                 }
 
-                if ($type == 'configurable' || $type == 'grouped' || $type == 'bundle')
+                if ($type == 'grouped' || $type == 'bundle')
                 {
                     $min = PHP_INT_MAX;
                     $max = 0;
@@ -383,7 +383,7 @@ private function handlePrice(&$product, $sub_products, &$customData)
                         list($min, $max) = $_priceModel->getTotalPrices($product, null, $with_tax, true);
                     }
 
-                    if ($type == 'grouped' || $type == 'configurable')
+                    if ($type == 'grouped')
                     {
                         if (count($sub_products) > 0)
                         {
@@ -508,6 +508,7 @@ public function getObject(Mage_Catalog_Model_Product $product)
             $categoryCollection = Mage::getResourceModel('catalog/category_collection')
                 ->addAttributeToSelect('name')
                 ->addAttributeToFilter('entity_id', $_categoryIds)
+                ->addAttributeToFilter('include_in_menu', '1')
                 ->addFieldToFilter('level', array('gt' => 1))
                 ->addIsActiveFilter();
 

diff --git a/code/Helper/Logger.php b/code/Helper/Logger.php
@@ -2,10 +2,10 @@
 
 class Algolia_Algoliasearch_Helper_Logger extends Mage_Core_Helper_Abstract
 {
-    private $enabled;
-    private $config;
-    private $timers = array();
-    private $stores = array();
+    protected $enabled;
+    protected $config;
+    protected $timers = array();
+    protected $stores = array();
 
     public function __construct()
     {
@@ -53,11 +53,13 @@ public function stop($action)
 
     public function log($message)
     {
-        Mage::log($message, null, 'algolia.log');
+        if ($this->config->isLoggingEnabled()) {
+            Mage::log($message, null, 'algolia.log');
+        }
     }
 
-    private function formatTime($begin, $end)
+    protected function formatTime($begin, $end)
     {
         return ($end - $begin).'sec';
     }
-}
+}
diff --git a/code/Model/Indexer/Algolia.php b/code/Model/Indexer/Algolia.php
@@ -5,14 +5,14 @@ class Algolia_Algoliasearch_Model_Indexer_Algolia extends Mage_Index_Model_Index
     const EVENT_MATCH_RESULT_KEY = 'algoliasearch_match_result';
 
     /** @var Algolia_Algoliasearch_Model_Resource_Engine */
-    private $engine;
-    private $config;
+    protected $engine;
+    protected $config;
 
     public static $product_categories = array();
-    private static $credential_error = false;
+    protected static $credential_error = false;
 
     /** @var Algolia_Algoliasearch_Helper_Logger */
-    private $logger;
+    protected $logger;
 
     public function __construct()
     {
@@ -196,7 +196,15 @@ protected function _registerCatalogProductEvent(Mage_Index_Model_Event $event)
                 }
 
                 $reindexData['catalogsearch_force_reindex'] = TRUE;
-                $reindexData['catalogsearch_product_ids'] = $actionObject->getProductIds();
+
+                if ($actionObject->getIsDeleted())
+                {
+                    $reindexData['catalogsearch_delete_product_id'] = $actionObject->getProductIds();
+                }
+                else
+                {
+                    $reindexData['catalogsearch_product_ids'] = $actionObject->getProductIds();
+                }
 
                 foreach ($reindexData as $k => $v)
                 {

diff --git a/code/Model/Indexer/Algoliaadditionalsections.php b/code/Model/Indexer/Algoliaadditionalsections.php
@@ -5,8 +5,8 @@ class Algolia_Algoliasearch_Model_Indexer_Algoliaadditionalsections extends Mage
     const EVENT_MATCH_RESULT_KEY = 'algoliasearch_match_result';
 
     /** @var Algolia_Algoliasearch_Model_Resource_Engine */
-    private $engine;
-    private $config;
+    protected $engine;
+    protected $config;
 
     public function __construct()
     {

diff --git a/code/Model/Indexer/Algoliacategories.php b/code/Model/Indexer/Algoliacategories.php
@@ -5,10 +5,10 @@ class Algolia_Algoliasearch_Model_Indexer_Algoliacategories extends Mage_Index_M
     const EVENT_MATCH_RESULT_KEY = 'algoliasearch_match_result';
 
     /** @var Algolia_Algoliasearch_Model_Resource_Engine */
-    private $engine;
-    private $config;
+    protected $engine;
+    protected $config;
 
-    private static $credential_error = false;
+    protected static $credential_error = false;
 
     public function __construct()
     {
@@ -70,15 +70,14 @@ protected function _registerCatalogCategoryEvent(Mage_Index_Model_Event $event)
                 $category   = $event->getDataObject();
                 $productIds = $category->getAffectedProductIds();
 
-                if (! $category->getData('is_active'))
+                if (! $category->getData('is_active') || ! $category->getData('include_in_menu'))
                 {
                     $event->addNewData('catalogsearch_delete_category_id', array_merge(array($category->getId()), $category->getAllChildren(TRUE)));
 
                     if ($productIds)
                     {
                         $event->addNewData('catalogsearch_update_product_id', $productIds);
                     }
-
                 }
                 elseif ($productIds)
                 {

diff --git a/code/Model/Indexer/Algoliapages.php b/code/Model/Indexer/Algoliapages.php
@@ -5,8 +5,8 @@ class Algolia_Algoliasearch_Model_Indexer_Algoliapages extends Mage_Index_Model_
     const EVENT_MATCH_RESULT_KEY = 'algoliasearch_match_result';
 
     /** @var Algolia_Algoliasearch_Model_Resource_Engine */
-    private $engine;
-    private $config;
+    protected $engine;
+    protected $config;
 
     public function __construct()
     {

diff --git a/code/Model/Indexer/Algoliaqueuerunner.php b/code/Model/Indexer/Algoliaqueuerunner.php
@@ -3,9 +3,9 @@
 class Algolia_Algoliasearch_Model_Indexer_Algoliaqueuerunner extends Mage_Index_Model_Indexer_Abstract
 {
     const EVENT_MATCH_RESULT_KEY = 'algoliasearch_match_result';
-    private $config;
+    protected $config;
     /** @var Algolia_Algoliasearch_Model_Queue */
-    private $queue;
+    protected $queue;
 
     public function __construct()
     {

diff --git a/code/Model/Indexer/Algoliasuggestions.php b/code/Model/Indexer/Algoliasuggestions.php
@@ -5,8 +5,8 @@ class Algolia_Algoliasearch_Model_Indexer_Algoliasuggestions extends Mage_Index_
     const EVENT_MATCH_RESULT_KEY = 'algoliasearch_match_result';
 
     /** @var Algolia_Algoliasearch_Model_Resource_Engine */
-    private $engine;
-    private $config;
+    protected $engine;
+    protected $config;
 
     public function __construct()
     {

diff --git a/code/Model/Queue.php b/code/Model/Queue.php
@@ -12,7 +12,7 @@ class Algolia_Algoliasearch_Model_Queue
     /** @var Algolia_Algoliasearch_Helper_Logger */
     protected $logger;
 
-    private $by_page;
+    protected $by_page;
 
     public function __construct()
     {
@@ -51,7 +51,7 @@ public function runCron()
         $this->run($nbJobs);
     }
 
-    private function mergeable($j1, $j2)
+    protected function mergeable($j1, $j2)
     {
         if ($j1['class'] !== $j2['class'])
             return false;
@@ -79,7 +79,7 @@ private function mergeable($j1, $j2)
         return true;
     }
 
-    private function sortAndMergeJob($old_jobs)
+    protected function sortAndMergeJob($old_jobs)
     {
         usort($old_jobs, function ($a, $b) {
             if (strcmp($a['class'], $b['class']) !== 0)

diff --git a/code/Model/Resource/Engine.php b/code/Model/Resource/Engine.php
@@ -6,12 +6,12 @@ class Algolia_Algoliasearch_Model_Resource_Engine extends Mage_CatalogSearch_Mod
 {
     const ONE_TIME_AMOUNT = 100;
     /** @var Algolia_Algoliasearch_Helper_Logger */
-    private $logger;
-    private $queue;
-    private $config;
-    private $product_helper;
-    private $category_helper;
-    private $suggestion_helper;
+    protected $logger;
+    protected $queue;
+    protected $config;
+    protected $product_helper;
+    protected $category_helper;
+    protected $suggestion_helper;
 
     public function _construct()
     {
@@ -37,11 +37,11 @@ public function removeProducts($storeId = null, $product_ids = null)
     {
         $ids = Algolia_Algoliasearch_Helper_Entity_Helper::getStores($storeId);
 
+        if (is_array($product_ids) == false)
+            $product_ids = array($product_ids);
+
         foreach ($ids as $id)
         {
-            if (is_array($product_ids) == false)
-                $product_ids = array($product_ids);
-
             $by_page = $this->config->getNumberOfElementByPage();
 
             if (is_array($product_ids) && count($product_ids) > $by_page)
@@ -236,7 +236,7 @@ public function rebuildProductIndex($storeId = null, $productIds = null)
         return $this;
     }
 
-    private function _rebuildCategoryIndex($storeId, $categoryIds = null)
+    protected function _rebuildCategoryIndex($storeId, $categoryIds = null)
     {
         if ($categoryIds == null || count($categoryIds) == 0)
         {
@@ -257,7 +257,7 @@ private function _rebuildCategoryIndex($storeId, $categoryIds = null)
         return $this;
     }
 
-    private function _rebuildProductIndex($storeId, $productIds = null)
+    protected function _rebuildProductIndex($storeId, $productIds = null)
     {
         if ($productIds == null || count($productIds) == 0)
         {

diff --git a/code/Model/Resource/Fulltext.php b/code/Model/Resource/Fulltext.php
@@ -5,11 +5,11 @@ class Algolia_Algoliasearch_Model_Resource_Fulltext extends Mage_CatalogSearch_M
 
     /** Empty because we need it to do nothing (no mysql stuff), Indexing is handled by Model/Indexer/Algolia */
 
-    private $engine;
-    private $config;
+    protected $engine;
+    protected $config;
 
     /** @var Algolia_Algoliasearch_Helper_Logger */
-    private $logger;
+    protected $logger;
 
     public function __construct()
     {