Опишите ошибку
при фильтрации разрешенных типов сообщений с помощью distributable_post_types
я заметил, что раскрывающийся список в пользовательском интерфейсе de Pull содержит все типы сообщений в раскрывающемся списке.
Действия по воспроизведению
Отфильтруйте разрешенные типы сообщений:
add_filter( 'distributable_post_types', function() {
return [ 'post' => 'post' ];
});
Перейдите на страницу обзора Pull и проверьте раскрывающийся список.
Ожидаемое поведение
Я ожидал, что в раскрывающемся списке будет только тип публикации.
Информация об окружающей среде
Версия | 1.6.2
Переопределить авторскую строку: Да
Работа с медиа: рекомендуемое изображение и прикрепленные изображения
Внутренние соединения |
Маас Вийконтвиккелинг: - Идентификатор блога: 2 - URL: https: //waal.local/maasontwikkel - Зарегистрирован: 01.02.2021 13:52:41 - Последнее обновление: 13.02.2021 17:09:18
Woonslim: - Идентификатор блога: 3 - URL: https: //waal.local/woonslim -
Внешние соединения | N / A
Дополнительный контекст
Спасибо!
@jmslbam На самом деле существует другой фильтр для изменения типов сообщений, доступных для извлечения: dt_available_pull_post_types
: https://github.com/10up/distributor/blob/develop/includes/utils.php#L257
Я могу понять путаницу между этим фильтром и distributable_post_types
. Один используется при отправке контента, другой - для извлечения контента.
Обозначим это для документов, чтобы мы могли внести больше ясности в блок документов на distributable_post_types
и, надеюсь, сделать сценарий push vs. pull немного понятнее.
Ах, спасибо, что указали на другой фильтр!
Мне кажется, что distributable_post_types
(или новый dt_distributable_post_types
) может быть общим общим фильтром, а новый dt_distributable_pull_post_types
и новый dt_distributable_push_post_types
могут быть фильтром для каждого сценария. .
Это делает фильтр единообразным по соглашению об именах и логике. Отсутствие в нем слов «толкать» или «тянуть» делает его похожим на общий фильтр, действующий в обоих направлениях.
Мы можем отказаться от старых фильтров с помощью do_action_deprecated
или apply_filters_deprecated
в WP.
Если это новое соглашение об именах покажется мне нормальным, я могу исправить где-нибудь в этом месяце PR.
@jmslbam Я думаю, что все это звучит правильно и является правильным для меня подходом. Если у вас есть время составить PR, я буду рад его просмотреть. Спасибо!
Я все еще готов к этому, но у проекта «Дистрибьютор», над которым я работаю, есть небольшие проблемы :) когда он жив, у меня появляется немного свободного времени!