Distributor: фильтр Distributable_post_types не соблюдается на экране Pull UI

Созданный на 13 февр. 2021  ·  5Комментарии  ·  Источник: 10up/distributor

Опишите ошибку
при фильтрации разрешенных типов сообщений с помощью distributable_post_types я заметил, что раскрывающийся список в пользовательском интерфейсе de Pull содержит все типы сообщений в раскрывающемся списке.

Действия по воспроизведению
Отфильтруйте разрешенные типы сообщений:

add_filter( 'distributable_post_types', function() {
    return [ 'post' => 'post' ];
});

Перейдите на страницу обзора Pull и проверьте раскрывающийся список.

Ожидаемое поведение
Я ожидал, что в раскрывающемся списке будет только тип публикации.

Информация об окружающей среде

  • Версия WordPress: 5.6.1

Версия | 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

Дополнительный контекст
Спасибо!

Documentation good first issue help wanted

Все 5 Комментарий

@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, я буду рад его просмотреть. Спасибо!

Я все еще готов к этому, но у проекта «Дистрибьютор», над которым я работаю, есть небольшие проблемы :) когда он жив, у меня появляется немного свободного времени!

Была ли эта страница полезной?
0 / 5 - 0 рейтинги