[Bug] OnDocPublished

[0test]

Описание

Событие OnDocPublished работает странно, если работает.

Как воспроизвести.

Делаем эвент

Event::listen(['evolution.OnDocPublished'], function ($params) {
    evo()->logEvent(404, 1,  $params['docid'], 'OnDocPublished случился');
});

Создаём новый ресурс - эвент не вызывается.
Редактируем существующий неопубликованный ресурс так, чтобы стояла галка "Публиковать". Также нет.

Жмём в дереве правой кнопкой мыши "Опубликовать" - эвент работает.

Наблюдаемое поведение

No response

Ожидаемое поведение

No response

Связанные issue(s)/PR(s)

No response

Версия EvoCMS

Evolution CE 3.1.29

Версия Apache/nginx

No response

Версия PHP

8+

Версия MySQL

No response

Другая информация

No response

[Pathologic]

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

[0test]

Возможно, но какое-то оно непредсказуемое. Т.е. мне надо было отследить именно событие публикации новости. И не важно, как это сделано: поставлена галка "Публиковать" и сохранён материала, просто сохранён материал, опубликован в планировщике.
Задачу решил, конечно, но другими способами.

[helgispbru]

@0test я сейчас переделываю процессоры и document manager, так там вот какая логика:

Создаём новый ресурс - эвент не вызывается.
Редактируем существующий неопубликованный ресурс так, чтобы стояла галка "Публиковать". Также нет.

-- это события OnDocCreate и OnDocEdit (новые названия событий, но смысл понятен), тут просто меняются поля, а published это просто одно из полей

Жмём в дереве правой кнопкой мыши "Опубликовать" - эвент работает.

-- это явное действие "публиковать" (событие OnDocPublish) или "снять публикацию" (событие OnDocUnpublish), тут всё четко, всё вызывается

Таким образом логика не кривая, всё нормально.

По идее можно при создании/редактировании проверять дополнительно поле published и при его переключении дёргать и публикационные события, идея хорошая.

[helgispbru]

Хаха, надо было завтра ответить, чтобы ровно год спустя, поторопился я, поспешил....

[Kolya1222]

Мне кажется, самый простой вариант добавить эти события в save_content.processor.php:
В нем есть switch ($actionToTake) { в самом начале case 'new': вставить вызов

        if ($published == 1) {
            $modx->invokeEvent('OnDocPublished', array(
                'id' => $key,
                'doc' => $resourceArray
            ));
        }

Еще в этом же файле при редактировании case 'edit': (Я добавил после вызова события OnDocFormSave):

        if (!$was_published && $published) {
            $modx->invokeEvent('OnDocPublished', array(
                'id' => $id,
                'doc' => $resourceArray
            ));
        }