Distributor: Der distributable_post_types-Filter wird im Pull-UI-Bildschirm nicht berücksichtigt

Erstellt am 13. Feb. 2021 · 5Kommentare · Quelle: 10up/distributor

Beschreibe den Fehler
Beim Filtern der erlaubten Beitragstypen über distributable_post_types ist mir aufgefallen, dass das Dropdown in der Pull-UI alle Beitragstypen im Dropdown enthält.

Schritte zum Reproduzieren
Filtern Sie die zulässigen Beitragstypen:

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

Gehen Sie zur Pull-Übersichtsseite und überprüfen Sie das Dropdown-Menü.

Erwartetes Verhalten
Ich habe nur den Post-Post-Typ in der Dropdown-Liste erwartet.

Umweltinformationen

WordPress-Version: 5.6.1

Version | 1.6.2

Autor-Byline überschreiben: Ja

Umgang mit Medien: Ausgewähltes Bild und angehängte Bilder

Interne Verbindungen |
Maas Wijkontwikkeling: - Blog ID: 2 - URL: https://waal.local/maasontwikkel - Registriert: 2021-02-01 13:52:41 - Zuletzt aktualisiert: 2021-02-13 17:09:18
Woonslim: - Blog-ID: 3 - URL: https://waal.local/woonslim - Registriert: 2021-02-01 13:52:56 - Zuletzt aktualisiert: 2021-02-03 08:33:12

Externe Anschlüsse | N / A

Zusätzlicher Kontext
Dankeschön!

Documentation good first issue help wanted

Quelle

jmslbam

Alle 5 Kommentare

@jmslbam Es gibt tatsächlich einen anderen Filter, um die zum dt_available_pull_post_types : https://github.com/10up/distributor/blob/develop/includes/utils.php#L257

Ich kann die Verwirrung zwischen diesem Filter und distributable_post_types verstehen. Einer wird zum Pushen von Inhalten verwendet, der andere zum Abrufen von Inhalten.

dkotter am 15. Feb. 2021

Beschriften Sie dies für Docs, damit wir dem Docblock auf distributable_post_types mehr Klarheit verleihen und das Push-versus-Pull-Szenario hoffentlich etwas klarer machen können.

jeffpaul am 16. Feb. 2021

Ah danke für den Hinweis auf den anderen Filter!

Ich habe das Gefühl, dass distributable_post_types (oder ein neues dt_distributable_post_types ) der allgemeine Gesamtfilter sein kann und ein neues dt_distributable_pull_post_types und ein neues dt_distributable_push_post_types der Filter für jedes Szenario sein können .

Dies macht die Filter in ihrer Namenskonvention und Logik konsistent. Ohne das Wort "drücken" oder "ziehen" sieht es wie ein generischer Filter aus, der in beide Richtungen funktioniert.

Wir können die alten Filter mit do_action_deprecated oder apply_filters_deprecated in WP ablehnen.

Wenn sich diese neue Namenskonvention in Ordnung anfühlt, kann ich diesen Monat irgendwo eine PR zusammenstellen.

jmslbam am 16. Feb. 2021

@jmslbam Ich denke, das klingt alles richtig und der richtige Ansatz für mich. Wenn Sie Zeit haben, eine PR zu erstellen, würde ich diese gerne überprüfen. Vielen Dank!

dkotter am 19. Feb. 2021

👍1

Ich bin immer noch bereit dafür, aber das Vertriebspartnerprojekt, an dem ich arbeite, hat ein paar kleine Probleme :) Wenn es live ist, habe ich etwas Freizeit!

jmslbam am 20. Apr. 2021

❤1

War diese Seite hilfreich?

0 / 5 - 0 Bewertungen

Distributor: Der distributable_post_types-Filter wird im Pull-UI-Bildschirm nicht berücksichtigt

Alle 5 Kommentare

Verwandte Themen