[Помогите] Убрать слеш в урл категории

ivanuch90 · 20 авг 2013

Всем привет, подскажите кто знает как убрать в урл категории слеш в конце добавить просто .html, а в производителе наоборот, добавить в конце / ?
OcStore 1.5.4.1

nix · 6 сен 2013

В стандартном seo_url нет такой проблемы со слешем, ето тольео в seo_pro
можно в модуле catalog/controller/common/seo_pro.php костыль сделать типа такой вот
вместо етого

PHP:

if (isset($postfix)) { $seo_url .= trim($this->config->get('config_seo_url_postfix')); } else { $seo_url .= '/'; }

ето (костыль)

PHP:

if (isset($postfix)) { $seo_url .= trim($this->config->get('config_seo_url_postfix')); } elseif ($route == 'product/category') { $seo_url .= '.html'; } else { $seo_url .= '/'; }

Ваган · 10 фев 2014

А как в той же версии opencarta в категориях добавить слеш / в конце вместо .html

nix · 10 фев 2014

Ваган сказал(а): ↑

А как в той же версии opencarta в категориях добавить слеш / в конце вместо .html
Нажмите, чтобы раскрыть...

а так разве не работает изначально?

Ваган · 10 фев 2014

nix сказал(а): ↑

а так разве не работает изначально?
Нажмите, чтобы раскрыть...

Щас такой вид домен/zhenskiye/norka Применил ваш костыль стало домен/zhenskiye/norka.html может в .htaccess что то убирает его?

nix · 10 фев 2014

PHP:

if (isset($postfix)) { $seo_url .= trim($this->config->get('config_seo_url_postfix')); } elseif ($route == 'product/category') { $seo_url .= '/'; } else { $seo_url .= '/'; }

Ваган · 10 фев 2014

И так тоже пробовал, все равно убирает слеш. Даже ради эксперимента ставил так ///////////// и это тоже обрезает. но если поставлю ./ то показывает только точку.

nix · 10 фев 2014

Ваган сказал(а): ↑

И так тоже пробовал, все равно убирает слеш. Даже ради эксперимента ставил так ///////////// и это тоже обрезает. но если поставлю ./ то показывает только точку.
Нажмите, чтобы раскрыть...

где-то дальше идет подрез слеша, покажите свой файл catalog/controller/common/seo_pro.php

Ваган · 10 фев 2014

Спойлер

