Артур Шевченко

Господа, не ссортесь, право слово программирование штука гибкая и у задачи может быть много решений, в какой-то ситуации лучше одно решение, в какой-то другое. getCollection кушает много памяти это факт, но если ресурсов немного, то данное обстоятельство некритично. Хотя я склоняюсь на сторону Артёма использование процессора сложновато и плохо вписывается в понятие «по-быстрому», я сам новичок и писал для новичков, Артём подсказал, где поправить, чтобы было быстрее и безопаснее получился, на мой взгляд вполне простой и понятный код, неуниверсальный конечно, но такая задача и не ставилась.

Таки согласен, см. в-третьих. Постараюсь написать и такой вариант. Хотя я тот ещё любитель ООП))

А ошибки в консоли браузера есть?

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

Валидатор у вас странный. Он данные берёт из базы, а должен с фронта. Я бы проверял $_FILES на количество файлов.

Ну в вашем случае достаточно в случае изменения селекта ставить в скрытое поле 1 если что-то выбрано и 0 если не выбрано.

Тогда отдельное скрытое поле

вот так

languages[]:required

Но есть одно НО, ошибку всё равно не покажет потому что js ругается на селектор с [].
В таких случаях обычно делают скрытое поле и проверяют его.

Давайте так, кусок кода который первым вы вставили, загружает фото в профиль пользователя и к Tickets отношения не имеет. Второй кусок кода, проверят сколько файлов уже было загружено к тикету. Компонент Tickets насколько мне известно не генерирует системных событий, т.е. плагин написать как minishop2 не получится, придётся править исходники. Я бы сделал копию файла assets/components/tickets/js/web/default.js, изменил системную настройку tickets.frontend_js и на 265 строке перед отправкой формы вставил бы проверку

if( document.getElementById('files').files.length < 3){
                    Tickets.Message.error('Файлов должно быть не меньше 3');
                    return false;
                    }

Так у вас же написано Ошибка при загрузке файлов, т.е. на сервер как бы попадает 3 файла, но с загрузкой одного из них какая-то проблема. Т.е. вам нужно сначала делать проверку на количество, а потом загружать.

<?php
if(!$order_id){return false;}
$pdoTools = $modx->getService('pdoTools');
$orderFields = array('id','num', 'createdon', 
                    'updatedon', 'cart_cost', 
                    'delivery_cost', 'comment', 
                    'status', 'cost');
$orderProductFields = array('id','product_id', 
                            'name', 'count', 
                            'price', 'cost', 'options');
                        
$addressFields = array('city', 'street',
                        'receiver', 'phone',
                        'comment');
$output = array(
    'meta' => [],
    'address' => [],
    'products' => []
    );

$order = $modx->getObject('msOrder', $order_id);
if($order){        
    //заполняем массив основных данных заказа
    for($i = 0; $i < count($orderFields); $i++){
        $output['meta'][$orderFields[$i]] = $order->get($orderFields[$i]); 
    }
    
    //вычисляем срок выполнения заказа
    $output['meta']['duration'] = ceil($output['meta']['cost'] / $modx->getOption('base_cost'));
            
    //заполняем адрес и дополнительные данные
    $address = $order->getOne('Address');
    $properties = $address->get('properties');
    for($i = 0; $i < count($addressFields); $i++){
        if($addressFields[$i] != 'properties'){
            $output['address'][$addressFields[$i]] = $address->get($addressFields[$i]); 
        }else{
            foreach($properties as $name => $prop){
                $output['address'][$name] = $prop; 
            }
        }
    }
    //заполняем продукты
    $products = $order->getMany('Products');
    if($products){
        foreach($products as $key => $product){
            for($i = 0; $i < count($orderProductFields); $i++){
                $output['products'][$key][$orderProductFields[$i]] = $product->get($orderProductFields[$i]); 
            }
        }  
    }
    
    
    //$modx->log(1, 'OUTPUT getUserOrder ' . print_r($output, 1));
    
    if($tpl){
        return $pdoTools->getChunk($tpl, $output);
    }
    //$modx->log(1, print_r($output,1));
    return $output;
}else{
    return false;
}

