Вывод секций для публикаций Tickets

При создании нового тикета, в форме выводится список секций, доступных для пользователя. Чтобы иметь возможность публиковать тикеты, юзер должен быть авторизован и входить в группу, у которой есть разрешение section_add_children для секций тикетов.
По умолчанию, Tickets выбирает все доступные секции, что не очень удобно, если сайт большой. Поэтому, я написал простенький сниппет, который позволяет указать
  • &parents - список контейнеров, где искать секции тикетов
  • &depth - глубина выборки
  • &tpl - шаблон оформления секции
  • &sortby - сортировка, по умолчанию - "menuindex".
  • &sortdir - направление сортировки, по умолчанию "desc".
  • &where - дополнительные параметры выборки.
Возможно, он войдёт в состав Tickets на постоянной основе, но не факт.
<?php
if (!isset($parents)) {$parents = 0;}
if (!isset($depth)) {$depth = 0;}
if (empty($tpl)) {$tpl = '@INLINE <option value="[[+id]]" [[+selected]]>[[+pagetitle]]</option>';}
if (empty($sortby)) {$sortby = 'menuindex';}
if (empty($sortdir)) {$sortdir = 'asc';}

if (!$modx->getService('pdoFetch')) {return false;}
$pdoFetch = new pdoFetch($modx, array());

$class = 'TicketsSection';
$where = array('deleted' => 0, 'published' => 1, 'class_key' => 'TicketsSection');

// Получаем возможных родителей секций тикетов
if (!empty($parents)) {
    $parents = array_map('trim', explode(',', $parents));
    if (!empty($depth) && $depth > 0) {
        foreach ($parents as $pid) {
            $parents = array_merge($parents, $modx->getChildIds($pid, $depth));
        }
    }
    if (!empty($parents)) {
        $where['parent:IN'] = $parents;
    }   
}

// Определяем столбцы для получения из таблицы
$resourceColumns = array_keys($modx->getFieldMeta($class));
if (empty($includeContent)) {
    $key = array_search('content', $resourceColumns);
    unset($resourceColumns[$key]);
}
$select = array($class => implode(',',$resourceColumns));

// Добавляем дополнительные условия, если есть
foreach (array('where','select') as $v) {
    if (!empty($scriptProperties[$v])) {
        $tmp = $modx->fromJSON($scriptProperties[$v]);
        if (is_array($tmp)) {
            $$v = array_merge($$v, $tmp);
        }
    }
    unset($scriptProperties[$v]);
}

// Собираем всё в кучу
$default = array(
    'class' => $class,
    'where' => $modx->toJSON($where),
    'select' => $modx->toJSON($select),
    'sortby' => $sortby,
    'sortdir' => $sortdir,
    'limit' => 0,
    'return' => 'data',
    'checkPermissions' => 'section_add_children'
);

// Добавляем в конфиг и запускаем
$pdoFetch->setConfig(array_merge($default, $scriptProperties), false);
$rows = $pdoFetch->run();
$output = '';

// Перекомпонуем массив, чтобы id секций были его ключами
$sections = array();
foreach ($rows as $v) {
    $sections[$v['id']] = $v;
}

// Если это форма редактирования тикета - нужно получить id его секции
$cid = 0;
if (!empty($_REQUEST['tid']) && $current = $modx->getObject('Ticket', $_REQUEST['tid'])) {
    $cid = $current->get('parent');
    // Если текущая секция не была выбрана - нужно добавить отдельно
    if (!isset($sections[$cid]) && $parent = $modx->getObject('TicketsSection', $cid)) {
        $sections[$cid] = $parent->toArray();
    }
}

// Оформляем секции в чанк
foreach ($sections as $row) {
    $row['selected'] = ($cid == $row['id']) ? 'selected' : '';
    $output .= $pdoFetch->getChunk($tpl, $row);
}

