Ну вот и покритиковать нельзя :-)

Критика должна быть конструктивной, а твоё пустословие никому неинтересно.

Учитывает ли модуль кол-во товаров, вес товара и габариты?

С версии 2.4.2 учитывает

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

Можно. Для этого нужно указать соответствующие номера тарифов в системных настройках ms_cdek2_tariffs и в ms_cdek2_deliveries.

если я 1 раз выбрал ПВЗ, то потом я не могу уже никак выбрать другой

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

понятно, что карта это доп нагрузка, но можно и в модалку положить — только мнение

Если положить карту в модалку это никак не снизит нагрузку на сеть и не увеличит скорость загрузки страницы.

По умолчанию это происходит при двойном клике на карту

Выбор пункта на карте не интуитивно понятен :-(. При клике на маркер с цифрой лучше чтоб происходило увеличение маштаба карты. А при выборе пвз какое-нибудь сообшение. А то я не сразу понял что пункт выбран.

И плиз напиши плиз все-таки обработку ошибки со строкой в описании доставки. Когда компонент ждет целое.
А то корзина ложиться и не понятно из-за чего. Мы 3 часа пропарились из-за этого.

У нас что-то при установке не прописалось. Раздела minishop2\ms2_delivery_cdekhandler нет в системных настройках. Подскажи что там должно быть?

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

Ну вот и покритиковать нельзя :-).
У нас api-maps.yandex.ru/v3/?apikey=*** заработало когда поставили ограничение по домену.

С msDeliveryProps ошибка

Fatal error: Uncaught Error: Call to a member function getOption() on null in 
/home/s/semweb/orliman.shop/public_html/core/components/minishop2/model/minishop2/msdeliveryhandler.class.php:3 
Stack trace: #0 /home/s/semweb/orliman.shop/public_html/core/components/msdeliveryprops/ConfigurableDeliveryHandler.class.php(12): 
require_once() #1 /home/s/semweb/orliman.shop/public_html/core/cache/includes/elements/modplugin/46.include.cache.php(9): require_once('/home/s/semweb/...') 
#2 /home/s/semweb/orliman.shop/public_html/core/model/modx/modscript.class.php(76): include('/home/s/semweb/...') #3 
/home/s/semweb/orliman.shop/public_html/core/model/modx/modx.class.php(1674): modScript->process(NULL) #4 
/home/s/semweb/orliman.shop/public_html/core/model/modx/modmanagercontroller.class.php(153): 
modX->invokeEvent('OnManagerPageBe...', Array) #5 /home/s/semweb/orliman.shop/public_html/core/model/modx/modmanagerresponse.class.php(79): 
modManagerController->render() #6 /home/s/semweb/orliman.shop/public_html/core/model/modx/modmanagerrequest.class.php(187): 
modManagerResponse->outputContent(Array) #7 /home/s/semweb/orliman.shop/public_html/core/model/modx/modmanagerrequest.class.php(135): 
modManagerRequest->prepareResponse() #8 /home/s/semweb/orliman.shop/public_html/manager/index.php(63): 
modManagerRequest->handleRequest() #9 {main} thrown in /home/s/semweb/orliman.shop/public_html/core/components/minishop2/model/minishop2/msdeliveryhandler.class.php on line 3

Я в файле core\components\minishop2\model\minishop2\msdeliveryhandler.class.php
Закоментировал $this->modx->getOption('log_deprecated')

<?php

// if (!empty($this->modx->getOption('log_deprecated'))) {
    // $this->modx->log(
        // xPDO::LOG_LEVEL_ERROR,
        // 'Deprecated: use handlers from catalog core/components/minishop2/handlers/'
    // );
// }
require_once dirname(__FILE__, 3) . '/handlers/msdeliveryhandler.class.php';

И заработало. Не понятно Эта ошибка из-за наших особенностей или что-то неправильно в msdeliveryhandler. По идее файл же не класс и $this-> вообще не должен присутствовать О_О.

Учитывает ли модуль кол-во товаров, вес товара и габариты?
Можно ли выбрать разные способы, например экспресс или обыная?
Также может в демо не обновлена версия, но у меня работает пока немного странно: если я 1 раз выбрал ПВЗ, то потом я не могу уже никак выбрать другой, даже после обновления страницы. Ну и не всегда удобно выбрать без карты (понятно, что карта это доп нагрузка, но можно и в модалку положить — только мнение).