<?php
class ControllerCommonSeoPro extends Controller {
private $cache_data = null;

public function __construct($registry) {
parent::__construct($registry);
$this->cache_data = $this->cache->get('seo_pro');
if (!$this->cache_data) {
$query = $this->db->query("SELECT LOWER(`keyword`) as 'keyword', `query` FROM " . DB_PREFIX . "url_alias");
$this->cache_data = array();
foreach ($query->rows as $row) {
$this->cache_data['keywords'][$row['keyword']] = $row['query'];
$this->cache_data['queries'][$row['query']] = $row['keyword'];
}
$this->cache->set('seo_pro', $this->cache_data);
}
}

public function index() {
// Add rewrite to url class
if ($this->config->get('config_seo_url')) {
$this->url->addRewrite($this);
} else {
return;
}

// Decode URL
if (!isset($this->request->get['_route_'])) {
$this->validate();
} else {
$route = $this->request->get['_route_'];
unset($this->request->get['_route_']);
$parts = explode('/', trim(utf8_strtolower($route), '/'));
/* BEGIN Actions */
if (preg_match('#^(actions/)(.*)#', $route, $matches)) {
$parts = explode('/', trim(utf8_strtolower($matches[2]), '/'));
if($parts[0] == '' AND count($parts) == 1) {
$parts = explode('/', trim(utf8_strtolower($route), '/'));
}
}
/* END Actions */
/* BEGIN News & Reviews */
if (preg_match('#^(news/)(.*)#', $route, $matches)) {
$parts = explode('/', trim(utf8_strtolower($matches[2]), '/'));
if($parts[0] == '' AND count($parts) == 1) {
$parts = explode('/', trim(utf8_strtolower($route), '/'));
}
}
/* END News & Reviews */
list($last_part) = explode('.', array_pop($parts));
array_push($parts, $last_part);

$rows = array();
foreach ($parts as $keyword) {
if (isset($this->cache_data['keywords'][$keyword])) {
$rows[] = array('keyword' => $keyword, 'query' => $this->cache_data['keywords'][$keyword]);
}
}

if (count($rows) == sizeof($parts)) {
$queries = array();
foreach ($rows as $row) {
$queries[utf8_strtolower($row['keyword'])] = $row['query'];
}

reset($parts);
foreach ($parts as $part) {
$url = explode('=', $queries[$part], 2);

if ($url[0] == 'category_id') {
if (!isset($this->request->get['path'])) {
$this->request->get['path'] = $url[1];
} else {
$this->request->get['path'] .= '_' . $url[1];
}
} elseif (count($url) > 1) {
$this->request->get[$url[0]] = $url[1];
}
}
/* BEGIN Actions */
} elseif ( (isset($keyword_in[0]) AND $keyword_in[0] == 'actions') OR (isset($parts[0]) AND $parts[0] == 'actions') ) {
$this->request->get['route'] = 'information/actions';
/* END Actions */
/* BEGIN News & Reviews */
} elseif ( (isset($keyword_in[0]) AND $keyword_in[0] == 'news') OR (isset($parts[0]) AND $parts[0] == 'news') ) {
$this->request->get['route'] = 'information/news';
/* END News & Reviews */
} else {
$this->request->get['route'] = 'error/not_found';
}

if (isset($this->request->get['product_id'])) {
$this->request->get['route'] = 'product/product';
if (!isset($this->request->get['path'])) {
$path = $this->getPathByProduct($this->request->get['product_id']);
if ($path) $this->request->get['path'] = $path;
}
} elseif (isset($this->request->get['path'])) {
$this->request->get['route'] = 'product/category';
} elseif (isset($this->request->get['manufacturer_id'])) {
$this->request->get['route'] = 'product/manufacturer/info';
} elseif (isset($this->request->get['information_id'])) {
$this->request->get['route'] = 'information/information';
/* BEGIN Actions */
} elseif (isset($this->request->get['actions_id'])) {
$this->request->get['route'] = 'information/actions';
/* END Actions */
/* BEGIN News & Reviews */
} elseif (isset($this->request->get['news_id'])) {
$this->request->get['route'] = 'information/news';
/* END News & Reviews */

} else {
if (isset($queries[$parts[0]])) {
$this->request->get['route'] = $queries[$parts[0]];
}
}

$this->validate();

if (isset($this->request->get['route'])) {
return $this->forward($this->request->get['route']);
}
}
}

public function rewrite($link) {
if (!$this->config->get('config_seo_url')) return $link;

$seo_url = '';

$component = parse_url(str_replace('&', '&', $link));

$data = array();
parse_str($component['query'], $data);

$route = $data['route'];
unset($data['route']);

switch ($route) {
/* BEGIN Actions*/
case 'information/actions':
if (!isset($data['actions_id'])) {
$seo_url .= '/actions';
}
break;
/*END Actions*/
/* BEGIN News & Reviews */
case 'information/news':
if (!isset($data['news_id'])) {
$seo_url .= '/news';
}
break;
/* END News & Reviews */
case 'product/product':
if (isset($data['product_id'])) {
$tmp = $data;
$data = array();
if ($this->config->get('config_seo_url_include_path')) {
$data['path'] = $this->getPathByProduct($tmp['product_id']);
if (!$data['path']) return $link;
}
$data['product_id'] = $tmp['product_id'];
if (isset($tmp['tracking'])) {
$data['tracking'] = $tmp['tracking'];
}
}
break;

case 'product/category':
if (isset($data['path'])) {
$category = explode('_', $data['path']);
$category = end($category);
$data['path'] = $this->getPathByCategory($category);
if (!$data['path']) return $link;
}
break;

case 'product/product/review':
case 'information/information/info':
return $link;
break;

default:
break;
}

if ($component['scheme'] == 'https') {
$link = $this->config->get('config_ssl');
} else {
$link = $this->config->get('config_url');
}

$link .= 'index.php?route=' . $route;

if (count($data)) {
$link .= '&' . urldecode(http_build_query($data, '', '&'));
}

$queries = array();
foreach ($data as $key => $value) {
switch ($key) {
/* BEGIN Actions */
case 'actions_id':
$query = $this->db->query("SELECT * FROM " . DB_PREFIX . "url_alias WHERE `query` = '" . $this->db->escape($key . '=' . (int)$value) . "'");
if ($query->num_rows) {
$seo_url .= '/actions/' . rawurlencode($query->row['keyword']);
unset($data[$key]);
$postfix = 1;
}
break;
/* END Actions */
/* BEGIN News & Reviews */
case 'news_id':
$query = $this->db->query("SELECT * FROM " . DB_PREFIX . "url_alias WHERE `query` = '" . $this->db->escape($key . '=' . (int)$value) . "'");
if ($query->num_rows) {
$seo_url .= '/news/' . rawurlencode($query->row['keyword']);
unset($data[$key]);
$postfix = 1;
}
break;
/* END News & Reviews */
case 'product_id':
case 'manufacturer_id':
case 'category_id':
case 'information_id':
$queries[] = $key . '=' . $value;
unset($data[$key]);
$postfix = 1;
break;

case 'path':
$categories = explode('_', $value);
foreach ($categories as $category) {
$queries[] = 'category_id=' . $category;
}
unset($data[$key]);
break;

default:
break;
}
}

if(empty($queries)) {
$queries[] = $route;
if ($route != 'common/home') {$postfix = 1;} // Fix postfix
}

$rows = array();
foreach($queries as $query) {
if(isset($this->cache_data['queries'][$query])) {
$rows[] = array('query' => $query, 'keyword' => $this->cache_data['queries'][$query]);
}
}

if(count($rows) == count($queries)) {
$aliases = array();
foreach($rows as $row) {
$aliases[$row['query']] = $row['keyword'];
}
foreach($queries as $query) {
$seo_url .= '/' . rawurlencode($aliases[$query]);
}
}

if ($seo_url == '') return $link;

$seo_url = trim($seo_url, '/');

if ($component['scheme'] == 'https') {
$seo_url = $this->config->get('config_ssl') . $seo_url;
} else {
$seo_url = $this->config->get('config_url') . $seo_url;
}

if (isset($postfix)) {
$seo_url .= trim($this->config->get('config_seo_url_postfix'));
} elseif ($route == 'product/category') {
$seo_url .= '/';
} else {
$seo_url .= '/';
}
$seo_url = trim($seo_url, '//');

if (count($data)) {
$seo_url .= '?' . urldecode(http_build_query($data, '', '&'));
}

return $seo_url;
}

private function getPathByProduct($product_id) {
$product_id = (int)$product_id;
if ($product_id < 1) return false;

static $path = null;
if (!is_array($path)) {
$path = $this->cache->get('product.seopath');
if (!is_array($path)) $path = array();
}

if (!isset($path[$product_id])) {
$query = $this->db->query("SELECT category_id FROM " . DB_PREFIX . "product_to_category WHERE product_id = '" . $product_id . "' ORDER BY main_category DESC LIMIT 1");

$path[$product_id] = $this->getPathByCategory($query->num_rows ? (int)$query->row['category_id'] : 0);

$this->cache->set('product.seopath', $path);
}

return $path[$product_id];
}

private function getPathByCategory($category_id) {
$category_id = (int)$category_id;
if ($category_id < 1) return false;

static $path = null;
if (!is_array($path)) {
$path = $this->cache->get('category.seopath');
if (!is_array($path)) $path = array();
}

if (!isset($path[$category_id])) {
$max_level = 10;

$sql = "SELECT CONCAT_WS('_'";
for ($i = $max_level-1; $i >= 0; --$i) {
$sql .= ",t$i.category_id";
}
$sql .= ") AS path FROM " . DB_PREFIX . "category t0";
for ($i = 1; $i < $max_level; ++$i) {
$sql .= " LEFT JOIN " . DB_PREFIX . "category t$i ON (t$i.category_id = t" . ($i-1) . ".parent_id)";
}
$sql .= " WHERE t0.category_id = '" . $category_id . "'";

$query = $this->db->query($sql);

$path[$category_id] = $query->num_rows ? $query->row['path'] : false;

$this->cache->set('category.seopath', $path);
}

return $path[$category_id];
}

private function validate() {
if (empty($this->request->get['route']) || $this->request->get['route'] == 'error/not_found') {
return;
}

if (isset($this->request->server['HTTP_X_REQUESTED_WITH']) && strtolower($this->request->server['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
return;
}

if (isset($this->request->server['HTTPS']) && (($this->request->server['HTTPS'] == 'on') || ($this->request->server['HTTPS'] == '1'))) {
$url = str_replace('&', '&', $this->config->get('config_ssl') . ltrim($this->request->server['REQUEST_URI'], '/'));
$seo = str_replace('&', '&', $this->url->link($this->request->get['route'], $this->getQueryString(array('route')), 'SSL'));
} else {
$url = str_replace('&', '&', $this->config->get('config_url') . ltrim($this->request->server['REQUEST_URI'], '/'));
$seo = str_replace('&', '&', $this->url->link($this->request->get['route'], $this->getQueryString(array('route')), 'NONSSL'));
}

if (rawurldecode($url) != rawurldecode($seo)) {
header($this->request->server['SERVER_PROTOCOL'] . ' 301 Moved Permanently');

$this->response->redirect($seo);
}
}

private function getQueryString($exclude = array()) {
if (!is_array($exclude)) {
$exclude = array();
}

return urldecode(http_build_query(array_diff_key($this->request->get, array_flip($exclude))));
}
}
?>

--- Добавлено, 10 фев 2014 ---

Нашел решение, спасибо Вам и вашему костылю. Надо чуть изменить:
if (isset($postfix)) {$seo_url .= trim($this->config->get('config_seo_url_postfix'));
} elseif ($route == 'product/category') {
$seo_url .= '/';
} else {$seo_url .= '/';
}
На этот:
if (isset($postfix)) {$seo_url .= trim($this->config->get('config_seo_url_postfix'));
} elseif ($route == 'product/category') {
$seo_url .= '/';
}
В этом: $seo_url = trim($seo_url, '//'); убрать слеши и получится $seo_url = trim($seo_url, '');

Но вот в чем вопрос, не будут ли подводные камни?

nix · 10 фев 2014

если есть сомнения насчет безопасности то можите трим не убирать, после него конкатнуть в конец слеш при етом ничего ненадо менять с того что меняли выше.
после

PHP:

$seo_url = trim($seo_url, '//');

вставляем

PHP:

$seo_url = . '/';

но ето если сомнения закрались

Ваган · 10 фев 2014

nix сказал(а): ↑

если есть сомнения насчет безопасности то можите трим не убирать, после него конкатнуть в конец слеш при етом ничего ненадо менять с того что меняли выше.
после

PHP:

$seo_url = trim($seo_url, '//');

вставляем

PHP:

$seo_url = . '/';

но ето если сомнения закрались
Нажмите, чтобы раскрыть...

конкатнуть - ?

Делаю как вы показали и все ломается и не работает.

nix · 10 фев 2014

Ваган сказал(а): ↑

конкатнуть - ?

Делаю как вы показали и все ломается и не работает.
Нажмите, чтобы раскрыть...

сорри, ошибся, спешил на работу
так надо

PHP:

$seo_url .= '/';

Ваган · 10 фев 2014

nix сказал(а): ↑

сорри, ошибся, спешил на работу
так надо

PHP:

$seo_url .= '/';

Нажмите, чтобы раскрыть...

Спасибо так работает, но теперь почему то страницы с .html стали такие .html/
Как сделать чтоб / в конце был только в категориях

nix · 10 фев 2014

PHP:

if ($route == 'product/category') { $seo_url .= '/'; }

вместо того что выше

Ваган · 10 фев 2014

Ваган сказал(а): ↑

Спасибо так работает, но теперь почему то страницы с .html стали такие .html/
Как сделать чтоб / в конце был только в категориях
Нажмите, чтобы раскрыть...

А если вот этот отрезок кода

PHP:

if (isset($postfix)) {$seo_url .= trim($this->config->get('config_seo_url_postfix')); } elseif ($route == 'product/category') { $seo_url .= '/'; } else {$seo_url .= '/'; } $seo_url = trim($seo_url, '//');

Заменить на этот

PHP:

if (isset($postfix)) {$seo_url .= trim($this->config->get('config_seo_url_postfix')); } else { $seo_url .= '/'; } $seo_url = trim($seo_url, '//'); if ($route == 'product/category') { $seo_url .= '/'; }

Ваган · 10 фев 2014

А что Вы скажете про этот вариант? Он тоже действует вроде!

PHP:

if (isset($postfix)) { $seo_url .= trim($this->config->get('config_seo_url_postfix')); } else { $seo_url .= '/'; } if(substr($seo_url, -2) == '//') { $seo_url = substr($seo_url, 0, -1); }

nix · 10 фев 2014

ну знаете, трим можно заменить многими реплейсами сабстрингами и т.д... скажем так, у вас велосипед поедет быстрее если вы его покрасите на другой цвет?
Или от перестановки доданков сумма не меняется....

Ваган · 11 фев 2014

nix сказал(а): ↑

ну знаете, трим можно заменить многими реплейсами сабстрингами и т.д... скажем так, у вас велосипед поедет быстрее если вы его покрасите на другой цвет?
Или от перестановки доданков сумма не меняется....
Нажмите, чтобы раскрыть...

Да, но как бы и нет! С выше указанным кодом у меня почему то не получалось сравнивать:

PHP:

<li <?php if ($home.$_SERVER['REQUEST_URI'] == $shopping_cart) { ?>class="active"<?php } ?>><span class="hidden-link" data-link="<?php echo $shopping_cart; ?>" rel="nofollow"><noindex><?php echo $text_shopping_cart; ?></noindex></span></li>

для определение на какой странице находишься, чтобы показать в меню.

Войти или зарегистрироваться

[Помогите] Убрать слеш в урл категории

ivanuch90

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган

Ваган

nix php, MySQL, UNIX, MikroTik ROSAPI

Ваган