Neu Fehler in: JTL Exportformat Google Shopping v2.2.0, Bereich: Exportformat Google Review XML

[Marco Stark]

Hallo liebes Dev-Team, Hey Felix @FMoche

Bei der Einrichtung des Review Feeds im Google Merchant Center ist unser Feed mit mehreren Fehlern als "invalid" eingestuft, respektive abgelehnt worden. Bei der Überprüfung der ursächlichen "GoogleReviewXML.php" im Ordner "jtl_google_shopping/Exportformat/" ist mir ein Fehler aufgefallen. Weiterhin gibt es Verbesserungspotenzial im Bereich der XML-Knoten: title, content und product_name.

Fehler
Im Übergang/Zeilenumbruch von Zeile 22 zu 23 ist euch ein Leerzeichen zu viel rein gerutscht. Hinter dem = Zeichen in Zeile 22 sollte direkt der String aus Zeile 23 folgen. Aktuell befindet sich in Zeile 23 aber ein Leerzeichen am Anfang.
Original: https://gitlab.com/jtl-software/jtl...aster/Exportformat/GoogleReviewXML.php#L20-23

Lösung
Um den Code etwas lesbarer zu machen, war ich mal so frech und habe die Schema Namespace/Instance Namespace optimiert sowie den Leerzeichenfehler behoben:
Patch: https://gist.github.com/muemarco/19fce9ab05c0abee18b077b4a3234a22#file-googlereviewxml-php-L20-L23

---

Verbesserungspotenzial (XML-Knoten)
Die Inhalte der folgenden XML-Knoten werden mit "user generated content" befüllt. Hier ergibt sich die Herausforderung, dass speziell in den Knoten title und content von unseren Kunden oft Steuerzeichen in die Produktbewertungen getippelt werden. Das beliebteste ist "&". Das gleiche gilt für den Knoten product_name. Sofern das passiert, validiert der Feed nicht mehr.

Lösungsvorschlag
Das Problem lässt sich mittels CDATA-Abschnitten unkompliziert lösen. Die Datentypen der genannten XML-Knoten sind in der product_reviews.xsd mit "nonEmptyStringType" angegeben, validieren daher erfolgreich gegen die eingefügten CDATA-Abschnitte. Hier die geänderten Zeilen in der Datei "GoogleReviewXML.php":

title-Knoten, Zeile 261
Original: https://gitlab.com/jtl-software/jtl.../master/Exportformat/GoogleReviewXML.php#L261
Patch: https://gist.github.com/muemarco/19fce9ab05c0abee18b077b4a3234a22#file-googlereviewxml-php-L261

content-Knoten, Zeile 262
Original: https://gitlab.com/jtl-software/jtl.../master/Exportformat/GoogleReviewXML.php#L262
Patch: https://gist.github.com/muemarco/19fce9ab05c0abee18b077b4a3234a22#file-googlereviewxml-php-L262

product_name-Knoten, Zeile 308
Original: https://gitlab.com/jtl-software/jtl.../master/Exportformat/GoogleReviewXML.php#L308
Patch: https://gist.github.com/muemarco/19fce9ab05c0abee18b077b4a3234a22#file-googlereviewxml-php-L308

Alternative Lösung via sanitize-Funktion
Das Steuerzeichen & in str_replace hinzufügen und durch & ersetzen. Lösung:

    /**
     * @param string $content
     * @return string
     */
    private function sanitize(string $content): string
    {
        return \str_replace(['>', '<', '&'], ['&gt;', '&lt;', '&amp;'], \html_entity_decode($content));
    }

Falls ihr einen Feed für interne Tests benötigt (unverändertes Shoping Plugin Version 2.2.0), in dem alle nicht erlaubten Steuerzeichen in den Bewertungen verwendet wurden, könnt ihr euch gerne bei uns bedienen (ca. 7.700 Produktbewertungen / 6,4 MB). Den neuen Feed, resultierend aus der gepatchten GoogleReviewXML.php findet ihr an gewohnter Stelle. Upload im Merchant Center sowie XML Validierung waren erfolgreich.

Mit den besten Grüßen,
Marco

 

[FMoche]

