mpn2005

И по сути 301 по всем параметрам некорректен, т.к. если требуется залогигится, то это именно 302, страница же не перехала, а просто не доступна временно.

Именно это и говорит браузеру 302 редирект.

9 минут назад, dinox сказал:

Обновлен класс файлового кеша и изменен редирект на 301 в файле response(аналогичные правки как в 2.3.0.2.4). Уже в гите

Вот тут как раз не самый лучший вариант.

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

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

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

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

Их нужно убирать именно из генерируемых ссылок, если такое случается.

А что касается таких редиректов,  то по хорошему, это вне компетенции движка.

Я не про хайп, не раз просто встречал такие странные вещи в ТЗ от сеошников.

В 30.06.2021 в 17:06, zhu4koff сказал:

- нет редиректа с ссылки вида https://shop.com//////

А с чего вы взяли, что он вообще должен быть?

Это больше похоже на надуманную проблему. Одно дело, что движок не должен генерировать такие ссылки, а другое - это надуманный редирект от сомнительных гуру СЕО.

Вот пример:

https://www.google.com////////?q=test

Где тут редирект?

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

Эта нет) Упадёт с пятисотой. Я сильно сомневаюсь что кто то бы в здравом уме ошибся с переменной, дописав её через конкатенацию. Стояла б там запятая, то да)

Рассмотрим ситуацию сейчас:

1) Выдаёт ошибку, что переменной нет. Мы чётко по файлу видим, что её нет.

2) Нет переменной, значит выполнение идёт, и вместо переменной в строку попадает пустое место.

3) Изначально вопрос не стоит про ошибку 500 - и караул у нас всё пропало. Значит какой-то шаблон там есть.

Исходя из этого убирание переменной не ухудшает ситуацию уж точно. И если не было ошибки 500 с несуществующей переменной, то она и не появится.

С какой целью все эти заочные изыскания?

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

Куда ей уйти, если файла с таким шаблоном не будет? Вывалится в 500 ошибку, хоть ранд напишешь, хоть совсем удалишь переменную.

Не понимаете вы шуток.

Неужели после правки вот эта ошибка сохранится?

Цитата

PHP Notice:  Undefined variable: header in /var/www/vhosts/demo.com/httpdocs/catalog/controller/tmarket/header.php on line 108

Конечно не сохранится.

Если переменная лишняя, то всё заработает. А если нет, то тут заочно никак не поможешь. Надо смотреть и шаблоны в том числе.

25 минут назад, iglin сказал:

А вдруг там header.twig нету? Оно ведь очевидно что то дописать туда хотит)

Может. Но наиболее вероятно, что это просто лишняя переменная.

Ну а так - да, можно посоветовать перед использованием добавить:

$header = rand();

Текущая ошибка так точно уйдёт.

Просто замените это:

return $this->load->view('tmarket/page_section/header/header' . $header, $data);

На это:

return $this->load->view('tmarket/page_section/header/header', $data);

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

А вот блоки почему едут? В хроме все ровно 

Ничего в хроме не ровно. У вас никаких шагов не предпринято по поводу выравнивания данных блоков.

Вот в чём причина, блоки 1, 2 и 3 разной высоты:

Надо или делать блоки одинаковой высоты.

Или решать это стилями. Например, как сделано в дефолтной теме:

@media (min-width: 1200px) {
	#content .col-lg-2:nth-child(6n+1),
	#content .col-lg-3:nth-child(4n+1),
	#content .col-lg-4:nth-child(3n+1),
	#content .col-lg-6:nth-child(2n+1) {
		clear:left;
	}
}
@media (min-width: 992px) and (max-width: 1199px) {
	#content .col-md-2:nth-child(6n+1),
	#content .col-md-3:nth-child(4n+1),
	#content .col-md-4:nth-child(3n+1),
	#content .col-md-6:nth-child(2n+1) {
		clear:left;
	}
}
@media (min-width: 768px) and (max-width: 991px) {
	#content .col-sm-2:nth-child(6n+1),
	#content .col-sm-3:nth-child(4n+1),
	#content .col-sm-4:nth-child(3n+1),
	#content .col-sm-6:nth-child(2n+1) {
		clear:left;
	}
}

А ещё у вас левые элементы в BS сетке, которые не являются элементами этой сетки:

34 минуты назад, brogrammer сказал:

@DEVHUB так видно?

 

Если вы про отступ слева от меню, то это у вас в стилях прописано:

Увеличьте значение параметра max_input_vars

Обычно он по умолчанию 1000, но этого мало для больших форм.

Поставьте 20000, должно хватить.

Подробнее тут:

https://www.php.net/manual/ru/info.configuration.php#ini.max-input-vars

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

