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

[diamante]

Здравствуйте, возник вопрос кто из авторов более прав и как все таки исправить дефект работы сайта

 

приобрел я как то на сайт (официально) два модуля, которые судя по описаниям закрывают мои хотелки

это SEO мультиязык / мультирегион 31 Автор markimax и Пошаговый подбор товаров по характеристикам (атрибутам) Автор mazein

вроде все работает красиво, но обнаружился нюанс

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

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

о чем я написал автору markimax

 

здравствуйте, у меня на сайте https://ххххххх.ru помимо вашего модуля установлен фильтр 

и есть такой дефект

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

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

в чем может быть дело? может я что то в настройках модуля не доглядел?

markimax   2 378

Отвечено: 31 марта

А потому что форма у вас вызывается мягко сказать "по тупому" при submit

index.php?route=extension/module/mmfilter_products/category

Т е  без префикса (не оптимизирован под мультиязычные url)
И причем прописано это во внешнем JS скрипте
Т е сам скрипт не заточен
Если бы хотя бы  он был inline то можно было шорткодом изменить url сабмита формы
А так вообще нельзя
Это вам надо обратится к разработчику чтобы он генерировал URL сабмит формы через стандарт opencart $this->url->link  inline кодом

 

после этого обратился к mazein

Вы можете это исправить?

mazein 

Добрый день.

 

 

Смотрим стандартный файлик common.js:

Здесь так же, как у меня - внешний скрипт, ссылка прописана.

 

Почему у Вас на странице поиска язык не переключается?

diamante

может вы меня не совсем правильно поняли, происходит следующее

 

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

mazein

Это я понял.

Нужно как-то адаптировать под модуль SEO мультиязык

diamante  3

Цитита: Это вам надо обратится к разработчику чтобы он генерировал URL сабмит формы через стандарт opencart $this->url->link  inline кодом

 

ну видимо да, подсказку вроде дали

mazein

Видимо, теперь Вы меня не так поняли.

Проблем в моём модуле нет.

Выше я привёл пример из штатного файла опенкарт, в котором аналогичный url.

diamante 

вы считаете что товарищ не прав? http://www.screencapture.ru/file/345bC8bA

mazein

Выносите вопрос на всеобщее обсуждение. Я считаю, что он не прав.

diamante

Я решил не торопиться с темой, это всегда успеется, я отправил ему ваши аргументы, вот что дядька ответил

             цитата         markimax

Добавьте в исключения модуля
extension/module/mmfilter_products/category

Но это всего то "костыль"
Лучше пусть автор как то перепишет скрипт
К примеру на ajax чтобы добавлял в hidden формы генерируемую URL через $this->url->link
А потом по сабмиту брал эту URL

 

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

 

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

программиста чтобы это исправить, однако модуль от mazein кубирован, соответственно правки может внести только он сам

все остальные методы считаются нечестными

 

или все таки ошибается markimax ? 

[chukcha]

	/* Search */
	$('#search input[name=\'search\']').parent().find('button').on('click', function() {
		var url = $('base').attr('href') + 'index.php?route=product/search';

		var value = $('header #search input[name=\'search\']').val();

		if (value) {
			url += '&search=' + encodeURIComponent(value);
		}

		location = url;
	});

Т.е. route=product/search находится в исключениях?
 

 

[diamante]

не совсем понял вопрос, и как заметил mazein по адресу /index.php?route=product/search язык почему то не переключается

Змінено 29 серпня 2021 користувачем diamante

[chukcha]

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

не совсем понял вопрос

так вопрос в общем-то не к вам, а к одному из авторов

 

Т.к вы задели вопрос о "кубе", то у одного из авторов код открыт, но вы попробуйте тужа что-то свое внести.

[diamante]

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

[chukcha]

10 минут назад, diamante сказал:

а кто из авторов в правильном направлении мыслит,

@mazein Не нарушает принятый codestyle
Вы сами можете это все сделать и поправить, как скрипт модуля, так и шаблон, и контроллер, наверное открыт для хотелки от @markimax

[diamante]

получил я ответ от markimax позволю себе донести его сюда, чтобы не ждать еще месяц

 

Добрый
К сожалению там (в теме) ответить не могу, в бане до 27 сентября
Не стоит вопрос "ошибся ли markimax"
Что за вопрос вообще такой? 
Нет конечно - не ошибся
Ошибся автор mazein (архитектурно)
Вопрос надо ставить "ошибся ли mazein"
А он ошибся в архитектуре разработки своего модуля
Его модуль не оптимизирован под мультиязычность (как и opencart замечу)
И не нужно указывать на common.js  (я про product/search) Даниэля
У Даниэля мультиязычность в opencart - одна БОЛЬШАЯ ОШИБКА
Её нету по сути. Как может быть одинаковый URL для РАЗНЫХ языков? Такого быть не должно, это грубая ошибка
И доказывать свою правоту прикрываясь ОШИБКОЙ common.js вызова product/search ... это плохо