return $output;
Требуется версия pdoTools не ниже 1.7.0-beta2 - она умеет проверять права на доступ к ресурсам.

7 комментариев

Алексей Карташов
Требуется версия pdoTools не ниже 1.7.0-beta2 — она умеет проверять права на доступ к ресурсам.
Уже?? Вот это скорость! Мне б с такой же скоростью разрабатывать =)
Василий Наумкин
Ну я умничать не стал, сделал проверку методами MODX - джойнить таблицы в один запрос не выйдет.
Недостатка 2: - Немного замедляется работа. Но это нивелируется возможностью включать\отключать проверку прав. - В выборке pdoResources, с разбивкой на страницы, могут пропадать записи. Типа лимит был 10, а вывело 9, потому что один ресурс не прошел проверку.
Но, в отличии от getResources, у меня хотя-бы плейсхолдер [[+total]] покажет верное значение, потому что при исключении документа из выборки ему делается -1.
Поэтому проверка прав остаётся на совести разработчика. Если не надо - не включаем. Если выводим меню, или личный кабинет, то:
[[!pdoResources?
    &checkPermissions=`view`
]]
По умолчанию, понятно, это отключено.
Алексей Карташов
Ну вообще, по хорошему надо проверять не 'view', а 'list'. И нужным группам ресурсов назначать (или нет), соответственно, тоже 'list'-разрешение.
Потому что сам modx в своих getlist-процессорах проверяет именно 'list'. (это я пишу для тех, кто потом будет бездумно код копировать)). 'View' проверяется при просмотре объекта (в get-процессоре).
p.s. да по-любому можно как-то хитро к запросу приджойнить нужные условия. Надо будет на досуге погуглить-поэкспериментировать. Главное правильно запрос написать, а уж в pdoTools встроить будет делом техники =)
Василий Наумкин
Это ты уж сам решай - указывать можно любые разрешения, даже через запятую.
Проверяться будет выбираемый ресурс, что ты к нему приджойнишь - не важно.
Виталий Князь
Теперь бы еще в Tickets была возможность добавлять статьи в избранное (как на habrahabr) и тэги, иначе спустя какое-то время день со днем не сыщешь, хотя помнишь что где-то было на сайте bezumkin.ru.
Василий Наумкин
Когда вернусь к разработке Tickets - сделаю.
Ты подвесь пока задачу на github, чтобы я не забыл.
Николай
Если сниппет вызывать с параметром depth вываливает ошибку (ERROR @ /connectors/resource/index.php) Instantiated a derived class modDocument that is not a subclass of the requested class TicketsSection
Если не указывать,то ищет разделы с тикетами только на уровне родителя, глубже не идет.
bezumkin.ru
Personal website of Vasily Naumkin
Прямой эфир
Василий Наумкин
01.07.2024, 11:56:41
Да, верно, именно так. А в контроллере, скорее всего, ловить данные методом post.
Василий Наумкин
26.06.2024, 09:38:15
О, точно, вылезает если не залогинен. Спасибо, исправил!
Василий Наумкин
09.04.2024, 04:45:01
> Ошибка 500 Это не похоже на ошибку Nginx, это скорее всего ошибка PHP - надо смотреть его логи. ...
Futuris
04.04.2024, 08:56:12
Я просто немного запутался. Когда в абзаце "Vesp/Core" ты пишешь про "новый trait FileModel", я поду...
Василий Наумкин
20.03.2024, 21:21:52
Volledig!
Андрей
14.03.2024, 13:47:10
Василий! Как всегда очень круто! Моё почтение!
russel gal
09.03.2024, 20:17:18
> А этот стоило написать хотя бы затем, чтобы получить комментарий от юзера, который ничего не писал...
Александр Наумов
27.01.2024, 03:06:18
Василий, спасибо! Извини, тупанул.
Василий Наумкин
22.01.2024, 07:43:20
Давай-давай!
Василий Наумкин
24.12.2023, 14:26:13
Спасибо!