В кабинете яндекса написано такое:

Написано, ага, только я когда добавил ограничения по домену ключ перестал работать.

В описании системной настройки ms_cdek2_yandex_map_api_key написано какой API и где

Код тарифа теперь зачем-то в настройке описания тарифа.

Установи msDeliveryProps и прописывай там

Понять, где должна происходить инициализации ymaps3, не получилось.

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

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

В кабинете яндекса написано такое:
Если вы используете Java Script API версии 3.0, необходимо настроить ограничения для API-ключа, чтобы API работали корректно. В предыдущих версиях это не обязательно, но рекомендуется сделать, чтобы защитить ключ.

Попробуйте прописать ограничения, может в этом причина.
yandex.ru/maps-api/docs/js-api/limit.html

Понять, где должна происходить инициализации ymaps3, не получилось. Баг разработчика какой-то

Или неправильная настройка. У нас тут пишет invalid api key. В яндексе много апи. Какое надо? Мы пробовали JavaScript API и HTTP Геокодер

3 часа веселия Артур нам обеспечил :-). Код тарифа теперь зачем-то в настройке описания тарифа.

Без нее корзина ложиться наглухо. Даже если никакие доставки сдек не включены.
Теперь буду знать, что в степени бардака в програмировании нам есть куда стремиться :-).

Только карта ПВЗ так и не заработала. В консоле ошибки


Понять, где должна происходить инициализации ymaps3, не получилось. Баг разработчика какой-то.
@Артур Шевченко поправь плиз.

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

Пока ни один в теме не сказал, что у него сайт заразился на каком-то другом хостинге в эти дни.
У меня на поддержке больше 5 сайтов с разными версиями modx и разными дополнениями, везде все ок, только один на таймвебе заразился.
Причем после чистки этого сайта, я ничего абсолютно не обновлял, а заражений новых с того момента нет — если бы была дырка или уязвимость в modx и его компонентах — почти наверняка уже бы снова заразили.

Да, конечно. Разве что кроме архива core.transport.zip.

Смысл же в том, чтобы сравнить свои файлы с эталонными на предмет изменений. Файлы ядра меняться не должны.

Да, ещё добавлю момент важный, с которым я столкнулся в самом начале, после установки модуля. В системных настройках MiniShop3 есть параметр ms3_frontend_assets, в нём указан перечень css/js файлов что будут подключены на страницах сайта. Собственно, в этом списке есть опечатка/ошибка, которая не позволяет добавить эти ссылки скриптом. По умолчанию этот список выглядит вот так:

[
	"[[+cssUrl]]web\/lib\/izitoast\/iziToast.min.css",
	"[[+jsUrl]]web\/lib\/izitoast\/iziToast.js",
	"[[+jsUrl]]web\/ms3.js",
	"[[+jsUrl]]web\/modules\/hooks.js",
	"[[+jsUrl]]web\/modules\/form.js",
	"[[+jsUrl]]web\/modules\/request.js", 
	"[[+jsUrl]]web\/modules\/cart.js",
	"[[+jsUrl]]web\/modules\/customer.js",
	"[[+jsUrl]]web\/modules\/order.js",
	"[[+jsUrl]]web\/modules\/message.js",
]

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

[
	"[[+cssUrl]]web\/lib\/izitoast\/iziToast.min.css",
	"[[+jsUrl]]web\/lib\/izitoast\/iziToast.js",
	"[[+jsUrl]]web\/ms3.js",
	"[[+jsUrl]]web\/modules\/hooks.js",
	"[[+jsUrl]]web\/modules\/form.js",
	"[[+jsUrl]]web\/modules\/request.js", 
	"[[+jsUrl]]web\/modules\/cart.js",
	"[[+jsUrl]]web\/modules\/customer.js",
	"[[+jsUrl]]web\/modules\/order.js",
	"[[+jsUrl]]web\/modules\/message.js"
]

Надеюсь это кому-нибудь поможет.

И ещё. Хочу сказать огромное спасибо разработчикам за проделанную титаническую работу! Очень жду, как и многие, модуль в общем доступе уже в репозиториях, хотя, думаю, из-за объёма работ и параллельных забот, это произойдёт не скоро.