ФРОНТ

{set $order = '!getOrder' | snippet:[
    'order_id' => $.get.order_id
]}

{if $order}
<p class="h-6"><span class="text-orange">Номер заказа:</span> {$order['meta']['num']}</p>
<p class="h-6"><span class="text-orange">Дата заказа:</span> {$order['meta']['createdon'] | date: 'd.m.Y H:i:s'}</p>
<p class="h-6"><span class="text-orange">Имя заказчика:</span> {$order['address']['receiver']}</p>
{if $_modx->isAuthenticated('mgr') || $.get.admin}
<p class="h-6"><span class="text-orange">Телефон заказчика:</span> <a class="service-link" href="tel:{$order['address']['phone'] | replace:'-':'' }">{$order['address']['phone'] | replace:'-':'' }</a></p>
{/if}
<p class="h-6"><span class="text-orange">Общая стоимость:</span> <span class="jsTotalOrderCost">{$order['meta']['cost']}</span> руб.</p>
<p class="h-6"><span class="text-orange">Срок выполнения:</span> <span class="jsOrderDuration">{$order['meta']['duration']}</span> дн.</p>
<p class="h-6 mb-0"><span class="text-orange">Перечень включенных в состав услуг <small class="small">(в стоимость услуг входит стоимость платных дополнений)</small>:</span></p>
    {if $_modx->isAuthenticated('mgr') || $.get.admin}
        <div class="pl-0" id="productList">
        {set $i = 1}    
        {foreach $order['products'] as $key => $product}
            {set $description = $product['product_id'] | resource: 'desc'}
            {set $introtext = $product['product_id'] | resource: 'introtext'}
            {set $add_cost = $product['product_id'] | resource: 'add_cost'}
            {set $weight = $product['product_id'] | resource: 'weight'}
            {$_modx->parseChunk('@FILE chunks/shop/editOrderProduct.html', [
                'description' => $product['options']['desc']?:$description,
                'introtexxt' => $introtext,
                'index' => $i,
                'add_cost' => $add_cost,
                'price' => $product['price'],
                'weight' => $weight,
                'count' => $product['count'],
                'name' => $product['options']['name']?:$product['name'],
                'cost' => $product['cost'],
                'id' => $product['id'],
                'options' => $product['options'],
                'order_id' => $.get.order_id,
                'total_cost' => $order['meta']['cost'],
                'product_id' => $product['product_id']
            ])}
            {set $i = $i + 1}
        {/foreach}
        </div>
        <form class="row py-3 jsProductRow border-gray-bottom jsAddForm">
            <p class="h-5">Добавить услугу</p>
            <input type="hidden" name="weight" value="1">
            <input type="hidden" name="order_id" value="{$.get.order_id}">
            <input type="hidden" name="total_cost" value="{$order['meta']['cost']}">
            <input type="hidden" name="action" value="editOrder">
            <input type="hidden" name="product_id" value="130">
            <div class="col-lg-7 col-md-5 col-6 d-flex flex-column justify-content-center align-items-start">
                <label class="label-text par mb-0 w-100">
                    <input type="text" class="input pr-1" name="name" placeholder="{130 | resource: 'pagetitle'}" value="" required>
                </label>
            </div>
            <div class="col-lg-1 col-md-1 col-6 d-flex justify-content-center align-items-center">
                <label class="label-text par mb-0">
                    <input type="number" class="input pr-1" min="1" name="count" value="1">
                </label>
            </div>
            <div class="col-lg-2 col-md-3 col-6 d-flex justify-content-center align-items-center">
                <label class="label-text par mb-0">
                    <input type="text" class="input pr-1" name="price" placeholder="Цена" value="" required>
                </label>
            </div>
            <div class="col-lg-2 col-md-3 col-6 d-flex justify-content-center align-items-center">
                <button class="btn btn-orange btn-cart" type="submit">
                    <i class="fa fa-cart-plus"></i>
                </button>
            </div>
            <div class="col-12 order-5 mt-3">
                <label class="label-text par mb-0">
                    <textarea name="options[desc]" class="textarea w-100" rows="4" placeholder="Описание"></textarea>
                </label>
            </div>
        </form>
    {else}
     <ul class="pl-0">
    {foreach $order['products'] as $key => $product}
        {set $description = $product['product_id'] | resource: 'description'}
        {set $introtext = $product['product_id'] | resource: 'introtext'}
        {set $add_cost = $product['product_id'] | resource: 'add_cost'}
        <li class="row py-3 jsProductRow border-gray-bottom">
            <input type="hidden" name="id" value="{$id}">
            <div class="col-lg-7 col-md-5 col-6 d-flex flex-column justify-content-center align-items-start">
                <span class="par service-link mb-3">{$product['options']['name']?:$product['name']}</span>
                <small class="small">{$product['options']['desc']?:$description}</small>
                {if $add_cost}
                    <small class="small text-orange">Добавочная стоимость: {$add_cost} руб.</small>
                {/if}
            </div>
            <div class="col-lg-1 col-md-1 col-6 d-flex justify-content-center align-items-center">
                <span class="price text-orange h-6">{$product['count']} шт.</span>
            </div>
            <div class="col-lg-2 col-md-3 col-6 d-flex justify-content-center align-items-center">
                    <span class="text-orange h-6">{$product['price']} руб.</span>
            </div>
            <div class="col-lg-2 col-md-3 col-6 d-flex justify-content-center align-items-center">
                <span class="text-orange h-6">{$product['cost']} руб.</span>
            </div>
        </li>
    {/foreach}
    </ul>
    {/if}
{else}
<p class="h-4 text-center">Заказ с id = {$.get.order_id} не существует в системе</p>
{/if}
<div class="row">
        <div class="col-lg-3 offset-lg-3 col-sm-6 mt-3">
        <div class="btn-wrap">
            <a href="/" class="btn btn-no-bg w-100">На Главную</a>
        </div>
    </div>
    <div class="col-lg-3 col-sm-6 mt-3">
        <div class="btn-wrap">
            <a href="{9 | url}" class="btn btn-no-bg w-100">К Услугам</a>
        </div>
    </div>
