// Fix bugs on module wishlist
This commit is contained in:
@@ -272,17 +272,17 @@ class WishList extends ObjectModel
|
|||||||
$products = Db::getInstance()->executeS('
|
$products = Db::getInstance()->executeS('
|
||||||
SELECT wp.`id_product`, wp.`quantity`, p.`quantity` AS product_quantity, pl.`name`, wp.`id_product_attribute`, wp.`priority`, pl.link_rewrite, cl.link_rewrite AS category_rewrite
|
SELECT wp.`id_product`, wp.`quantity`, p.`quantity` AS product_quantity, pl.`name`, wp.`id_product_attribute`, wp.`priority`, pl.link_rewrite, cl.link_rewrite AS category_rewrite
|
||||||
FROM `'._DB_PREFIX_.'wishlist_product` wp
|
FROM `'._DB_PREFIX_.'wishlist_product` wp
|
||||||
JOIN `'._DB_PREFIX_.'product` p ON p.`id_product` = wp.`id_product`
|
LEFT JOIN `'._DB_PREFIX_.'product` p ON p.`id_product` = wp.`id_product`
|
||||||
'.Shop::addSqlAssociation('product', 'p').'
|
'.Shop::addSqlAssociation('product', 'p').'
|
||||||
JOIN `'._DB_PREFIX_.'product_lang` pl ON pl.`id_product` = wp.`id_product`'.Shop::addSqlRestrictionOnLang('pl').'
|
LEFT JOIN `'._DB_PREFIX_.'product_lang` pl ON pl.`id_product` = wp.`id_product`'.Shop::addSqlRestrictionOnLang('pl').'
|
||||||
JOIN `'._DB_PREFIX_.'wishlist` w ON w.`id_wishlist` = wp.`id_wishlist`
|
LEFT JOIN `'._DB_PREFIX_.'wishlist` w ON w.`id_wishlist` = wp.`id_wishlist`
|
||||||
LEFT JOIN `'._DB_PREFIX_.'category_lang` cl ON cl.`id_category` = product_shop.`id_category_default` AND cl.id_lang='.(int)$id_lang.Shop::addSqlRestrictionOnLang('cl').'
|
LEFT JOIN `'._DB_PREFIX_.'category_lang` cl ON cl.`id_category` = product_shop.`id_category_default` AND cl.id_lang='.(int)$id_lang.Shop::addSqlRestrictionOnLang('cl').'
|
||||||
WHERE w.`id_customer` = '.(int)($id_customer).'
|
WHERE w.`id_customer` = '.(int)($id_customer).'
|
||||||
AND pl.`id_lang` = '.(int)($id_lang).'
|
AND pl.`id_lang` = '.(int)($id_lang).'
|
||||||
AND wp.`id_wishlist` = '.(int)($id_wishlist).
|
AND wp.`id_wishlist` = '.(int)($id_wishlist).
|
||||||
(empty($id_product) === false ? ' AND wp.`id_product` = '.(int)($id_product) : '').
|
(empty($id_product) === false ? ' AND wp.`id_product` = '.(int)($id_product) : '').
|
||||||
($quantity == true ? ' AND wp.`quantity` != 0': '').'
|
($quantity == true ? ' AND wp.`quantity` != 0': '').'
|
||||||
GROUP BY p.id_product');
|
GROUP BY p.id_product, wp.id_product_attribute');
|
||||||
if (empty($products) === true OR !sizeof($products))
|
if (empty($products) === true OR !sizeof($products))
|
||||||
return array();
|
return array();
|
||||||
for ($i = 0; $i < sizeof($products); ++$i)
|
for ($i = 0; $i < sizeof($products); ++$i)
|
||||||
@@ -292,7 +292,7 @@ class WishList extends ObjectModel
|
|||||||
{
|
{
|
||||||
$result = Db::getInstance()->executeS('
|
$result = Db::getInstance()->executeS('
|
||||||
SELECT al.`name` AS attribute_name, pa.`quantity` AS "attribute_quantity"
|
SELECT al.`name` AS attribute_name, pa.`quantity` AS "attribute_quantity"
|
||||||
FROM `'._DB_PREFIX_.'product_attribute_combination` pac
|
FROM `'._DB_PREFIX_.'product_attribute_combination` pac
|
||||||
LEFT JOIN `'._DB_PREFIX_.'attribute` a ON (a.`id_attribute` = pac.`id_attribute`)
|
LEFT JOIN `'._DB_PREFIX_.'attribute` a ON (a.`id_attribute` = pac.`id_attribute`)
|
||||||
LEFT JOIN `'._DB_PREFIX_.'attribute_group` ag ON (ag.`id_attribute_group` = a.`id_attribute_group`)
|
LEFT JOIN `'._DB_PREFIX_.'attribute_group` ag ON (ag.`id_attribute_group` = a.`id_attribute_group`)
|
||||||
LEFT JOIN `'._DB_PREFIX_.'attribute_lang` al ON (a.`id_attribute` = al.`id_attribute` AND al.`id_lang` = '.(int)($id_lang).')
|
LEFT JOIN `'._DB_PREFIX_.'attribute_lang` al ON (a.`id_attribute` = al.`id_attribute` AND al.`id_lang` = '.(int)($id_lang).')
|
||||||
|
|||||||
@@ -124,6 +124,7 @@
|
|||||||
|
|
||||||
/* form add ****************************************** */
|
/* form add ****************************************** */
|
||||||
#form_wishlist {}
|
#form_wishlist {}
|
||||||
|
#form_wishlist fieldset { padding: 20px }
|
||||||
#form_wishlist label {
|
#form_wishlist label {
|
||||||
display:inline-block;
|
display:inline-block;
|
||||||
padding:6px 15px;
|
padding:6px 15px;
|
||||||
|
|||||||
@@ -31,6 +31,7 @@ $useSSL = true;
|
|||||||
require_once(dirname(__FILE__).'/../../config/config.inc.php');
|
require_once(dirname(__FILE__).'/../../config/config.inc.php');
|
||||||
require_once(dirname(__FILE__).'/../../init.php');
|
require_once(dirname(__FILE__).'/../../init.php');
|
||||||
require_once(dirname(__FILE__).'/WishList.php');
|
require_once(dirname(__FILE__).'/WishList.php');
|
||||||
|
require_once(dirname(__FILE__).'/blockwishlist.php');
|
||||||
$context = Context::getContext();
|
$context = Context::getContext();
|
||||||
if ($context->customer->isLogged())
|
if ($context->customer->isLogged())
|
||||||
{
|
{
|
||||||
@@ -63,10 +64,11 @@ if ($context->customer->isLogged())
|
|||||||
continue;
|
continue;
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
if ($products[$i]['id_product_attribute'] != 0 && isset($combination_imgs[$products[$i]['id_product_attribute']][0]))
|
if ($products[$i]['id_product_attribute'] != 0)
|
||||||
{
|
{
|
||||||
$combination_imgs = $obj->getCombinationImages($context->language->id);
|
$combination_imgs = $obj->getCombinationImages($context->language->id);
|
||||||
$products[$i]['cover'] = $obj->id.'-'.$combination_imgs[$products[$i]['id_product_attribute']][0]['id_image'];
|
if (isset($combination_imgs[$products[$i]['id_product_attribute']][0]))
|
||||||
|
$products[$i]['cover'] = $obj->id.'-'.$combination_imgs[$products[$i]['id_product_attribute']][0]['id_image'];
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
|
|||||||
Reference in New Issue
Block a user