То есть нужен не костыль, а чтобы опции товара, такие как, цвет, размер итд
имели свой url или префикс к основному url (это маркет требует, возможно и др маркетплейсы), свой артикул или префикс к артикулу,
свою цену (например телефон с разным объемом памяти)
И конечно, чтобы остальные модули были совместимы с этими параметрами иначе толку от этого ноль.
Нет совместимость, давай ручками все заполняй и хз как выгружай

Можно делать полноценными отдельными товарами. Тогда не будет никаких проблем с выгрузками на маркет и у каждого варианта будет свой url.

А при выводе их группировать.

Но это не касается темы релиза ocStore.

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

к тому же вы смотрите код в категории, а парсится с карточки товара, совершенно разные вещи.

Если нужно сохранять указанные связи, то и парсить нужно страницу с этими связями.

А не страницу, на которой нет этих связей - и потом всё заполнять вручную.

Это только вопрос парсинга.

Я же отвечал на вопрос про реализацию. Как это можно реализовать.

3 минуты назад, egor23 сказал:

@mpn2005 я еще раз повторю, если вы не поняли, откуда и как вы собираетесь вытаскивать названия принтеров ?

Они все уже связаны и структурированы на доноре:

17 минут назад, egor23 сказал:

1. откуда и как вы собираетесь вытаскивать модель принтера ?

Вот отсюда тут как раз ID такой группы, типа общая линейка для модели картриджей. Именно по ним и идёт группировка.

И этот ID как раз и совпадает у сгруппированных моделей.

18 минут назад, egor23 сказал:

2. товар, это картридж, он может подходить к н-ному количеству принтеров, значит вам надо в поле UPC закладывать все модели. вы сможете запихнуть туда хотя бы пяток моделей ? нет. надо менять тип поля UPC в базе.

3. при чем тут категории ?

Товар - это картридж.

Список картриджей(товаров) - это категория

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

мне кажется вы поспешили с выводами и до конца не вникли во все проблемы.

Вам это кажется

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

очень интересное предложение и идея, а вот вопрос по полю UPC, закидывать если несколько принтеров все списком через запятую или только к одному картриджу один принтер получится? например тут 7 шт принтеров к которым подходит картридж и потом посадочные по принтеру как на доноре возможно будет сделать?

Я мел ввиду модельный ряд принтеров. Вот тут они сгруппированы.

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

Это может быть ID статьи в поле UPC. А в самой статье уже перечисление моделей и ссылки на нужные категории.

И именно по этому полю делать группировку.

Можно ещё дублировать в штатные фильтры привязку именно к конкретным моделям принтера.

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

т.е. штатными средствами АОПЛ и модулями не получится обойтись самостоятельно?

Можете спарсить товары. Модель принтера положить в свободное поле, например UPC.

А потом просто переверстать вывод товаров в категории и сгруппировать их по совпадению модели принтера в поле UPC,

Таким образом заполнение штатное, весь кастом только по выводу на фронте.

Можно сделать такой вывод только для определённых категорий. Это если в магазине есть и товары других типов.

Я бы взялся за такую работу от 5-10к руб.

Это без парсинга, только доработка вывода.

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

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

Всё просто. Ctrl+C а потом Ctrl+V не туда, и всё.

Вот это:

$sort_order = array($method_data);

Замените на это:

$sort_order = array();

sort_order - это отдельный массив и заполняется ниже в цикле.

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

Спасибо за ответ! А там оно сложное или можно самостоятельно справится

В 90% случаев там или сразу работает или решается совсем косметическими правками. Тогда делаются доработки без доплаты.

Но бывает, что архитектура темы такова, что придётся изрядно повозится. В таком случае речь идёт уже о платной адаптации.

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

А как узнать номер лицензии? приобрела модуль сегодня?

Напишите в личку  @HyperLabTeam и укажите домен, для которого нужно выдать ключи.

9 часов назад, Vladimir1989 сказал:

Попробуйте в поле товара jan указать номер цвета через #, а в поле вывода указать JAN.

Так выглядит с доработкой.

5 часов назад, enique сказал:

@Vladimir1989 спасибо!

Если надо так же, скиньте в личку @HyperLabTeam доступ на фтп и в админку и ссылку на образец товара.

Из коробки цвет можно задавать у товаров на закладке HPM. Где указан список связки.

Кликнуть на картинку и заменить на нужную.

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

Какая версия php?

Понизьте до версии 7.2 или 7.3

Озвученных данных не хватит для реализации обхода.

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

Если нет, то цена на решение задачи существенно возрастёт, т.к. задача потребует исследования робота РРЦ, а потом уже решение вопроса с подстановкой данных.

Есть ли дополнительный выборочный контроль РРЦ вручную? Если есть, то тут будет проблема.

Иногда данную задачу решают по другому. Например - всякими акциями, типа: зарегистрируйтесь и получите скидку 15% 

Или что-то подобное.

В любом случае, задача совсем не тривиальная. Маловероятно, что найдётся какое-то готовое решение.