</div>

До первого обновления minishop2

Править исходники плохо.
Лучше написать свой сниппет, примерно такой

$order=$modx->getObject('msOrder', $id);
//Тут как-то обрабатыааете

Вызов такой

[[customGetOrder?id=17]]

Как в синтаксисе pHx получить get параметр не помню, погуглите, а лучше используйте fenom.

А этот код что делает?

[[!msGetOrder? &tpl=`tpl-get-order`]]

Ок, спасибо за разъяснения.

Потому что value="@CODE {$modx->removeCollection('modResource')}"

Понял. а если сделать проверку типа

if(strpos($_POST['tpl'], '@FILE') !== false){
$tpl = $_POST['tpl'];
}

достаточно просто

А если $_REQUEST['minQuery'] будет равна 'три', тут

if(mb_strlen(str_replace(' ', '', $query)) < $minQuery){
    $res = ['success' => false, 'message' => 'Количество символов в запросе должно быть больше ' . $minQuery];
    return json_encode($res);
}

будет ошибка, нет?

[[+modx.user.id:ismember=`Administrator`:then=`
[[!msGetOrder? &tpl=`tpl-get-order`]]
`:else=`Доступ запрещён`]]

Сорри, я правил, у меня в оригинале другой код))) Спасибо что заметили. Исправил, там конечно же должно быть submit.

Вот это очень плохо, вот прям очень. Не нужно прокидывать tpl с фронта, его нужно задавать строго на сервере.

Почему?

Объекты тут вообще не нужны

С одной стороны да, а с другой, изначально я не планировал использовать msProducts, поэтому и получал объекты.

неплохо хотя бы поверхностно обработать

Прям согласен.

$query = strip_tags($_POST['query']); // хватит?

Можно добавить (int).

Тогда уже так

if($_POST['minQuery'] && is_numeric($_POST['minQuery'])){
    $minQuery = (int)$_POST['minQuery'];
}
else{
    $minQuery = 3;
}

Удаляются, вроде бы))