Neu Bildname wird falsch gesetzt -> x-jpg.jpg

[dolfy]

Hallo,

neuste Wawi/ Connector und Shopware version.
Bildname wird falsch hochgeladen. Bild leistung.jpg wird als leistung-jpg.jpg gespeichert/gepushed.

Hier ein ausschnitt vom Connectorlog vom Client aus.

>>> image.push: {

  "method": "image.push",
  "params": [
    {
      "filename": "694_737_1eda2822b3e411cc2a55cbe91de6bbc2_ef915c1090e9d780754d835df2ab0dec54842219420b412ee991d9f83bb30dd5d3a79c23694e93715fb87dfd6f593d38e9cc2aa853e9b1a8d22c9199560121bf.jpg",
      "remoteUrl": null,
      "name": "leistung.jpg",
      "foreignKey": [
        "",
        31010
      ],
      "id": [
        "",
        9727
      ],
      "relationType": "product",
      "sort": 3,
      "i18ns": []
    }
  ],
  "jtlrpc": "2.0",
  "id": "a9cb4ef6dd804825bb5133d8768a9db0"
}
<<< image.push: {"result":[{"foreignKey":["53693_153",31010],"id":["a_1519_1896",9727],"filename":"https:\/\/www.xxxxxxx.com\/media\/image\/09\/80\/5b\/leistung-jpg.jpg","relationType":"product","remoteUrl":"","name":"leistung.jpg","sort":3,"i18ns":[]}],"error":null,"jtlrpc":"2.0","id":"a9cb4ef6dd804825bb5133d8768a9db0"}
(1116,8126 milliseconds)
JTL-Wawi: End sync

 

[McAvity]

@dolfy

Ich tippe auf einen Fehler in der Funktion "sanitizeImageName", Changelog dazu hier: https://github.com/jtl-software/connector-shopware5/commit/6aa0b8d8435129bec6b756eeb092bbd413a65771

Vielleicht schaut @Immanuel da mal drauf.

MfG

McAvity

 

[McAvity]

@dolfy @Immanuel

In der Funktion "sanitizeImageName($name)"

muss die Zeile 1175 von

 $name = preg_replace('#[^A-Za-z0-9\-_]#', '-', $name);

in

 $name = preg_replace('#[^A-Za-z0-9\-_.]#', '-', $name);

geändert werden (wichtig ist der Punkt vor der schliessenden eckigen Klammer - sonst wird der Punkt in "leistung.jpg" in "leistung-jpg" geändert).

MfG

McAvity

P.S.:

Hier kann man die Unterschiede mal testen:

ohne Punkt:https://www.phpliveregex.com/p/z36

mit Punkt: https://www.phpliveregex.com/p/z37

Wichtig: oben rechts von "preg_match" auf "preg_replace" wechseln!

 

[Immanuel]

Hallo zusammen,

Problem ist, dass JTL-Wawi neuerdings den Bildnamen mit Extension in das "name" Property packt. Müssen wir intern klären ob die Connectoren nachgezogen werden oder ob das in JTL-Wawi behoben wird.

PS: Schönes Wochenende!

 

[McAvity]

Problem ist, dass JTL-Wawi neuerdings den Bildnamen mit Extension in das "name" Property packt

Dann schneidet doch vor dem "sanitizen" die File-Extension ab.

MfG

McAvity

 

[dolfy]

Supi dank dir, hast du ja schnell gefunden. Komisch das es noch nicht aufgefallen ist.
Ich habe weiter in Dez gesucht wo was an die images geändert würde, dort aber nichts verdächtiges gefunden.
Added sanitizing image name for comparison habe ich glatt übersehen.

 

[dolfy]

Im Shopware Backend kann man ja Bilder ersetzen um Bildname/Pfad beizubehalten für z.B. Bilder SEO, interne/externe Links beizubehalten etc.
Was mir aufgefallen ist, man sollte nicht auf die idee kommen in der Wawi die funktion Bild -> Ersetzen zu verwenden.
Laut Log wird erst ein image.delete abgesetzt und dann ein image.push.
Beim image.push wird der komplette Pfad als Bildname verwendet.
"filename":"https:\/\/www.xxx.com\/media\/image\/39\/be\/b4\/C-Users-Dolfy-Shopware-WAWI-Bilder-Kategorie-Unterkategrie-leistung-jpg.jpg"😵

 

[dolfy]

Problem ist, dass JTL-Wawi neuerdings den Bildnamen mit Extension in das "name" Property packt. Müssen wir intern klären ob die Connectoren nachgezogen werden oder ob das in JTL-Wawi behoben wird.

Moin @Immanuel,
gibt es schon Neuigkeiten?

Aktuell werde ich etwas ausgebremst da einige Bildübertragungen anstehen.

 

[Immanuel]

Moin @Immanuel,
gibt es schon Neuigkeiten?

Aktuell werde ich etwas ausgebremst da einige Bildübertragungen anstehen.

Du kannst in JTL-Wawi am Bild unter Onlineshops den Namen des Bildes anpassen und dort die Extension entfernen. Dann wird der Bildname ohne Extension zum Shop geschickt.

 

[dolfy]

Als workaround OK für mich, aber keine Dauerlösung, bitte dran bleiben.

 

[McAvity]

den Namen des Bildes anpassen und dort die Extension entfernen. Dann wird das Bild ohne Extension zum Shop geschickt.

Löst das auch dieses Problem:

Beim image.push wird der komplette Pfad als Bildname verwendet.

MfG

McAvity

 

[dolfy]

Eigentlich ist das auch keine Lösung für mich als workaround.
Wenn mal später, aus welcher Grunde auch immer, ein zurücksetzen von Shopware nötig ist und ein Komplettabgleich gemacht werden muss, sind alle workaround Bildname ohne Endung dann vielleicht wieder eine Baustelle da die Endung fehlen.
Daher die bitte hier möglich Zeitnah euch Intern auf eine Lösung zu einigen und entweder WAWI oder Connector anzupassen.

 

[McAvity]

Daher die bitte hier möglich Zeitnah euch Intern auf eine Lösung zu einigen und entweder WAWI oder Connector anzupassen.

Die Frage ist doch, warum wurde das in der Wawi überhaupt geändert, @Immanuel ?

MfG

McAvity

 

[dolfy]

Beim image.push wird der komplette Pfad als Bildname verwendet.

Hier haben die ein ähnliche Problem bei Bildname scheinbar behoben im 1.5.38.2

 

[dolfy]

Das oben genannte Ticket ist doch was anderes, hier würde beim Import falsch gemappt, hat also nicht wirklich hiermit was tu tun.
@Immanuel, soll hierfür ein Ticket angelegt werden oder hast du es auf dem schirm?

 

[Immanuel]

Hallo @dolfy,

es gibt schon ein entsprechendes Ticket dazu in unserem Issue Tracker (https://issues.jtl-software.de/issues/WAWI-53154).

 

[dolfy]

Ich will ja nicht nerven, aber was muss ich unter Kurzfristig verstehen?

Es ist ja kein Problem die noch nach gedebug werden muss, verstehe daher nicht das es so lange dauert ......