Николай Савин

Позволю себе несколько замечаний.

1. Если заказов много — такой сниппет не сможет выполнить свою работу. Лучше вешать задачу на планировщик

2. Использование getCollection. Это не лучший вариант. Этот метод загружает в ОЗУ сразу всю коллекцию объектов заказа, что не лучшим образом отражается на производительности и ресурсах. Лучше использовать итератор через метод getIterator. Такой метод загружает в память только список объектов, при переборе по очереди достает из базы информацию. Этот метод существенно быстрее работает и кушает меньше ресурсов. К слову итератор это не изобретение MODX. Это паттерн PHP. Да и не только PHP — он почти во всех языках используется.

3. Это что за странная конструкция такая?

$modx->getOption('table_prefix')."ms2_order_addresses

Зачем так? Почему? Есть же гораздо более корректный метод обращения к таблице

$modx->getTableName('msOrderAddress');

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

4. Использовать getChunk я бы рекомендовал так:

$pdoTools = $modx->getService('pdoTools');
$pdoTools->getChunk('name', $params);

Этот вариант, в отличие от $modx->getChunk обработает конструкции и MODX и феном.

5. Не ошибка. Даже не предупреждение. Скорее глаз режет.
Использование прямых SQL конструкций.

Мы же в MODX работаем. Желательно использовать конструктор запроса. newQuery — его не просто так придумали. Помимо универсальности запроса, не привязанного к конкретной СУБД — это еще и выглядит красиво, аккуратно, профессионально. Сразу видно, что человек знает свое рабочее место.

Так то во втором комментарии это уже написано.

От компонентов зависит. сам MODX3 прекрасно работает на 8.1
Я сейчас делаю модуль интернет-магазина MiniShop3 на 8.1 — ничего нигде не пукает

или хотя бы обновиться до последней версии (2.8.6)

В связи с последними событиями… такое себе

А как эти товары потом разделить обратно, если соединил по ошибке? Разрешать делать что-то, без возможности отмены, мне кажется несовсем правильным.

Ну удаление из корзины у нас тоже всегда было без confirm. И ничего страшного. На данный момент то что сделано — это в любом случае прогресс. Сообщество у нас умное, накидает идей и правок, было бы куда.

Вы думаете что в ядре MODX на каждую системную настройку прям свой отдельный код есть? Это немного не так устроено.

Тот случай когда не нужно торопиться обновлять софт, просто потому что новое — лучше.
На всякий случай уточню. Большинство русскоязычных компонентов основаны на единой зависимости — библиотеке pdoTools. Она не заявлена в поддержке PHP8. В целом работает конечно, но ее никто, включая текущего владельца не тестировал основательно. Возможны проблемы как с самой библой, так и с отдельными компонентами.

Спасибо ))

Отличий хватает.

• Другое расположение элементов ядра, например моделей, процессоров
• Другие процессоры в принципе
• Обязательное использование namespace при обращении к классам (и при их создании)
• По другому работает $modx->getService. Так больше писать нельзя
• Ряд отличий в админке.

Я не видел каких то исчерпывающих инструкций. Но понемногу тут на форуме была информация. В том числе от меня. Мне кажется если очень захотеть то можно разобраться.
Опять же есть готовый компонент-шаблон ModExtra3 — можно пошагово посмотреть как он устроен и чем отличается от modExtra

Я же вам выше написал, что ms2 рассчитан на 7.4
Поддержки версий выше нет и не планируется.

Давай я заберу компонент в сообщество, опубликую исходный код (если его сейчас нет). Сделаешь PR с исправлениями?

Некоторые куски точно не будут работать. Местами есть существенные проблемы. Вплоть до того, что используемый функционал вырезан в версии 8.1
Подогнать конечно можно при желании, но для этого нужна квалификация.

А это как то мешает работе компонента? Ну да, чуть-чуть подправить верстку нужно. И Автор этим точно заниматься не будет. Но как будто в тему поста подобные баги не совсем ложатся. ИМХО.

Этот компонент вполне себе работает. Я им постоянно пользуюсь в разработке MiniShop3. Зачем и как его адаптировать?

Рекомендуемая версия PHP 7.4

В приведенном логе нет не единой ошибки. Про индекс полей только что-то, но это мелкая мелочь

Мне тоже не нравится. Рука всегда в 100% случае в верх тянется за журналом ошибок или настройками

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

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

Не то вставил ))
Исправил ссылку.