Error while tranplanting a module

[blister47]

I am trying to change the postion of the ProductsCategory module in the product page.

Currently it displays a horizontal list of products.

I want to display it vertically in the right column of the page.

I have adjusted the module template and its css accordingly.

However i get these errors when i change the position of the module.

 

Notice: Undefined index: product in /clientdata/apache-www/s/o/softgothic.com.au/www/modules/productscategory/productscategory.php on line 110

Notice: Trying to get property of non-object in /clientdata/apache-www/s/o/softgothic.com.au/www/modules/productscategory/productscategory.php on line 110

Notice: Undefined index: product in /clientdata/apache-www/s/o/softgothic.com.au/www/modules/productscategory/productscategory.php on line 111

Notice: Trying to get property of non-object in /clientdata/apache-www/s/o/softgothic.com.au/www/modules/productscategory/productscategory.php on line 113

Notice: Undefined variable: category in /clientdata/apache-www/s/o/softgothic.com.au/www/modules/productscategory/productscategory.php on line 127

 

Any help would be appreciated. Thanks.

 

 

 

Below is the code of the ProductsCategory module page:

 

///////////////

 

if (!defined('_PS_VERSION_'))
    exit;

class productsCategory extends Module
{
     private $_html;

    public function __construct()
     {
          $this->name = 'productscategory';
          $this->version = '1.4';
        $this->author = 'PrestaShop';
          $this->tab = 'front_office_features';
        $this->need_instance = 0;
        
        parent::__construct();
        
        $this->displayName = $this->l('Products Category');
        $this->description = $this->l('Display products of the same category on the product page.');
        
        if (!$this->isRegisteredInHook('header'))
            $this->registerHook('header');
     }

    public function install()
    {
        Configuration::updateValue('PRODUCTSCATEGORY_DISPLAY_PRICE', 0);
        $this->_clearCache('productscategory.tpl');
         return (parent::install()
            && $this->registerHook('rightColumn')
            && $this->registerHook('header')
            && $this->registerHook('addproduct')
            && $this->registerHook('updateproduct')
            && $this->registerHook('deleteproduct')
        );
    }
    
    public function uninstall()
    {
        Configuration::deleteByName('PRODUCTSCATEGORY_DISPLAY_PRICE');
        $this->_clearCache('productscategory.tpl');
         return parent::uninstall();
    }
    
    public function getContent()
    {
        $this->_html = '';
        if (Tools::isSubmit('submitCross') AND Tools::getValue('displayPrice') != 0 AND Tools::getValue('displayPrice') != 1)
            $this->_html .= $this->displayError('Invalid displayPrice');
        elseif (Tools::isSubmit('submitCross'))
        {
            Configuration::updateValue('PRODUCTSCATEGORY_DISPLAY_PRICE', Tools::getValue('displayPrice'));
            $this->_clearCache('productscategory.tpl');
            $this->_html .= $this->displayConfirmation($this->l('Settings updated successfully'));
        }
        $this->_html .= '
        <form action="'.Tools::safeOutput($_SERVER['REQUEST_URI']).'" method="post">
        <fieldset><legend><img src="'.$this->_path.'logo.gif" alt="" title="" />'.$this->l('Settings').'</legend>
            <label>'.$this->l('Display price on products').'</label>
            <div class="margin-form">
                <input type="radio" name="displayPrice" id="display_on" value="1" '.(Configuration::get('PRODUCTSCATEGORY_DISPLAY_PRICE') ? 'checked="checked" ' : '').'/>
                <label class="t" for="display_on"> <img src="../img/admin/enabled.gif" alt="'.$this->l('Enabled').'" title="'.$this->l('Enabled').'" /></label>
                <input type="radio" name="displayPrice" id="display_off" value="0" '.(!Configuration::get('PRODUCTSCATEGORY_DISPLAY_PRICE') ? 'checked="checked" ' : '').'/>
                <label class="t" for="display_off"> <img src="../img/admin/disabled.gif" alt="'.$this->l('Disabled').'" title="'.$this->l('Disabled').'" /></label>
                <p class="clear">'.$this->l('Show the price on the products in the block.').'</p>
            </div>
            <center><input type="submit" name="submitCross" value="'.$this->l('Save').'" class="button" /></center>
        </fieldset>
        </form>';
        return $this->_html;
    }
    
