Funktioniert nicht bei Upload per Drag & Drop in Gutenberg bzw. über REST API about de-de HOT 17 CLOSED

@goaround nun nachgeholt, MGM hat einen neuen Tag.

from de-de.

@bueltge hast du etwas Zeit dir das mal etwas anzuschauen?

from de-de.

Hallo @goaround
Das plugin wird aktuell ohne Hook inititiert, da es als Dropin daher kam und dies nicht nötig war. Damit rennt es zu früh los und die REST API bekommt die Korrektur der Dateien nicht mit. Heißt, die Konstanten Abfragen am ersten if-Statement sind nie true, wenn man REST_REQUEST einbaut. Wir müssen umbauen, das Plugin bei plugins_loaded oder admin_init laufen lassen und parallel die Dateinamen Anpassung ebenso bei rest_api_init inititieren und bei wp_handle_upload_prefilter laufen lassen.

from de-de.

@goaround Ich habe die Lösung komplett neu geschrieben, da wir so triggern können und die REST API auch angesprochen werden kann. Kannst du das php File aus dem master-Branch testen, also Dropin/Plugin für alle Funktionen? In einem ersten Test Upload Files hat alles geklappt. Setzen des Feed-Sprach Key klappt auch.

from de-de.

@bueltge direkt getestet und der erste Test sah auf jeden Fall sehr gut aus. Zwar wird aus dem Titel auch die Umlaute entfernt. Ich denke, das ist aber zu verschmerzen.

Super, dass du dir die Mühe gemacht hast! Wirklich ein sehr nützliches Plugin. So muss ich nicht jedem Autor erklären, dass er ja alle Umlaute aus dem Dateinamen vor dem Upload entfernen soll. Klappt leider immer nur zu 80%...

from de-de.

Sollen den im Titel die Umlaute bleiben, also im entstandenen Permalink? Der Title des Artikels oder andere CPT dürfen nicht verändert werden.

from de-de.

Wenn ich ein Bild süße Katzen.jpg hochlade, wäre es schon cool, wenn der Dateiname als Titel erhalten bleibt.

Wenn ich das direkt im Media Model mache, bleibt der Titel ja genau so erhalten. Per Drag & Drop in Gutenberg wird daraus dann suesse katzenbilder. Die Groß- und Kleinschreibung geht leider auch verloren.

Der Permalink sollte natürlich ohne Umlaute sein.

from de-de.

Der Title wird aber nur so, wenn du das via Gutenberg, also REST API, tust - richtig? Sind sonst alle Funktionen sauber, wo man Bilder hochladen kann bzw. Permalink-Anpassung?

from de-de.

Genau, sonst funktioniert alles sauber.

from de-de.

Ich habe diverse Sachen untersucht, aber das Ablegen des File passiert bei _wp_handle_upload() und darin habe ich nur File, ein Array mit dem Namen. Der 'post' wird via ' wp_insert_post' erzeugt. Wir könnten ggf. nur die ID des Attachments holen und dann den post für dieses Attachment neu schreiben, also einen ganzen Query mehr.

from de-de.

Wie oben bereits geschrieben: Ich denke, das kann man verschmerzen. Hauptsache es sind keine Umlaute im Dateinamen/Slug

from de-de.

Da es eine „Performance“ Frage ist, könnte man den aktuellen Stand lassen und eine Konstante abfragen um den komplexeren Weg zu aktivieren. Oder deaktivieren.

Wenn man den. Zeit hat den noch einzuführen

from de-de.

Aktuell ist der Permalink noch ohne Korrektur, wenn man via Gutenberg (REST API) Posts erstellt. Da will ich noch ran. Gibt es sonst noch Themen wo ggf. die alte Logik ohne REST API nicht mehr greift? Leider sind die Hooks zw. Classic und Gutenberg sehr unterschiedlich.

from de-de.

@goaround Ich habe nun einen finalen Stand, würdest du nochmal rundherum testen.
An den Title via Rest komme ich aktuell nicht ran, ggf. findet da jemand was
Ebenso bei dem Permalink im Gutenberg, der wird via AJAX gesetzt, generiert aus dem Title. Ich kann den zwar in der DB anpassen, dann aber mit Aufwand, da bereits 'sanitized' via WP, bspw. von ä nach a etc. Das wird schwer. Für den Eingriff in den AJAX Part fehlt mir das Wissen, ggf. auch jemand anderes.

from de-de.

Bin alle Möglichkeiten durchgegangen und konnte keinen Fehler mehr finden. Freue mich sehr auf das Update (wenn du schon dabei bist gerne auch einen neuen Release von Multisite Global Media mit meinem Fix für die REST API) :)

from de-de.

Version 1.0.0 ist da und auch auf wp.org hinterlegt.

@goaround MGM - deinen PR habe ich bereits gemergt, nun wäre ein Tag und zip gut, das meinst du? Da es ja nicht auf wp.org abgelegt ist.

from de-de.

@bueltge genau, ich lade MGM über Composer, würde mich über 0.1.1 freuen :)

from de-de.