SeoPro для oc3 уже существует?

[markimax]

4 часа назад, chukcha сказал:

ткните пальцем, что не так?

Обычный "стандартный" рабочий код opencart чуть допиленный
 

Спойлер

 

            if (!empty($data['filter_name'])) {
                $sql .= " AND LCASE(pd.name) LIKE '%" . $this->db->escape(utf8_strtolower($data['filter_name'])) . "%'";
            }

 

 

 

Вообще это тема seo_pro 3
 

[Desserro]

Ребята может я не в ту тему пишу!

вопрос такого характера

Сайт ложится после 320 000 товаров

с VPS все в норме

один разработчик указал на проблему сеопро для 2.3

"производительность" указал или намекнул на такие строки 

/public_html/catalog/controller/startup/seo_pro.php по строке 58

кто то с этим сталкивался?

и как бороться с этим?

[markimax]

36 минут назад, Desserro сказал:

Ребята может я не в ту тему пишу!

вопрос такого характера

Сайт ложится после 320 000 товаров

с VPS все в норме

один разработчик указал на проблему сеопро для 2.3

"производительность" указал или намекнул на такие строки 

/public_html/catalog/controller/startup/seo_pro.php по строке 58

кто то с этим сталкивался?

и как бороться с этим?

Совершенно верно
json_decode кеша URL занимает много ресурсов
При 320к товаров кеш файл довольно большой получается
Вам надо версия seo_pro без кеширования в файл с динамическим кешированием
 

[Desserro]

3 minutes ago, markimax said:

Совершенно верно
json_decode кеша URL занимает много ресурсов
При 320к товаров кеш файл довольно большой получается
Вам надо версия seo_pro без кеширования в файл с динамическим кешированием
 

А вы не могли бы подсказать где такой?

У меня opencart_pro 2.3.4

Змінено 17 серпня 2018 користувачем Desserro

[chukcha]

4 минуты назад, Desserro сказал:

А вы не могли бы подсказать где такой?

надо пилить :)
 

[markimax]

8 минут назад, Desserro сказал:

А вы не могли бы подсказать где такой?

У меня opencart_pro 2.3.4

Ну так как я занимаюсь  оптимизацией - у мня есть

[Desserro]

пилить пилить пилить ох как оно уже....

ну дело такое

и сколько такая услуга стоит в столярном цехе?

[chukcha]

29 минут назад, markimax сказал:

у мня есть

Вы не один :)

[chukcha]

1 час назад, Desserro сказал:

и сколько такая услуга стоит в столярном цехе?

По цене 6 бутылок пива :)

[Гість]

В 16.08.2018 в 17:51, chukcha сказал:

ткните пальцем, что не так?

цикл не нужен вообще

[chukcha]

4 минуты назад, mehanikweb сказал:

цикл не нужен вообще

та вы ШО?
там вопрос про langiage_id я бы задал
А про цикл.. то ваще.. вы хоть в коде разобрались?

[markimax]

39 минут назад, mehanikweb сказал:

цикл не нужен вообще

Там с кодом все в порядке.
Но вопрос не в этом
У всех бывает "ночной", "усталый" код и это не показатель совершенно (никак не влияющий на функциональность и безопасность ... просто "не красивый")
Как я говорю - "И на солнышке пятна есть"
Главное - архитектура кода. Вот когда архитектурно сделано не правильно, хуже всего. К примеру, mfp всеми любимый: формирование  ЧПУ для ajax и пагинации "сделано" костылями.. простите, парсом JS кода (вот это уже 3.14, из-за которого потом у многих проблемы). И да код там написан "красиво" но это 3.14 архитектурный,читаем "индусский говнокод"

[AlexDW]

 

по идее

$this->db->escape(utf8_strtolower($word))

раз filter_tag разбираем

 

как раз из разряда усталый/ночной

[markimax]

6 минут назад, AlexDW сказал:

по идее

$this->db->escape(utf8_strtolower($word))

раз filter_tag разбираем

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

Т е поиск по "нескольким"  тегам сразу. Своего рода фильтр OR

[AlexDW]

так чтобы он был - и нужно $this->db->escape(utf8_strtolower($word))

