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

modx.pro/news/24265

Причем низко в ноги.

Аминь, или как говорят в ваших краях Adiós Amigo ))

Нет. Автор немного схитрил и не указал в коде саму отправку SMS. Он просто оставил комментарий // ОТПРАВКА КОДА В СЕРВИС РАССЫЛКИ СМС

По факту там будет подключение к оператору и отправка SMS

Для такого есть раздел избранное

Я имел в виду как раз после неудавшейся установки свежей версии, пробовали ее установить еще раз?

А MODX какой?

А без обновления эта копия работает? Никаких проблем нет?
Пробовали повторную установку MiniShop2?
Какое точно рабочее окружение используете?

С Василием, по поводу VESP нужно разговаривать на его профильном форуме, где он отвечает на подобные вопросы. bezumkin.ru/

Сделай, чтобы без авторизации работало. За что только деньги платим

Там нет отдельного архива — он формируется автоматом в момент нажатия кнопки.

Спасибо за обратную связь. Расскажи откуда ты берешь zip и как понял что он не рабочий?

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

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. В целом работает конечно, но ее никто, включая текущего владельца не тестировал основательно. Возможны проблемы как с самой библой, так и с отдельными компонентами.

Спасибо ))