Hallo @Marco Stark und vielen Dank für die Hinweise.
Das werde ich so gerne übernehmen, wobei das Sanitizen hier eher komplett unnötig erscheint, oder?
Wenn die Felder in CDATA gewrappt werden, kann ja auch genauso direkt ">", "<" und "&" genutzt werden.
Zumindest validiert das XML dann auch via
xmllint --schema http://www.google.com/shopping/reviews/schema/product/2.3/product_reviews.xsd --noout FILE.xml

Darauf, dass Google hier HTML-Entities erwarten würde, finde ich jedenfalls keinen Hinweis.

 

[Marco Stark]

Hey Felix,

ja, in dem Fall wäre "sanitize" überflüssig.

 

[FMoche]

Dann könntest du die Änderungen aus https://gitlab.com/jtl-software/jtl-shop/plugins/jtl_google_shopping/-/merge_requests/20 testen.

 

[Marco Stark]

Super Felix, vielen Dank. Ich geb dir Feedback, ob alles wie gewünscht funktioniert.

VG,
Marco

 

[Marco Stark]

Guten Morgen Felix,

der Upload des neuen/geänderten Feeds war erfolgreich. Goggle hat ihn vollständig verarbeitet.

 

[FloX]

Aktuell meldet mein Exportformat Google Shopping Plugin laut PHP-Log folgenden Fehler:

Cron-Reschedule-Event-Fehler für Hook: kaliforms_delete_entries, Fehlercode: could_not_set, Fehlermeldung: Die Cron-Ereignisliste konnte nicht gespeichert werden., Daten: {"schedule":"daily","args":[],"interval":86400}

Das ganze tritt seit Shop-Update auf 5.2.2 & Umstellung auf aktuellere PHP Version 8.2 auf.
Wer kann helfen?
Ich habe bereits diverse Dinge probiert... Neuinstallation Plugin, Löschen und Neu erstellen der Cronjobs, löschen der Exportdateien usw.
Externes aufrufen des Cronjobs (bringt manchmal Fehler 500, manchmal nichts), usw.

Die Google Shopping XML-Datei wird mit 5 Produkten erstellt und bricht dann ab... laut Export > Exporte-Manager 0/808 Artikel exportiert.
memory_limit ist auf 512MB gestellt, Ausführungszeit 120s, also daran sollte es nicht scheitern.
PHP-Version ist die 8.2

Danke soweit...
MfG

 

[FMoche]

Die Fehlermeldung hat doch nichts mit dem Plugin zu tun?
Wohl eher ein Plugin "kaliforms"?

 

[NETZdinge.de]

@FMoche : Noch ein weiteres Problem: Google sieht Bewertungen mit Unicode als Problem ("Unnötige Tags")... Kunden verwenden aber gerne mal Emojis, die dann unter anderem als Unicode Decimal Code enthalten sind...

Hier eine mögliche Funktion das zu entfernen:

public function removeEmoji(string $string): string
{
    
        $string = iconv('UTF-8', 'ISO-8859-15//IGNORE', $string);
        $string = preg_replace('/\&#\d{4,};/', ' ', $string);
        return iconv('ISO-8859-15', 'UTF-8', $string);
}

 

[NETZdinge.de]

Weiterhin: es wäre imho sinnvoll, die Bewertungen von Vaterartikeln auf die Kindartikel (optional?) zu verteilen, da, wenn ich das richtig sehe, die Vaterartikel nicht in den Feed kommen... Diese Bewertungen gehen also verloren...

Vorschlag: in loadProduct() in der GoogleReviewXML.php hinter Zeile 122 einfügen:

 $this->exportProducts[$product->kArtikel]->Bewertungen->oBewertung_arr = array_merge($this->exportProducts[$product->kArtikel]->Bewertungen->oBewertung_arr, $this->exportProducts[$product->kVaterArtikel]->Bewertungen->oBewertung_arr);

 

[ChrisTS]

Bei mir weichen die Bewertungen bei google Shopping erheblich von den Bewertungen im Shop ab.
Muss ich hier als Subfeed im Merchant Center die googlereview.xml hocladen oder woher nimmt sich google hier die Bewertungen und wie werden Sie aktualisiert?
Gibt es eine Anleitung?