Философия opencart построения SEO URL - только через $this->url->link (а если для search есть SEO URL   то что - лишний редирект или дубль! )
Если сделано не так - это априори ОШИБКА, и доказывать АКСИОМУ  (что это ошибка) даже нет смысла
Если автор имеет квалификацию - он просто должен переделать эту часть своего кода, чтобы URL вызова формировался через $this->url->link
К примеру, если модуль на странице, можно сформированный SEO URL выводить в hidden поле. а потом внешним скриптом его брать для вызова
Это проще простого реализовать
Надо понимать что есть стандарт генерации SEO URL - $this->url->link
А не прописывать костыли руками-это моветон

[buslikdrev]

Выбирайте кто и за сколько готов подружить.

 

Один говорит обработка только через $this->url->link

Другой может сказать, а чего ты переадресацию не сделал в зависимости от данных в куках (или для определённых роутов).

[diamante]

 заявленый функционал модуля работает как надо, вопросов нет, так же как не нашел достойной альтернативы для моей задачи, смущает одно-неужели только у меня такая проблема, и никто больше не использует такую комбинацию модулей

[chukcha]

49 минут назад, diamante сказал:

Не стоит вопрос "ошибся ли markimax"

стоит

 

49 минут назад, diamante сказал:

Ошибся автор mazein (архитектурно)

Нет, не ошибся
 

 

50 минут назад, diamante сказал:

У Даниэля мультиязычность в opencart - одна БОЛЬШАЯ ОШИБКА

И что? Значит надо исправлять., (да-да и имея прямые роуты вида product/search в том числе

51 минуту назад, diamante сказал:

Как может быть одинаковый URL для РАЗНЫХ языков? Такого быть не должно, это грубая ошибка

Элементарно, для технических страниц в том числе.. есть и кука есть и сессия)

52 минуты назад, diamante сказал:

Если автор имеет квалификацию - он просто должен переделать эту часть своего кода,

Если @markimax  имеет квалификацию - он просто должен переделать эту часть своего кода,
Найдите одно отличие..

 

Как @mazeinможет решить это вопрос, это не тема обсуждения и будет ли ее решать в том числе

 

И.. имея 1000 продаж модуля мультиязыка и 10 продаж минифильтра написанного по канонам codestyle опенкарта - кто должен решать проблему, тем более за несколько лет таких случаев немало, а воз и ныне там..

[dreanei23]

никого мое мнение не интересует, но я считаю, что все ссылки должны по возможности формироваться через $this->url->link

[diamante]

ой, а почему странные?

[chukcha]

Только что, nikifalex сказал:

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

Не совсем так

 

один считает что все допускают архитектурные ошибки, кроме него, а предположить(предусмотреть) что link  может  быть не ЧПУ

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

И кстати

про язык

	// Search
	$('#search input[name=\'search\']').parent().find('button').on('click', function() {
		var url = $('base').attr('href') + 'index.php?route=product/search&language=' + $(this).attr('data-lang');

		var value = $('header #search input[name=\'search\']').val();

		if (value) {
			url += '&search=' + encodeURIComponent(value);
		}

		location = url;
	});

И не надо говорить, что у даниеля кривой язык

 

[chukcha]

12 часов назад, nikifalex сказал:

поэтому ну mazein в данном случае неправ.

приведенный мною код - это код от 4-ки

А модуль @mazein для 2.3

 

[buslikdrev]

12 минут назад, chukcha сказал:

А модуль @mazein для 2.3

3.0

[chukcha]

5 минут назад, buslikdrev сказал:

3.0

ок

Ой, бида

url: 'index.php?route=checkout/cart/edit',
('#cart > ul').load('index.php?route=common/cart/info ul li');

location = 'index.php?route=checkout/cart';

rl: 'index.php?route=checkout/cart/remove',

там еще есть

 

Следуя канонам  "архитектуры" от @markimax получается если ты на укр языке добавил товар, то у тебя товар попадет в заказ на русском?

 

 

 

И это все в исключения?

 

[diamante]

Крутил-вертел, заметил еще один косяк, выбираю атрибуты в фильтре, жму поиск, фильтр выдает результат (index.php?route=extension/module/mmfilter_products/category&attribute_0=BMW&attribute_id_0=14&attribute_1=3%27%20(E92)&attribute_id_1=15&attribute_2=2010&attribute_id_2=16&row=3&module_id=38)     и в этом окне язык изменить уже невозможно, сколько кнопку не тыкаю он не меняется

Змінено 10 жовтня 2021 користувачем diamante
свежая мысль

[markimax]

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

extension/module/mmfilter_products/category

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

[diamante]

зашел в схемы , сделал так https://prnt.sc/1vl1uds что то не работает, в документации тоже ничего не нашел про меню модуля https://prnt.sc/1vl27gc 

[markimax]

2 часа назад, diamante сказал:

зашел в схемы , сделал так https://prnt.sc/1vl1uds что то не работает, в документации тоже ничего не нашел про меню модуля https://prnt.sc/1vl27gc 

Все сделали не правильно
В opencart создайте схему
Это мат часть даже не модуля. а opencart