LEOnidUKG

Отменяем всё это. В этом же файле ищем первый: $sql .= " AND p2c.category_id IS NULL"; И заменяем на: $sql = str_replace('SELECT *','SELECT p.*,pd.*',$sql); Иначе у нас в некоторых категориях названия товаров в админке в списке исчезает.

Смысл понятен. В файле: /admin/model/catalog/product.php Ищем строчку: $sql = $sql_crutch_matches[1] . " LEFT JOIN " . DB_PREFIX . "product_to_category p2c ON (p.product_id = p2c.product_id)" . $sql_crutch_matches[2]; После неё вставляем: $sql = str_replace('SELECT *','SELECT p.*',$sql); Иначе у нас mysql теряется из какой таблице нам нужны данные.

Соскучились по багам? А они есть у меня ещё. Не правильно формируется SEO урл для статей в блогах. Не учитывается категория, хотя в настройках выставлено учитывать её. Так же не правильно формируется кэш категорий блогов. С кэшем вообще беда какая-то в этой версии. Поехали исправлять: /system/library/seopro.php 1. Ищем код: if (isset($data['blog_category_id'])) { $blog_path = $this->getBlogPathByArticle($article_id); } Меняем на: $blog_path = $this->getBlogPathByArticle($article_id); 2. Ищем код: if (!is_array($blog_path)) { if ($this->config->get('config_seo_url_cache')) $blog_path = $this->cache->get($cache); if (!is_array($blog_path)) $blog_path = []; } Меняем на: if ($this->config->get('config_seo_url_cache')) { $blog_path = $this->cache->get($cache); } if (!is_array($blog_path)) { $blog_path = []; } В первом случаи мы убираем проверку т.к. нам всегда важно знать есть ли категория, для формирования урлов. Во втором случаи не понятная проверка на массив, хотя выше уже объявляется, что переменная массив. Скорее всего хотели проверить что-то другое, но вышло как есть. p.s. кидал на git хаб, там от разработчика 0 внимания, а тут хоть народ читает.

Зачем, мне играться с чем либо, если я код весь пересмотрел? Да, надо выключить prev,next т.к. он не правильно работает.

Печаль беда в логике модуля ocstore Параметры: Выводить тег canonical на основную страницу и Вывести prev, next теги для пагинации Эти два параметра не могут работать одновременно т.к. используется стандартный метод простановки ссылок: $this->document->addLink А он умеет только 1 ссылку уникальную принимать и из-за этого canonical на второй странице заменяется на prev т.к. ссылки одинаковые. Чтобы это всё работало одновременно надо переделывать логику работу функции $this->document->addLink Ну или просто отключать вторую функцию с prev,next которые по сути не важны уже.

да, действительно, трудно заметить разницу. Но опять же, в 3.0.3.8 версии: https://github.com/opencart/opencart/blob/master/upload/system/library/session/db.php Убрали очистку автоматическую и код самой очистки поменяли. Опять что-то пошло у них не так.

Из коробки? Так там вообще практически нет индексов. Никто не заморачивается по этому поводу из разработчиков. Если делать Pull это надо разбирать полностью чистый магазин, у меня увы его нет. У меня скрипт расставляет индексы автоматически везде, где можно если их нет.

И чем оно отличается от текущего? https://github.com/ocStore/ocStore/blob/v3.0.3.7/upload/system/library/session/db.php Всё тоже самое, долбёжка БД через раз на удаление сессий.

Я профессионально занимаюсь оптимизацией сайтов, У меня есть 677 индексов для Opencart практически на все случаи жизни и популярные плагины. Поэтому проблема не в этом.

У кого хостер ругается на нагрузку и бывают тормоза при работе сайта возможно вам будет полезен мой запрос на изменения: https://github.com/ocStore/ocStore/pull/70 Коротко говоря, проблема вновь с очистками кэша сессий, это просто бич 3х версии. Грубо говоря в файле: /system/library/session/db.php ищем строчку: $this->gc(); И меняем на: # ClearSessionDB START # if (date("N") == 7 && date("G") == 3) { // Clear Sessions only: Sunday, 3 AM $this->db->query("DELETE FROM `" . DB_PREFIX . "session` WHERE expire < DATE_SUB(NOW(), INTERVAL 7 DAY);"); } # ClearSessionDB END #

Ещё баг: /catalog/model/blog/article.php $article_data = $this->cache->get('article.total.' . (int)$this->config->get('config_language_id') . '.' . (int)$this->config->get('config_store_id') . '.' . (int)$customer_group_id . '.' . $cache); $article_data = []; if (!$article_data) { Кэш сбрасывается в любом случаи. Надо написать так: $article_data = $this->cache->get('article.total.' . (int)$this->config->get('config_language_id') . '.' . (int)$this->config->get('config_store_id') . '.' . (int)$customer_group_id . '.' . $cache); if (!$article_data) { $article_data = [];

/system/library/seopro.php public function __destruct() { if(!$this->config->get('config_seo_pro')) return; if ($this->config->get('config_seo_url_cache')){ $this->cache->set('seopro.keywords', $this->keywords); $this->cache->set('seopro.queries', $this->queries); $this->cache->set('seopro.cat_tree', $this->cat_tree); $this->cache->set('seopro.product_categories', $this->product_categories); } } Я не знаю какую цель преследовал автор, может быть он что-то забыл? Возможно надо вот так хотя бы: public function __destruct() { if(!$this->config->get('config_seo_pro')) return; if ($this->config->get('config_seo_url_cache') && !$this->cache->get('seopro.keywords')){ $this->cache->set('seopro.keywords', $this->keywords); $this->cache->set('seopro.queries', $this->queries); $this->cache->set('seopro.cat_tree', $this->cat_tree); $this->cache->set('seopro.product_categories', $this->product_categories); } }

даже уже и не помню, нужно читать у них на сайте.