Форум

Так как мне не понятно как это выглядит и где находится, то отвечу так:

Может быть любой вариант, как модуль, так и частная доработка.

Добавил проверку такую же как в Manager::getById чтобы не вызывалось, правда минус такой что информации о доставке в письме в этом случае может не быть, но это лучше чем ловить исключение и крики клиентов "Аааа, оно сломалось"
[CODE]if(!empty($DELIVERY_ID) && intval($DELIVERY_ID)>0){
     $data['DELIVERY'] = \Bitrix\Sale\Delivery\Services\Manager::getById($DELIVERY_ID);
}else{$data['DELIVERY']=[];}[/CODE]

Я по похожему принципу Подарочные сертификаты делал.
Ну а в целом - как вариант, если нет особых условий для применения. Потому что если на сайте постоянно используются по несколько скидок, то такой способ будет очень неудобен. Простой пример: скидка применяется если не применилась другая скидка(т.е. скидки ограничены на текущем уровне применимости) и получается вам нужно получить все скидки, вычислить и в случае когда не нужно отменить действия каких-то из них. И еще нужно учитывать что в 1С передается цена со скидкой, а у вас скидка идет как часть оплаты и соотвественно в 1С эта информация не идет. В 1С вообще только одна оплата передается, нужно допиливать еще передачу что есть вторая оплата и в 1С на основании этого допиливать скидку чтобы пробить чек. В общем не все так просто с вашим решением.

Но как и сказал если что-то простое на сайте, то вариант рабочий.

Нет. На уровне шаблона вы уже не можете ее исправить.

Раз у вас свой обработчик, то внесите в него блок заполнения свойств, тем более комментарии вы же как-то заполняете. Примерно такой:
[CODE]$propertyCollection = $order->getPropertyCollection();
foreach ($propertyCollection as $property)
   {
       if($property->getField('CODE') == 'NAME'){ $property->setValue($requestArr['NAME']); }
   }[/CODE]

У нас есть периодическая ошибка по которой непонятно что делать.

На событии OnOrderAdd формируется массив для отдельной отправки письма
и на шаге [CODE]$data['DELIVERY'] = \Bitrix\Sale\Delivery\Services\Manager::getById($DELIVERY_ID);[/CODE]получаю ошибку
deliveryId(0)

Перед этим шагом сохраняю в файл значение DELIVERY_ID и вижу в нем цифру, но при этом выводится ошибка.
Делаю повторно создание заказа с теми же товарами, с той же доставкой - нет ошибки.

в Manager::getById есть проверка [CODE]$id = intval($deliveryId);
if($id <= 0)
  throw new SystemException("deliveryId");[/CODE]но почему она то возвращает 0, то норм

По существу подсказать не могу, давно было, да и не долго с этим разбирался, но когда-то сталкивался с похожей реализацией, там ставили редис и через него делали фильтр и соответсвенно структуру каталога. Т.е. инфоблок товаров как был так и был, но структура в нем была тоже плоская. А отдельно шел справочник, настройки где формировали, что хотим увидеть такую-то структуру разделов и условия вывода товаров в таком разделе, так же отдельный справочник на вывод фильтра.
Первоначальная настройка конечно была долгая, но каталог летал. Ну и если что-то нужно было допиливать, приходилось делать отдельные запросы в редис По времени у того прогера который это делал заняло около 4 месяцев все это настроить и допилить дополнительные правки в компонентах.

А разве не проще будет на событие отправки писем завязаться, например на OnBeforeEventSend, ведь в b_event попадают только события отправляемые через ::send.

И тот и другой вариант плох. Да и вообще если вы пришли к задаче такой как у вас где нужно или 10000 правил или внутри 100000 товаров, значит вы не правильно сформулировали задачу. В документации даже приведен пример что если нужно перечислить много товаров, то это делается другим путем - заводится свойство у товаров и в скидке не перечисляются все товары, а указывается что данное свойство должно равняться/не равняться/... и тогда вам не придется 10000 товаров в условие впихивать.

Ну и т.д. вплоть до того что создать нужное свое кастомное условие скидок.

[QUOTE]Переданный нам от клиента архив содержит в себе полную копию сайта (и ядро, и публичную часть). Мы развернули сайт из архива на нашем хостинге, привязали наш домен[/QUOTE]
а так же
[QUOTE]Архив был нам передан как новому агенству, которое будет заниматься сайтом.[/QUOTE]
Очень странно как-то. Зачем-то вы свой новый домен привязали к существующему сайту. Ну да ладно. Обычно если меняется агенство для поддержки проекта передаются доступы к админке, к хостингу. После того как получены доступы они меняются на новые. В этом случае уже старое агенство не получит полный контроль, ну конечно если не осталось дыр/шелов.
То что битрикс ломаный, опасно, но тут вопрос кто, как и для каких целей его ломал. Самый распространенный способ это грубо говоря заморозка срока, никаких вирусов, и тут опасность в том что любое обновление сделает его нерабочим. Запросить архив чтобы он не был взломанным у вас не выйдет. Чтобы это исправить ставите чистую установку с новой лицензией и также отдельно (можно локально, можно на отдельном домене) разворачиваете что вам дали и переносите шаблоны, информацию из БД через внутренние инструменты. Так вы избавитесь от всякого возможного мусора и получите чистый продукт, но это долгая задача, так как что там накручено это тот еще может быть квест.

по поводу многосайтовости почитайте [URL=https://dev.1c-bitrix.ru/learning/course/?COURSE_ID=103&LESSON_ID=281&LESSON_PATH=8799.281]https://dev.1c-bitrix.ru/learning/course/?COURSE_ID=103&LESSON_ID=281&LESSON_PATH=8799.281[/URL]
а если вкратце, то смысл такой что у одного ядра должна быть одна лицензия, а все дополнительные сайты должны подключать это ядро в виде ссылок на него и подключить его можно только в пределах одного хоста. Из-за того что одно ядро = одна лицензия, нельзя создавать 2 домена и на каждый ставить отдельно ядро с тем же ключом(исключение если отметить что это версия для разработки).

[QUOTE]3. И если я разворачиваю один сайт из уже существующего битрикс архива + хочу создать новый сайт на битриксе - как мне убедиться что я буду использовать одну копию программного продукта «1С-Битрикс: Управления сайтом»?[/QUOTE]
ну тут самое простое это - из админки вы будете видеть все файлы всех сайтов если правильно все настроили.