Ошибки seo_pro

[savage4pro]

Тема касается работы встроенного модуля seo_pro, все манипуляции будем производить в файле catalog/controller/common/seo_pro.php
поехали

• Спорный вопрос, на который натолкнул автор этого поста. Нужен ли именно 301-й редирект, а не 302
  если нужен именно он, то ищем
  

  					header($this->request->server['SERVER_PROTOCOL'] . ' 301 Moved Permanently');					$this->response->redirect($this->cache_data['queries'][$route_]);

  и 
  

  			header($this->request->server['SERVER_PROTOCOL'] . ' 301 Moved Permanently');			$this->response->redirect($seo);

  заменяем эти блоки на
  

  					$this->response->redirect($this->cache_data['queries'][$route_], 301);

  и
  

  			$this->response->redirect($seo, 301);

  соответственно
  
  а если нужен 302-й редирект, то может быть стоит убрать из обоих блоков эту строку (все равно итоговая страница будет перенаправляться через 302-й, как указано в дефолтном значении Response -> redirect()):
  

  header($this->request->server['SERVER_PROTOCOL'] . ' 301 Moved Permanently');

• Для того, чтобы ssl работал не на всем сайте, а только на страницах, где он должен быть при включении ssl в настройках магазина
  ищем
  

  			$seo = str_replace('&', '&', $this->url->link($this->request->get['route'], $this->getQueryString(array('route')), 'NONSSL'));

  заменяем на
  

  			$seo = str_replace('&', '&', $this->url->link($this->request->get['route'], $this->getQueryString(array('route')), false));

  Ошибка была найдена благодаря товарищу spown
   
• Чтобы исправить поведение ссылок на главную страницу, включающих дополнительные аргументы (http://сайт.ру/?utm_campaign=test), необходимо
  найти 
  

  			$link .= '&' . urldecode(http_build_query($data, '', '&'));

  и заменить на

  			$link .= (($route == 'common/home') ?  '?' : '&') . urldecode(http_build_query($data, '', '&'));

  Проблема найдена благодаря этой теме

[savage4pro]

18 час назад, Yoda сказал:

Кстати не мешало бы протестить.

работает, но видимо, новое форумное форматирование зохавало "&"

предлагаю все же вынести и использование ssl в settings (для дефолтного и отдельных магазинов):
полное или частичное (стандартное), а там каждый сам решит, что ему больше нравится
по стандарту, конечно весь сайт переводить в случае включения ssl

просто по собственному опыту могу сказать, что иногда требуется отдельные технические ссылки выносить в ssl, для curl'ов каких-нибудь внешних, а не весь сайт, тем более когда контора жопится на живой сертификат

[admin]

с настройками мне идея тоже больше нравится

[Bogdan]

Добрый день!
 

На сайте были проблемы с меткой gclid

Я взял код предложенный Йодой и всё заработало как нужно.

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

Fatal error: Uncaught exception 'ErrorException' with message 'Error: Table 'travy_db.oc_article_to_blog_category' doesn't exist<br />Error No: 1146<br />SELECT blog_category_id FROM oc_article_to_blog_category WHERE article_id = '126' ORDER BY main_blog_category DESC LIMIT 1' in /home/travy/travyspecii.com.ua/www/system/database/mysqli.php:41 Stack trace: #0 /home/travy/travyspecii.com.ua/www/vqmod/vqcache/vq2-system_library_db.php(20): DBMySQLi->query('SELECT blog_cat...') #1 /home/travy/travyspecii.com.ua/www/catalog/controller/common/seo_pro.php(371): DB->query('SELECT blog_cat...') #2 /home/travy/travyspecii.com.ua/www/catalog/controller/common/seo_pro.php(175): ControllerCommonSeoPro->getPathByArticle('126') #3 /home/travy/travyspecii.com.ua/www/system/library/url.php(71): ControllerCommonSeoPro->rewrite('http://travyspe...') #4 /home/travy/travyspecii.com.ua/www/vqmod/vqcache/vq2-catalog_controller_product_product.php(658): Url->link('blog/article', 'article_id=126') #5 [internal function]: ControllerProductProduct in /home/travy/travyspecii.com.ua/www/system/database/mysqli.php on line 41

Собственно вопрос как это побороть?
сборка ocshop 1.5.6.4

[Bogdan]

Данная ошибка вылезла, когда поставил СЕО Про.

Подскажите, что я сделал не так?

Я зашел сюда скачал архив с релизом. Потом открыл файл install/opencart.sql

 

В самом файле не нашел запроса на создание  'oc_article_to_blog_category'

Имеет смысл попробовать вручную создать эту таблицу или лучше выпилить этот запрос из СЕО про?

 

[admin]

12 часа назад, Bogdan сказал:

В самом файле не нашел запроса на создание  'oc_article_to_blog_category'

тут https://github.com/ocshop/ocshop/blob/oc-2.1.0.2/upload/install/opencart.sql

строка 8349

[Bogdan]

57 минут назад, admin сказал:

тут https://github.com/ocshop/ocshop/blob/oc-2.1.0.2/upload/install/opencart.sql

строка 8349

А я искал запрос в версии для 1.5.6.4.1

Теперь всё работает. Спасибо!
 

[admin]

13 минуты назад, Bogdan сказал:

А я искал запрос в версии для 1.5.6.4.1

Теперь всё работает. Спасибо!

главный вопрос куда у вас таблица то могла деться

вы случаем какой то модуль новостей не ставили ?

[Bogdan]

Нет не ставил.

Блог стоит родной.

Но опять же: запроса на создание таблицы в файле opencart.sql для версии 1.5.6.4.1 нет, я его взял из инсталла под 2.1.0.2 по вашей ссылке.

[admin]

11 час назад, Bogdan сказал:

opencart.sql для версии 1.5.6.4.1

а стоит у вас такая версия ?

если да то вы втулили сео про от 2х и я боюсь даже думать где и какие косяки у вас вылезут

в 1.5 таблица блога называется по другому

[Bogdan]

10 часов назад, admin сказал:

а стоит у вас такая версия ?

если да то вы втулили сео про от 2х и я боюсь даже думать где и какие косяки у вас вылезут

в 1.5 таблица блога называется по другому

Да у меня стоит  1.5.6.4.1
Я заменил содержание файла seo_pro.php на то, которое дал Yoda выше
Что б я ещё вспомнил, зачем мне это вообще понадобилось...
Но после того как я это сделал, точно помню, что проблема больше не выскакивала...

UPD. Я вспомнил, зачем мне это надо было:
У меня не работали метки Adwords:
вместо сайт.ру/категория/товар?gclid
автоматом выдавало сайт.ру/категория/товар

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

 

[admin]

14 часа назад, Bogdan сказал:

Сейчас такой проблемы нет, но какие новые проблемы это могло вызвать, я вот тоже не знаю

с блогом может быть а может и не быть :-)