Distributor: El filtro distributable_post_types no se respeta en la pantalla Pull UI

Creado en 13 feb. 2021 · 5Comentarios · Fuente: 10up/distributor

Describe el error
Al filtrar los tipos de publicaciones permitidas a través de distributable_post_types , noté que el menú desplegable en la interfaz de usuario de Pull tiene todos los tipos de publicaciones en el menú desplegable.

Pasos para reproducir
Filtra los tipos de publicaciones permitidas:

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

Vaya a la página de descripción general de Pull y consulte el menú desplegable.

Comportamiento esperado
Solo esperaba el tipo de publicación Publicar en el menú desplegable.

Información medioambiental

Versión de WordPress: 5.6.1

Versión | 1.6.2

Anular el byline del autor: Sí

Manejo de medios: imagen destacada e imágenes adjuntas

Conexiones internas |
Maas Wijkontwikkeling: - ID de blog: 2 - URL: https: //waal.local/maasontwikkel - Registrado: 2021-02-01 13:52:41 - Última actualización: 2021-02-13 17:09:18
Woonslim: - ID de blog: 3 - URL: https: //waal.local/woonslim - Registrado: 2021-02-01 13:52:56 - Última actualización: 2021-02-03 08:33:12

Conexiones externas | N / A

Contexto adicional
¡Gracias!

Documentation good first issue help wanted

Fuente

jmslbam

Todos 5 comentarios

@jmslbam En realidad, hay un filtro diferente para modificar los tipos de publicaciones disponibles para extraer: dt_available_pull_post_types : https://github.com/10up/distributor/blob/develop/includes/utils.php#L257

Puedo entender la confusión entre ese filtro y distributable_post_types . Uno se utiliza para enviar contenido y el otro para extraer contenido.

dkotter en 15 feb. 2021

Etiquetar esto para documentos para que podamos agregar más claridad al docblock en distributable_post_types y, con suerte, hacer que el escenario de empujar contra tirar sea un poco más claro.

jeffpaul en 16 feb. 2021

¡Ah, gracias por señalar el otro filtro!

Siento que distributable_post_types (o un nuevo dt_distributable_post_types ) puede ser el filtro general genérico y el nuevo dt_distributable_pull_post_types y un nuevo dt_distributable_push_post_types pueden ser el filtro para cada escenario .

Esto hace que el filtro sea consistente en su lógica y convención de nomenclatura. Al no tener la palabra "empujar" o "tirar", parece un filtro genérico que va en ambos sentidos.

Podemos desaprobar los filtros antiguos con do_action_deprecated o apply_filters_deprecated en WP.

Si esta nueva convención de nombres se siente bien, puedo arreglar un PR en algún lugar este mes.

jmslbam en 16 feb. 2021

@jmslbam Creo que todo esto suena correcto y el enfoque correcto para mí. Si tiene tiempo para armar un PR, me complacerá revisarlo. ¡Gracias!

dkotter en 19 feb. 2021

👍1

Todavía estoy listo para esto, pero el proyecto de Distribuidor en el que estoy trabajando tiene algunos pequeños inconvenientes :) cuando está en vivo, ¡tengo algo de tiempo libre!

jmslbam en 20 abr. 2021

❤1

¿Fue útil esta página

0 / 5 - 0 calificaciones