не работает сортировка по цене

[Zodiac]

Добрый день, уважаемые форумчане.

У меня есть две проблемы не дающие покоя

1. после включения SEO у меня перестала работать сортировка по цене, как это поправить?

2. как сделать так чтобы левое меню было всегда раскрыто?

http://topcomplect.ru/%D0%B7%D0%B8%D0%BC%D0%BD%D0%B8%D0%B9_%D0%BA%D0%BE%D0%BC%D0%BF%D0%BB%D0%B5%D0%BA%D1%82/%D0%BD%D0%B0%D0%BA%D0%B8%D0%B4%D0%BA%D0%B8_%D1%81_%D0%BF%D0%BE%D0%B4%D0%BE%D0%B3%D1%80%D0%B5%D0%B2%D0%BE%D0%BC/

Помогите, пожалуйста, советом.

[Yesvik]

1. Проблема не связана с SEO, ссылка сортировки - /index.php?route=product/category&path=&sort=p.price&order=DESC не содержит информации о категории

[Zodiac]

просто до включения SEO это работало, или до того как я поменял SEO псевдоним категории с английского на русский.

Я не силён в программировании, подскажите пожалуйста как можно починить этот баг?

[Zodiac]

Люди помогите плс. Человек, кто дорабатывал магазин на связь не выходит.

Подскажите, пожалуйста, какую строчку в какой файл вставить чтобы эта сортировка заработала?

[freelancer]

http://topcomplect.ru/index.php?route=product/category&path=&sort=p.price&order=DESC

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

[Zodiac]

так я с радостью их предоставлю :)

[freelancer]

catalog/controller/product/category.php

[Fizik87]

catalog/controller/product/category.php

и затем ищи строчку похожую на

$this->data['sorts'][] = array(
'text'  => $this->language->get('text_price_desc'),
'value' => 'p.price-DESC',
'href'  => $this->model_tool_seo_url->rewrite(HTTP_SERVER . 'index.php?route=product/category&path=' . $this->request->get['path'] . '&sort=p.price&order=DESC')
);

нужно удостовериться, что в переменной $this->request->get['path'] есть идентификатор категории

p.s. так понял версия OpenCart 1.4.x

навсякий случай, для 1.5.x

$this->data['sorts'][] = array(
'text'  => $this->language->get('text_price_desc'),
'value' => 'p.price-DESC',
'href'  => $this->url->link('product/category', 'path=' . $this->request->get['path'] . '&sort=p.price&order=DESC' . $url)
);

p.p.s это конечно если программер ничего нужного не затер...

[Zodiac]

Спасибо за советы.я посмотрел, там эти строки есть

что ещё можно глянуть?

category.php

[Yesvik]

Такая ссылка

<a href="/index.php?route=product/category&path=&sort=p.price&order=DESC">по цене</a>

скорее всего формируется в шаблоне...

/index.php....

Это явно не из контроллера который ты выложил

[Zodiac]

немного не понял вас

по данному пути /topcomplect.ru/www/catalog/controller/product я выложил файл category.php

скажу точно что до включения SEO у меня эта сортировка работала

Также после включения SEO у меня стали выдаваться такие же страницы(не найдено) когда я кликал на категории и товары у который SEO псевдоним был в латинице написан. Поправил на русский и они (ссылки на товары и категории) заработали.

Тут есть какая-нибудь причинно следственная связь?

[Yesvik]

В контроллере ссылка формируется вот так

'href'  => $this->model_tool_seo_url->rewrite(HTTP_SERVER . 'index.php?route=product/category&path=' . $this->request->get['path'] . '&sort=p.price&order=DESC')

и ссылка не может начинаться со слеша.

С вероятностью 99% ссылка со слешем прописана в шаблоне..

[Fizik87]

Выложи еще файл catalogviewthemeТЕМАtemplateproductproduct.tpl

[Zodiac]

файл во вложении переименовал в php

product.php

[Yesvik]

Нужен catalogviewthemeТЕМАtemplateproductcategory.tpl

[Zodiac]

выложил

переименовал его в php (tpl не загружается )

category.php

[Yesvik]

if ($get_order=="ASC") {
	  echo "<a href='/index.php?route=product/category&path=".$_GET['path']."&sort=p.price&order=DESC' class='act asc'>по цене</a>";
	} elseif ($get_order=="DESC") {
	  echo "<a href='/index.php?route=product/category&path=".$_GET['path']."&sort=p.price&order=ASC' class='act desc'>по цене</a>";
	}else{ echo "<a href='/index.php?route=product/category&path=".$_GET['path']."&sort=p.price&order=DESC'>по цене</a>";}

Не подпускай к сайту того кто это сделал.

Кроме нерабочего кода тебя наградили уязвимостью...

[freelancer]

выложил

переименовал его в php (tpl не загружается )

а кто делал-то? хотелось бы, так сказать, знать героя

[Zodiac]

на фрилансе один товарищ :(

[Yesvik]

Вот шаблон с исправленной сортировкой и соответственно без уязвимости.

category.tpl.zip

Не самое умное решение, но работать будет корректно...

По хорошему надо править контроллер категорий, а не перебирать все сортировки созданные в контроллере.

Кроме замены этого шаблона тебе надо изменить несколько переменных в файле catalog/language/russian/product/category.php

$_['text_sort']		 = 'Сортировать по:';

исправить на

$_['text_sort']		 = 'Сортировка:';

и

$_['text_price_asc']	= 'Ценам: Низкие > Высокие';
$_['text_price_desc']   = 'Ценам: Высокие < Низкие';

исправить на

$_['text_price_asc']	= 'по цене';
$_['text_price_desc']   = 'по цене';

Кроме этого в шаблоне прописаны ссылки на статью с id=6 без ЧПУ.

Конечно можно прописать в шаблоне ЧПУ, но если будеш менять псевдоним этой статьи или отключиш ЧПУ - прийдётся ручками менять ссылки в шаблонах...

Правильнее формировать ссылку на эту статью в контроллере.

Ещё с поисковиками у тебя будет не всё гладко...

Во первых у тебя ЧПУ с использованием кириллицы выполнено не по стандарту... Гугл к этому относится лояльно, а Яндекс будет плеваться на такие ЧПУ.

Во вторых не понятно зачем у тебя в шаблоне заголовок первого уровня заменён на параграф... если обратишся к сеошникам они обязательно будут не довольны тем что страница без H1.

Думаю при более тщательном анализе сайта будут найдены и другие проблемы... но сайт в плачевном состоянии и жалко тратить время на его анализ.

Я бы рекомендовал снести нафиг всё это безобразие и построить сайт с нуля... Особенно учитывая что безобразия типа $_GET['path'] могут быть и в других местах.

[Zodiac]

ЧПУ я переводил через переводчик гугла, что с ними не так?

[Yesvik]

В URL не должно быть кириллицы...

http://help.yandex.ru/webmaster/?id=1115485 читай последнее предложение.

[Zodiac]

получается мне надо заменить
http://topcomplect.ru/компрессоры
на
http://topcomplect.ru/kompressory ?

но после всключения ЧПУ такие именования http://topcomplect.ru/kompressory у меня перестали работать

[Yesvik]

Нет... можно оставить и компрессоры, но кодировать.

[Zodiac]

но они у меня кодированные уже

в браузере отобрадаются как http://topcomplect.ru/зимний_комплект/

а при копировании http://topcomplect.ru/%D0%B7%D0%B8%D0%BC%D0%BD%D0%B8%D0%B9_%D0%BA%D0%BE%D0%BC%D0%BF%D0%BB%D0%B5%D0%BA%D1%82/