    private function getCurrentProduct($products, $id_current)
    {
        if ($products)
            foreach ($products AS $key => $product)
                if ($product['id_product'] == $id_current)
                    return $key;
        return false;
    }
    
    public function hookRightColumn($params)
    {
        $id_product = (int)$params['product']->id;
        $product = $params['product'];
        
        $cache_id = 'productscategory|'.$id_product.'|'.(isset($params['category']->id_category) ? (int)$params['category']->id_category : $product->id_category_default);

        if (!$this->isCached('productscategory.tpl', $this->getCacheId($cache_id)))
        {
            /* If the visitor has came to this product by a category, use this one */
            if (isset($params['category']->id_category))
                $category = $params['category'];
            /* Else, use the default product category */
            else
            {
                if (isset($product->id_category_default) AND $product->id_category_default > 1)
                    $category = new Category((int)$product->id_category_default);
            }
            
            if (!Validate::isLoadedObject($category) OR !$category->active)
                return;

            // Get infos
            //$categoryProducts = $category->getProducts($this->context->language->id, 1, 100); /* 100 products max. */
            $categoryProducts = $category->getProducts($this->context->language->id, 1, 5); /* 100 products max. */
            $sizeOfCategoryProducts = (int)sizeof($categoryProducts);
            $middlePosition = 0;
            
            // Remove current product from the list
            if (is_array($categoryProducts) AND sizeof($categoryProducts))
            {
                foreach ($categoryProducts AS $key => $categoryProduct)
                    if ($categoryProduct['id_product'] == $id_product)
                    {
                        unset($categoryProducts[$key]);
                        break;
                    }

                $taxes = Product::getTaxCalculationMethod();
                if (Configuration::get('PRODUCTSCATEGORY_DISPLAY_PRICE'))
                    foreach ($categoryProducts AS $key => $categoryProduct)
                        if ($categoryProduct['id_product'] != $id_product)
                        {
                            if ($taxes == 0 OR $taxes == 2)
                                $categoryProducts[$key]['displayed_price'] = Product::getPriceStatic((int)$categoryProduct['id_product'], true, NULL, 2);
                            elseif ($taxes == 1)
                                $categoryProducts[$key]['displayed_price'] = Product::getPriceStatic((int)$categoryProduct['id_product'], false, NULL, 2);
                        }
            
                // Get positions
                $middlePosition = round($sizeOfCategoryProducts / 2, 0);
                $productPosition = $this->getCurrentProduct($categoryProducts, (int)$id_product);
            
                // Flip middle product with current product
                if ($productPosition)
                {
                    $tmp = $categoryProducts[$middlePosition-1];
                    $categoryProducts[$middlePosition-1] = $categoryProducts[$productPosition];
                    $categoryProducts[$productPosition] = $tmp;
                }
            
                // If products tab higher than 30, slice it
                if ($sizeOfCategoryProducts > 30)
                {
                    $categoryProducts = array_slice($categoryProducts, $middlePosition - 15, 30, true);
                    $middlePosition = 15;
                }
            }
            
            // Display tpl
            $this->smarty->assign(array(
                'categoryProducts' => $categoryProducts,
                'middlePosition' => (int)$middlePosition,
                'ProdDisplayPrice' => Configuration::get('PRODUCTSCATEGORY_DISPLAY_PRICE')));
        }
        return $this->display(__FILE__, 'productscategory.tpl', $this->getCacheId($cache_id));
    }
    
    public function hookHeader($params)
    {
        $this->context->controller->addCSS($this->_path.'productscategory.css', 'all');
        //$this->context->controller->addJS($this->_path.'productscategory.js');
        //$this->context->controller->addJqueryPlugin('serialScroll');
    }

    public function hookAddProduct($params)
    {
        $this->_clearCache('productscategory.tpl');
    }

    public function hookUpdateProduct($params)
    {
        $this->_clearCache('productscategory.tpl');
    }

    public function hookDeleteProduct($params)
    {
        $this->_clearCache('productscategory.tpl');
    }
}
 

////////////////////

[vekia]

instead of $params['product']; use object definition like $product = new Product(Tools::getValue('id_product'));

and important: define exceptions under modules > positions tab for this addon to display it only on product page.

[blister47]

instead of $params['product']; use object definition like $product = new Product(Tools::getValue('id_product'));

and important: define exceptions under modules > positions tab for this addon to display it only on product page.

Thanks vekia...that solved the problem...