Александр Лунегов

Приветствую!
Для начала сто́ит код модификатора then написать в кавычках:

[[+new:isnot=``:then=`
<img src="{$file['medium']}" alt="{$file['description']}" title="{$file['name']}" class="item new">
`:else=`
    [[+old_price:isnot=``:then=`
<img src="{$file['medium']}" alt="{$file['description']}" title="{$file['name']}" class="item sale">
    `:else=`
<img src="{$file['medium']}" alt="{$file['description']}" title="{$file['name']}" class="item">
    `]]
`]]

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

<img src="{$file['medium']}" alt="{$file['description']}" title="{$file['name']}" class="item[[+new:isnot=``:then=` new`:else=`[[+old_price:notempty=` sale`]]`]]">

Плюс избавились от второго else.

pdoResources по умолчанию выводит tv типа «Список» не обработанным. Что бы его обработать можно добавить эту тэвэшку в параметр &processTVs:

[[pdoResources?
    &processTVs=`param`
]]

Можно вместо имени tv поставить `1`, тогда будут обработаны все, что указаны в параметре &includeTVs.
Список будет обрабатываться как указано в параметрах вывода. По умолчанию разделителя не будет.

Другой вариант — воспользоваться модификатором вывода строки:
[[+tv.param:replace=`||==,`]] или [[+tv.param:stripString=`||`]]

Придется использовать свой сниппет.
Для этого код вывода категории переносим в отдельный чанк:

[[pdoMenu?
    &parents=`0`
    &level=`2`
    &tplInner=`@INLINE [[+wrapper]]`
    &tplParentRow=`svoeImyaChanka`
    &countChildren=`1`
]]

Чанк svoeImyaChanka:

<li [[+classes]]><a href="[[+link]]" [[+attributes]]>[[+menutitle]]</a> ([[childrenCounter? &id=`[[+id]]`]])</li>[[+wrapper]]

Сниппет childrenCounter:

<?php
/**
 * childrenCounter
 *
 * DESCRIPTION
 *
 * Сниппет выводит количество дочерних ресурсов для ресурса с указанным id.
 *
 * PROPERTIES:
 *
 * &id целое число. Default: ID текущего ресурса
 *
 * USAGE:
 *
 * [[childrenCounter? &id=`7`]]
 *
 */
$id = (int) $modx->getOption('id', $scriptProperties);
if (!isset($scriptProperties['id'])) {
    $id = $modx->resource->get('id');
}
$q = $modx->newQuery('modResource');
$q->where(array(
   'parent' => $id,
   'deleted' => 0,
   'published' => 1,
   'hidemenu' => 0
));
$counter = $modx->getCount('modResource', $q);
return $counter;

[[*TVname:mpy=`1.15`]]

Сергей Миллер, к сожалению в MODX нет решения «из коробки» для показа онлайн ли пользователь. Так как у Вас желание сделать именно через API, а не сторонними дополнениями (UsersOnline), то придется постараться. Для начала надо определить, что значить «пользователь онлайн»? Ведь могут быть различные варианты поведения. Пользователь открыл страницу сайта и отошел от компьютера, не закрывая браузера, или переключился на другую вкладку. Он онлайн? Или открыл страницу и сразу закрыл браузер, как узнать что он уже не онлайн. Самый простой и наверное самый распространенный способ, это хранить в базе данных точное время последней загрузки любой страницы пользователем и считать, что если это произошло недавно (минут 5 или 10 назад), то пользователь еще активен.
Для этого создадим плагин, например userLastAction и подключим ему системное событие OnLoadWebDocument (код отчасти взят у Василия Наумкина):

<?php
switch ($modx->event->name) {
	case 'OnLoadWebDocument':
		// Сохраняем дату открытия любой страницы сайта, если пользователь авторизован
		if ($modx->user->isAuthenticated($modx->context->key)) {
			// Здесь мы работаем с текущим пользователем — у него профиль уже загружен
			$profile = $modx->user->Profile;
			// Сохраняем дату и время в поле факс (а зачем еще это поле?) профиля пользователя
			$profile->set('fax', date('Y-m-d H:i:s'));
			$profile->save();
		}
		break;
}

Теперь при открытии любой страницы сайта любым пользователем будет сохранятся точное время этого действия, привязанное к профилю пользователя. Остается только при формировании списка проверять, давно ли он пользователь совершал действие на сайте.
Я например писал такой сниппет userOnline:

<?php
$output = '';
// Если есть ID пользователя
if ($input) {
    $userId = (int)$input;
    // Если в параметрах не установлено через какое количество секунд считать пользователя "офлайн", то берем 10 минут
    $idleTime = $options ? (int)$options : 600;
    // Получаем объект пользователя по ID
    $user = $modx->getObject('modUser', $userId);
    if (!empty($user)) { // Если пользователь существует, то...
        // Получаем его профиль
        $profile = $user->getOne('Profile');
        // Получаем время его последнего действия
        $userLastAction = $profile->get('fax');
        if (!empty($userLastAction)) {
            // Если есть время в базе, то проверяем меньше ли оно установленного нами времени признания пользователя активным
            if (date('U') - date('U', strtotime($userLastAction)) < $idleTime) {
                // Если да, будем выводить слово "online"
                $output = 'online';
            } else {
                // Если нет, то — время его последнего действия
                $output = $userLastAction;
            }
        }
    }
}
// Выводим результат
return $output;

Вам придется это еще переработать под себя.

Если известно, что пользователи состоят только в этой группе или это их главная группа, то тут всё легко.

$groupName = 'boys';
//Получаем ID группы
$groupId = $modx->getObject('modUserGroup', array('name' => $groupName))->get('id');

//Получаем коллекцию объектов пользователей с выборкой по группе
$users = $modx->getCollection('modUser', array('primary_group' => $groupId));

foreach ($users as $user) {
    
}

А вот если пользователи могут состоять в нескольких группах и список нужен не по главной группе, то тут немного сложнее...

$groupName = 'boys';
//Получаем ID группы
$groupId = $modx->getObject('modUserGroup', array('name' => $groupName))->get('id');

//Получаем коллекцию объектов групп пользователей с выборкой по группе
$userGroupMembers = $modx->getCollection('modUserGroupMember', array('user_group' => $groupId));

//прогоняем коллекцию через цикл
foreach ($userGroupMembers as $userGroupMember) {

//Получаем объект пользователя через объект группы
    $user = $userGroupMember->getOne('User');

//Делаем с объектом пользователя, что пожелаем...
}

Попробуйте лучше такой вариант, он правильный и очень лаконичный:

<a href="[[+tv.file-download]]" class="btn btn-success btn-sm[[+tv.file-download:empty=` disabled" tabindex="-1" aria-disabled="true`]]"><i class="fas fa-download"></i> Скачать</a>

Сделайте вызов сниппета некэшируемым:

[[!getImageList? 
        &tvname=`gallery`
        &docid=`[[+id]]`
        &tpl=`@CODE:
        <div class="item">
          <img src="[[+image]]" alt="" width="844" height="531">
        </div>`
        &tvname=`img-slide`
        ]]

Можно попробовать таким образом.
Сначала сниппетом getImageList создаем список id ресурсов через запятую:

[[!getImageList?
    &tvname=`migx_recommended_brands`
    &tpl=`@CODE: [[+brand]]`
    &toPlaceholder=`kakoytoplaceholder`
    &outputSeparator=`,`
]]

Инструкция по параметрам getImageList.

А потом используем для вывода сниппет pdoResources, который отфильтрует неопубликованные и удаленные.

[[!pdoResources?
    &parents=`0`
    &depth=`0`
    &resources=`[[+kakoytoplaceholder]]`
    &tpl=`recommended_service`
    &includeTVs=`preview_image`
]]

Чанк recommended_service:

<div class="col-md-6 mb-base col-xl-3" style="">
    <a class="brand" href="[[~[[+id]]]]">
        <span class="brandtext">[[+pagetitle]]</span>
        <img class="img-fluid" src="[[+tv.preview_image:pThumb=`&w=250&h=200&zc=0`]]" alt="">
    </a>
</div>

Но, если бы это был мой проект, то я бы прежде попробовал написать свой сниппет, который сначала проверял TV migx_recommended_brands: удалял бы там id несуществующих ресурсов, отфильтровывал неопубликованные и самостоятельно делал бы вывод.

Возможно проблема в фильтре вывода.
Попробуйте эту конструкцию

[[+tiles:isnot=``:then=`
  <div class="tiles-block">
    [[getImageList?
      &value=`[[+tiles]]`
      &tpl=`tiles-tpl`
    ]]
  </div>
`]]

поменять на

[[getImageList?
    &value=`[[+tiles]]`
    &tpl=`tiles-tpl`
    &wrapperTpl=`@CODE:<div class="tiles-block">[[+output]]</div>`
]]