иначе получается что filter_tag разбирается на слова

а сейчас в цикле используются не слова, а сам filter_tag

[markimax]

2 минуты назад, AlexDW сказал:

так чтобы он был - и нужно $this->db->escape(utf8_strtolower($word))

иначе получается что filter_tag разбирается на слова

а сейчас в цикле используются не слова, а сам filter_tag

Вспомнил, давно это было, вначале был заказ чтобы искал по нескольким тегам, а потом они отказались от этой "затеи" фильтрации
Просто заменил $word на $data['filter_tag']

"Ночной" код
Ctrl->C  ... Ctrl->V
Но он особо не критичный , это не ошибка, просто как я выше писал "не красивый" код не влияющий ни на что
Поправлю
https://opencartforum.com/topic/95177-seopro-dlya-oc3-uzhe-suschestvuet/?do=findComment&comment=1133432

 

[Гість]

В 19.08.2018 в 13:22, markimax сказал:

Вспомнил, давно это было, вначале был заказ чтобы искал по нескольким тегам, а потом они отказались от этой "затеи" фильтрации
Просто заменил $word на $data['filter_tag']

"Ночной" код
Ctrl->C  ... Ctrl->V
Но он особо не критичный , это не ошибка, просто как я выше писал "не красивый" код не влияющий ни на что
Поправлю
https://opencartforum.com/topic/95177-seopro-dlya-oc3-uzhe-suschestvuet/?do=findComment&comment=1133432

 

Когда вы строку разбиваете на слова и делаете поиск по вхождению этих слов через LIKE, то делать нужно через LIKE 'слово%', а не LIKE '%слово%'. И делается это для того, чтобы поиск имел хоть какой-то смысл, а не вываливал все подряд, не имеющее к поисковому запросу ничего общего

[chukcha]

14 минут назад, mehanikweb сказал:

LIKE, то делать нужно через LIKE 'слово%', а не LIKE '%слово%'.

Идите учите sql

[markimax]

15 минут назад, mehanikweb сказал:

Когда вы строку разбиваете на слова и делаете поиск по вхождению этих слов через LIKE, то делать нужно через LIKE 'слово%', а не LIKE '%слово%'. И делается это для того, чтобы поиск имел хоть какой-то смысл, а не вываливал все подряд, не имеющее к поисковому запросу ничего общего

Расскажите это Даниэлю

"Хочу бук" - Ok, не вопрос на тебе ноутбук

[markimax]

19 минут назад, mehanikweb сказал:

Когда вы строку разбиваете на слова и делаете поиск по вхождению этих слов через LIKE, то делать нужно через LIKE 'слово%', а не LIKE '%слово%'. И делается это для того, чтобы поиск имел хоть какой-то смысл, а не вываливал все подряд, не имеющее к поисковому запросу ничего общего

Почему послал к Даниэлю...
Понимаете это стандартная философия поиска в opencart
Я вам больше скажу LIKE '%поиск%' не использует индексы в отличии от LIKE 'поиск%'
Но нравится это мне или не нравится, есть философия opencart code style & usability
Ну вот такой стандарт он дал и приходиться в соответствии с ним делать код, чтобы у пользователей не было вопросов (и их не раздражало) почему поиск по товарам можно искать часть слова, а в поиске записей "не находит"

[chukcha]

@markimax тю, я думал, что товарищ нашел ошибку указанную @AlexDW, а он не знает основ LIKE

тююююю.

[Гість]

я хз че с вами речь вести, каждый себе на уме

стандартная не стандартная, делать нужно нормально, а не копировать гавнокод

[markimax]

39 минут назад, mehanikweb сказал:

я хз че с вами речь вести, каждый себе на уме

стандартная не стандартная, делать нужно нормально, а не копировать гавнокод

А  с чего вы решили что у Даниэля там говнокод!?
Как раз и задуман поиск по "частям" слов. И это правильно и логично

[chukcha]

@mehanikweb Э.. товариСЧ, расскажите
Что будет найдено
Тошиба ноутбук - супер пупер девайс

при search 'ноутбук'

[HyperLabTeam]

вот он