dinislage/PrestaShop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

// Blocklayered: Fix bug with urlrewrite

Browse Source
This commit is contained in:
mDeflotte
2012-03-29 15:25:02 +00:00
parent 29eb25facc
commit c4947ceb7d
4 changed files with 81 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files
classes/Dispatcher.php
+14 -2
View File
@@ -64,6 +64,18 @@ class DispatcherCore
'tags' => array('regexp' => '[a-zA-Z0-9-\pL]*'), 'tags' => array('regexp' => '[a-zA-Z0-9-\pL]*'),
), ),
), ),
'layered_rule' => array(
'controller' => 'category',
'rule' => '{id}-{rewrite}{/:selected_filters}',
'keywords' => array(
'id' => array('regexp' => '[0-9]+', 'param' => 'id_category'),
/* Selected filters is used by the module blocklayered */
'selected_filters' => array('regexp' => '.*', 'param' => 'selected_filters'),
'rewrite' => array('regexp' => '[a-zA-Z0-9-\pL]*'),
'meta_keywords' => array('regexp' => '[a-zA-Z0-9-\pL]*'),
'meta_title' => array('regexp' => '[a-zA-Z0-9-\pL]*'),
),
),
'category_rule' => array( 'category_rule' => array(
'controller' => 'category', 'controller' => 'category',
'rule' => '{id}-{rewrite}', 'rule' => '{id}-{rewrite}',
@@ -249,8 +261,8 @@ class DispatcherCore
$controllers['authentication'] = $controllers['auth']; $controllers['authentication'] = $controllers['auth'];
if (isset($controllers['compare'])) if (isset($controllers['compare']))
$controllers['productscomparison'] = $controllers['compare']; $controllers['productscomparison'] = $controllers['compare'];
if (isset($controllers['contact'])) if (isset($controllers['contact']))
$controllers['contactform'] = $controllers['contact']; $controllers['contactform'] = $controllers['contact'];
if (!isset($controllers[$this->controller])) if (!isset($controllers[$this->controller]))
$this->controller = $this->controller_not_found; $this->controller = $this->controller_not_found;
classes/Link.php
+14 -2
View File
@@ -133,9 +133,10 @@ class LinkCore
* @param mixed $category Category object (can be an ID category, but deprecated) * @param mixed $category Category object (can be an ID category, but deprecated)
* @param string $alias * @param string $alias
* @param int $id_lang * @param int $id_lang
* @param string $selected_filters Url parameter to autocheck filters of the module blocklayered
* @return string * @return string
*/ */
public function getCategoryLink($category, $alias = null, $id_lang = null) public function getCategoryLink($category, $alias = null, $id_lang = null, $selected_filters = null)
{ {
if (!$id_lang) if (!$id_lang)
$id_lang = Context::getContext()->language->id; $id_lang = Context::getContext()->language->id;
@@ -150,8 +151,19 @@ class LinkCore
$params['rewrite'] = (!$alias) ? $category->link_rewrite : $alias; $params['rewrite'] = (!$alias) ? $category->link_rewrite : $alias;
$params['meta_keywords'] = Tools::str2url($category->meta_keywords); $params['meta_keywords'] = Tools::str2url($category->meta_keywords);
$params['meta_title'] = Tools::str2url($category->meta_title); $params['meta_title'] = Tools::str2url($category->meta_title);
// Selected filters is used by the module blocklayered
$selected_filters = is_null($selected_filters) ? Tools::getValue('selected_filters') : $selected_filters;
if (empty($selected_filters))
$rule = 'category_rule';
else
{
$rule = 'layered_rule';
$params['selected_filters'] = $selected_filters;
}
return $url.Dispatcher::getInstance()->createUrl('category_rule', $params, $this->allow); return $url.Dispatcher::getInstance()->createUrl($rule, $params, $this->allow);
} }
/** /**
classes/controller/FrontController.php
+1 -1
View File
@@ -591,7 +591,7 @@ class FrontControllerCore extends Controller
$matchUrl = (($this->ssl && Configuration::get('PS_SSL_ENABLED')) ? 'https://' : 'http://').$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']; $matchUrl = (($this->ssl && Configuration::get('PS_SSL_ENABLED')) ? 'https://' : 'http://').$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'];
$matchUrl = rawurldecode($matchUrl); $matchUrl = rawurldecode($matchUrl);
if (!preg_match('/^'.Tools::pRegexp($canonicalURL, '/').'([&?].*)?$/', $matchUrl)) if (!preg_match('/^'.Tools::pRegexp(rawurldecode($canonicalURL), '/').'([&?].*)?$/', $matchUrl))
{ {
$params = array(); $params = array();
$excludedKey = array('isolang', 'id_lang', 'controller', 'fc'); $excludedKey = array('isolang', 'id_lang', 'controller', 'fc');
modules/blocklayered/blocklayered.php
+52 -30
View File
@@ -2150,8 +2150,7 @@ class BlockLayered extends Module
else else
$url = preg_replace('/\/(?:\w*)\/(?:[0-9]+[-\w]*)([^\?]*)\??.*/', '$1', Tools::safeOutput($_SERVER['REQUEST_URI'], true)); $url = preg_replace('/\/(?:\w*)\/(?:[0-9]+[-\w]*)([^\?]*)\??.*/', '$1', Tools::safeOutput($_SERVER['REQUEST_URI'], true));
$url_attributes = explode('/', $url); $url_attributes = explode('/', ltrim($url, '/'));
array_shift($url_attributes);
$selected_filters = array('category' => array()); $selected_filters = array('category' => array());
if (!empty($url_attributes)) if (!empty($url_attributes))
{ {
@@ -2477,12 +2476,17 @@ class BlockLayered extends Module
if (is_array($cache)) if (is_array($cache))
return $cache; return $cache;
if (version_compare(_PS_VERSION_,'1.5','>'))
$id_lang = Context::getContext()->language->id;
else
$id_lang = (int)$cookie->id_lang;
$id_parent = (int)Tools::getValue('id_category', Tools::getValue('id_category_layered', 1)); $id_parent = (int)Tools::getValue('id_category', Tools::getValue('id_category_layered', 1));
if ($id_parent == 1) if ($id_parent == 1)
return; return;
$parent = new Category((int)$id_parent); $parent = new Category((int)$id_parent, $id_lang);
if (version_compare(_PS_VERSION_,'1.5','>')) if (version_compare(_PS_VERSION_,'1.5','>'))
$id_shop = (int) Context::getContext()->shop->id; $id_shop = (int) Context::getContext()->shop->id;
@@ -2583,7 +2587,7 @@ class BlockLayered extends Module
ON a.id_attribute = lpa.id_attribute ON a.id_attribute = lpa.id_attribute
INNER JOIN '._DB_PREFIX_.'attribute_lang al INNER JOIN '._DB_PREFIX_.'attribute_lang al
ON al.id_attribute = a.id_attribute ON al.id_attribute = a.id_attribute
AND al.id_lang = '.(int)$cookie->id_lang.' AND al.id_lang = '.$id_lang.'
INNER JOIN '._DB_PREFIX_.'product as p INNER JOIN '._DB_PREFIX_.'product as p
ON p.id_product = lpa.id_product ON p.id_product = lpa.id_product
AND p.active = 1 AND p.active = 1
@@ -2591,11 +2595,11 @@ class BlockLayered extends Module
ON ag.id_attribute_group = lpa.id_attribute_group ON ag.id_attribute_group = lpa.id_attribute_group
INNER JOIN '._DB_PREFIX_.'attribute_group_lang agl INNER JOIN '._DB_PREFIX_.'attribute_group_lang agl
ON agl.id_attribute_group = lpa.id_attribute_group ON agl.id_attribute_group = lpa.id_attribute_group
AND agl.id_lang = '.(int)$cookie->id_lang.' AND agl.id_lang = '.$id_lang.'
LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_group_lang_value liagl LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_group_lang_value liagl
ON (liagl.id_attribute_group = lpa.id_attribute_group AND liagl.id_lang = '.(int)$cookie->id_lang.') ON (liagl.id_attribute_group = lpa.id_attribute_group AND liagl.id_lang = '.$id_lang.')
LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_lang_value lial LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_lang_value lial
ON (lial.id_attribute = lpa.id_attribute AND lial.id_lang = '.(int)$cookie->id_lang.') '; ON (lial.id_attribute = lpa.id_attribute AND lial.id_lang = '.$id_lang.') ';
$sql_query['where'] = 'WHERE a.id_attribute_group = '.(int)$filter['id_value'].' $sql_query['where'] = 'WHERE a.id_attribute_group = '.(int)$filter['id_value'].'
AND p.id_product IN ( AND p.id_product IN (
SELECT id_product SELECT id_product
@@ -2620,7 +2624,7 @@ class BlockLayered extends Module
ON a.id_attribute = lpa.id_attribute ON a.id_attribute = lpa.id_attribute
INNER JOIN '._DB_PREFIX_.'attribute_lang al INNER JOIN '._DB_PREFIX_.'attribute_lang al
ON al.id_attribute = a.id_attribute ON al.id_attribute = a.id_attribute
AND al.id_lang = '.(int)$cookie->id_lang.' AND al.id_lang = '.$id_lang.'
INNER JOIN '._DB_PREFIX_.'product as p INNER JOIN '._DB_PREFIX_.'product as p
ON p.id_product = lpa.id_product ON p.id_product = lpa.id_product
AND p.active = 1 AND p.active = 1
@@ -2628,11 +2632,11 @@ class BlockLayered extends Module
ON ag.id_attribute_group = lpa.id_attribute_group ON ag.id_attribute_group = lpa.id_attribute_group
INNER JOIN '._DB_PREFIX_.'attribute_group_lang agl INNER JOIN '._DB_PREFIX_.'attribute_group_lang agl
ON agl.id_attribute_group = lpa.id_attribute_group ON agl.id_attribute_group = lpa.id_attribute_group
AND agl.id_lang = '.(int)$cookie->id_lang.' AND agl.id_lang = '.$id_lang.'
LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_group_lang_value liagl LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_group_lang_value liagl
ON (liagl.id_attribute_group = lpa.id_attribute_group AND liagl.id_lang = '.(int)$cookie->id_lang.') ON (liagl.id_attribute_group = lpa.id_attribute_group AND liagl.id_lang = '.$id_lang.')
LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_lang_value lial LEFT JOIN '._DB_PREFIX_.'layered_indexable_attribute_lang_value lial
ON (lial.id_attribute = lpa.id_attribute AND lial.id_lang = '.(int)$cookie->id_lang.') ON (lial.id_attribute = lpa.id_attribute AND lial.id_lang = '.$id_lang.')
WHERE a.id_attribute_group = '.(int)$filter['id_value'].' WHERE a.id_attribute_group = '.(int)$filter['id_value'].'
@@ -2648,13 +2652,13 @@ class BlockLayered extends Module
$sql_query['from'] = ' $sql_query['from'] = '
FROM '._DB_PREFIX_.'feature_product fp FROM '._DB_PREFIX_.'feature_product fp
INNER JOIN '._DB_PREFIX_.'product p ON (p.id_product = fp.id_product AND p.active = 1) INNER JOIN '._DB_PREFIX_.'product p ON (p.id_product = fp.id_product AND p.active = 1)
LEFT JOIN '._DB_PREFIX_.'feature_lang fl ON (fl.id_feature = fp.id_feature AND fl.id_lang = '.(int)$cookie->id_lang.') LEFT JOIN '._DB_PREFIX_.'feature_lang fl ON (fl.id_feature = fp.id_feature AND fl.id_lang = '.$id_lang.')
INNER JOIN '._DB_PREFIX_.'feature_value fv ON (fv.id_feature_value = fp.id_feature_value AND (fv.custom IS NULL OR fv.custom = 0)) INNER JOIN '._DB_PREFIX_.'feature_value fv ON (fv.id_feature_value = fp.id_feature_value AND (fv.custom IS NULL OR fv.custom = 0))
LEFT JOIN '._DB_PREFIX_.'feature_value_lang fvl ON (fvl.id_feature_value = fp.id_feature_value AND fvl.id_lang = '.(int)$cookie->id_lang.') LEFT JOIN '._DB_PREFIX_.'feature_value_lang fvl ON (fvl.id_feature_value = fp.id_feature_value AND fvl.id_lang = '.$id_lang.')
LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_lang_value lifl LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_lang_value lifl
ON (lifl.id_feature = fp.id_feature AND lifl.id_lang = '.(int)$cookie->id_lang.') ON (lifl.id_feature = fp.id_feature AND lifl.id_lang = '.$id_lang.')
LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_value_lang_value lifvl LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_value_lang_value lifvl
ON (lifvl.id_feature_value = fp.id_feature_value AND lifvl.id_lang = '.(int)$cookie->id_lang.') '; ON (lifvl.id_feature_value = fp.id_feature_value AND lifvl.id_lang = '.$id_lang.') ';
$sql_query['where'] = 'WHERE p.`active` = 1 AND fp.id_feature = '.(int)$filter['id_value'].' $sql_query['where'] = 'WHERE p.`active` = 1 AND fp.id_feature = '.(int)$filter['id_value'].'
AND p.id_product IN ( AND p.id_product IN (
SELECT id_product SELECT id_product
@@ -2674,13 +2678,13 @@ class BlockLayered extends Module
FROM '._DB_PREFIX_.'feature_product fp FROM '._DB_PREFIX_.'feature_product fp
INNER JOIN '._DB_PREFIX_.'product p ON (p.id_product = fp.id_product AND p.active = 1) INNER JOIN '._DB_PREFIX_.'product p ON (p.id_product = fp.id_product AND p.active = 1)
LEFT JOIN '._DB_PREFIX_.'feature_lang fl ON (fl.id_feature = fp.id_feature AND fl.id_lang = '.(int)$cookie->id_lang.') LEFT JOIN '._DB_PREFIX_.'feature_lang fl ON (fl.id_feature = fp.id_feature AND fl.id_lang = '.$id_lang.')
INNER JOIN '._DB_PREFIX_.'feature_value fv ON (fv.id_feature_value = fp.id_feature_value AND (fv.custom IS NULL OR fv.custom = 0)) INNER JOIN '._DB_PREFIX_.'feature_value fv ON (fv.id_feature_value = fp.id_feature_value AND (fv.custom IS NULL OR fv.custom = 0))
LEFT JOIN '._DB_PREFIX_.'feature_value_lang fvl ON (fvl.id_feature_value = fp.id_feature_value AND fvl.id_lang = '.(int)$cookie->id_lang.') LEFT JOIN '._DB_PREFIX_.'feature_value_lang fvl ON (fvl.id_feature_value = fp.id_feature_value AND fvl.id_lang = '.$id_lang.')
LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_lang_value lifl LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_lang_value lifl
ON (lifl.id_feature = fp.id_feature AND lifl.id_lang = '.(int)$cookie->id_lang.') ON (lifl.id_feature = fp.id_feature AND lifl.id_lang = '.$id_lang.')
LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_value_lang_value lifvl LEFT JOIN '._DB_PREFIX_.'layered_indexable_feature_value_lang_value lifvl
ON (lifvl.id_feature_value = fp.id_feature_value AND lifvl.id_lang = '.(int)$cookie->id_lang.') ON (lifvl.id_feature_value = fp.id_feature_value AND lifvl.id_lang = '.$id_lang.')
WHERE p.`active` = 1 AND fp.id_feature = '.(int)$filter['id_value'].' WHERE p.`active` = 1 AND fp.id_feature = '.(int)$filter['id_value'].'
GROUP BY fv.id_feature_value'; GROUP BY fv.id_feature_value';
} }
@@ -2701,7 +2705,7 @@ class BlockLayered extends Module
WHERE cp.id_category = c.id_category '; WHERE cp.id_category = c.id_category ';
$sql_query['group'] = ') count_products $sql_query['group'] = ') count_products
FROM '._DB_PREFIX_.'category c FROM '._DB_PREFIX_.'category c
LEFT JOIN '._DB_PREFIX_.'category_lang cl ON (cl.id_category = c.id_category AND cl.id_lang = '.(int)$cookie->id_lang.') LEFT JOIN '._DB_PREFIX_.'category_lang cl ON (cl.id_category = c.id_category AND cl.id_lang = '.$id_lang.')
WHERE c.nleft > '.(int)$parent->nleft.' WHERE c.nleft > '.(int)$parent->nleft.'
AND c.nright < '.(int)$parent->nright.' AND c.nright < '.(int)$parent->nright.'
'.($depth ? 'AND c.level_depth <= '.($parent->level_depth+(int)$depth) : '').' '.($depth ? 'AND c.level_depth <= '.($parent->level_depth+(int)$depth) : '').'
@@ -3050,7 +3054,7 @@ class BlockLayered extends Module
LEFT JOIN `'._DB_PREFIX_.'layered_indexable_attribute_group` liag LEFT JOIN `'._DB_PREFIX_.'layered_indexable_attribute_group` liag
ON liag.id_attribute_group = agl.id_attribute_group ON liag.id_attribute_group = agl.id_attribute_group
WHERE indexable IS NULL OR indexable = 0 WHERE indexable IS NULL OR indexable = 0
AND id_lang = '.(int)$cookie->id_lang) as $attribute) AND id_lang = '.$id_lang) as $attribute)
$non_indexable[] = Tools::link_rewrite($attribute['public_name']); $non_indexable[] = Tools::link_rewrite($attribute['public_name']);
// Get all non indexable features // Get all non indexable features
@@ -3060,7 +3064,7 @@ class BlockLayered extends Module
LEFT JOIN `'._DB_PREFIX_.'layered_indexable_feature` lif LEFT JOIN `'._DB_PREFIX_.'layered_indexable_feature` lif
ON lif.id_feature = fl.id_feature ON lif.id_feature = fl.id_feature
WHERE indexable IS NULL OR indexable = 0 WHERE indexable IS NULL OR indexable = 0
AND id_lang = '.(int)$cookie->id_lang) as $attribute) AND id_lang = '.$id_lang) as $attribute)
$non_indexable[] = Tools::link_rewrite($attribute['name']); $non_indexable[] = Tools::link_rewrite($attribute['name']);
//generate SEO link //generate SEO link
@@ -3070,10 +3074,7 @@ class BlockLayered extends Module
$param_group_selected_array = array(); $param_group_selected_array = array();
$title_values = array(); $title_values = array();
$meta_values = array(); $meta_values = array();
$link = new Link();
$link_base = $link->getCategoryLink($id_parent, Category::getLinkRewrite($id_parent, (int)($cookie->id_lang)), (int)($cookie->id_lang));
//get filters checked by group //get filters checked by group
foreach ($filter_blocks as $type_filter) foreach ($filter_blocks as $type_filter)
{ {
@@ -3164,11 +3165,19 @@ class BlockLayered extends Module
foreach ($non_indexable as $value) foreach ($non_indexable as $value)
if (strpos($parameters, '/'.$value) !== false) if (strpos($parameters, '/'.$value) !== false)
$nofollow = true; $nofollow = true;
// Write link by mode rewriting
if (!Configuration::get('PS_REWRITING_SETTINGS')) if (version_compare(_PS_VERSION_,'1.5','>'))
$type_filter['values'][$key]['link'] = $link_base.'&selected_filters='.$parameters; $type_filter['values'][$key]['link'] = $this->getCategoryLink($parent, ltrim($parameters, '/'));
else else
$type_filter['values'][$key]['link'] = $link_base.$parameters; {
$link = new Link();
$link_base = $link->getCategoryLink($id_parent, Category::getLinkRewrite($id_parent, (int)($cookie->id_lang)), (int)($cookie->id_lang));
// Write link by mode rewriting
if (!Configuration::get('PS_REWRITING_SETTINGS'))
$type_filter['values'][$key]['link'] = $link_base.'&selected_filters='.$parameters;
else
$type_filter['values'][$key]['link'] = $link_base.$parameters;
}
$type_filter['values'][$key]['rel'] = ($nofollow) ? 'nofollow' : ''; $type_filter['values'][$key]['rel'] = ($nofollow) ? 'nofollow' : '';
} }
@@ -3203,6 +3212,19 @@ class BlockLayered extends Module
return $cache; return $cache;
} }
/**
* Create a link to a category with a filter params
*
* @param mixed $category Category object (can be an ID category, but deprecated)
* @param string $selected_filters
* @return string
*/
public function getCategoryLink($category, $selected_filters = '')
{
return Context::getContext()->link->getCategoryLink($category, null, null, $selected_filters);
}
public function cleanFilterByIdValue($attributes, $id_value) public function cleanFilterByIdValue($attributes, $id_value)
{ {
$selected_filters = array(); $selected_filters = array();