Михаил

как вариант надо перебирать привязанные разделы и вытаскивать с каждого элемента изображения, НО
это некорректное решение с точки зрения нагрузки.
почему бы не создать свойство Фото с типам файл в инфоблоке "Туры", сделать его множественным и настроить под изображения.
в таком случае вы сразу сможете получать массив файлов несколькими способами без лишних запросов и не занимая кучу не нужных таблиц.
инфоблоки не рентабельно создавать под единичные элементы, например под изображения, для этого есть доп.поля или highload-блоки, а иногда эффективнее заводить свои таблицы, но это уже когда middl'ом станите.
каждый инфоблок, в который входит разделы, элементы, доп.свойства, занимает несколько таблиц бд, запросы получаются огромные, все обработки у него тяжеловатые по весу.

у товара есть свойство доступность
в зависимости от настроек магазина можно привязаться к этому свойству

сам разобрался, может кому пригодится
отключил касперского и все заработало

столбец IBLOCK_SECTION_ID не содержится в b_sale_basket
надо через join доставать
соответственно добавляем runtime

'runtime' => [
'ELEMENT' => [
            'data_type' => '\Bitrix\Iblock\ElementTable',
            'reference' => ['=this.PRODUCT_ID' => 'ref.ID'],
            'join_type' => 'left'
        ],
]

а в фильтр добавляем
'=ELEMENT.IBLOCK_SECTION_ID' => 114
или в getMap корзины вроде есть PRODUCT связывающий с товаром, можно попробовать в фильтре без runtime
'=PRODUCT.IBLOCK_SECTION_ID' => 114

наверное order_ajax.js а не *php
вообще sale.order.ajax самый убогий и кривой компонент в Битриксе, по хорошему его переписывают, под себя, при этом сокращая объем в десятки раз.
если же менять верстку в дефолтном, то надо примерно так же курнуть как разработчик этого компонента, и лезть в js, добавлять класс с помощью addClass а в style.css описывать, думаю и местами поля там поменять можно, при условие если разобраться где что описано)

в настройках компонента должно быть поле связанное с первой сортировкой и второй сортировкой, если компонент именно на вывод разделов подключен вне комплексного.
иначе надо смотреть глубже, где описывается метод CNextCache::CIBlockSection_GetList, возможно получится

CNextCache::CIBlockSection_GetList(array('SORT' => array('NAME' => 'ASC'), 'CACHE' => array("MUL.....

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

во-первых, это событие со статичными свойствами, т.е. не универсальное. ты эти свойства завел? они точно имеют указанные ID?
во-вторых, а где ты пытаешься проверить этот код? ты добавил этот обработчик события хотя бы в init.php?
в-третьих, учись делать дебаг, Битрикс тебе укажет что именно не работает.
а вообще судя по вопросу, рекомендую начать изучение Битрикса с более легких вещей, что бы понять структуру и логику, тогда не будешь называть обработчики сниппетами, и такой функционал сможешь сам написать под свой проект.

для таких решений существует API

либо задача выполняется криво, либо не тот язык используется для обработки 500 картинок

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

похоже что js ядра что-то перебивает, попробуй отрубить скрипты подключенные в шаблоне поочередно

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

что касается замечаний не скажу, но если они есть, то их должны объяснить.
Тут скорее дело в другом, и причин может быть много.
сейчас программистов способных что-либо путное писать единицы.
Если задание выполнено и работает, если даже есть замечания, дальше ведут беседу выявляя, почему ты допустил "замечания" и вообще понял ли ты о них.
Очень часто встречается когда среди отбирающих присутствует программист, с которого начинались разработки компании, и который сам не умеет программировать, но с учетом что он когда-то написал для компании "Hello World" ориентир руководства на его мнение. Он может по разным причинам отклонить:
- может он не знает язык на столько, чтобы читать чужой код
- может он не хочет что бы кто-то лучше него разбирающийся работал
- может он вообще не хочет чтобы кто-то еще работал, но руководство дало задачу искать
я таких моментов за свою практику встречал примерно в половине компаний с которыми сталкивался как в поиске работы, так и сотрудничая по другим задачам.

Не стоит обращать внимания на такие компании и, главное, не опускать руки.
за 40к питонщика возьмут любого, даже с "замечаниями", т.к. хороший питонщик на такие копейки просто даже не откликнется.
ищи работу дальше, чтобы нарабатывать опыт. Если задаешь такие вопросы, думаю с таким подходом через годик другой увеличишь зарплатные запросы раза так в 3)

Сейчас эти запросы больше будут полезны в Яндекс Директ)
Тебе дали толчок, что бы вкинул деньги в Яндекс)
Как бы вымогательно не звучало, но по другому повысить трафик сегодня тяжело)