xhShop - Bilder Formate
xhShop - Bilder Formate
Ich hatte eben das Problem, dass ein Shop-Nutzer keine Bilder zur Auswahl angeboten bekam, obwohl sie hochgeladen und im richtigen Verzeichnis vorhanden waren.
Nach Untersuchung ergab sich folgendes:
Der User kann Bilder nur mit irgendeinem MS-Bildbearbeitungsprogramm erzeugen
Dieses schreibt prinzipiell alle Datei-Endungen in Großbuchstaben (JPG).
Eine manuelle Änderung aller Artikelbilder ist dem User "zu aufwendig".
Habe zunächst (auf reine Vermutung hin) im Filebrowser "JPG" hinzugefügt - was allerdings Quatsch war, denn der Shop verwendet eine eigene (hartkodierte) Liste der erlaubten Dateiendungen:
https://github.com/cmsimple-xh/xhshop/b ... r.php#L350
Also habe ich dort "JPG" hinzugefügt - der User kann weiterarbeiten.
Meine Frage:
Könnten mit der Großschreibung Probleme auftauchen?
Könnte der Shop nicht die Filebrowser-Konfiguration verwenden, damit es konfigurierbar wird?
Nach Untersuchung ergab sich folgendes:
Der User kann Bilder nur mit irgendeinem MS-Bildbearbeitungsprogramm erzeugen
Dieses schreibt prinzipiell alle Datei-Endungen in Großbuchstaben (JPG).
Eine manuelle Änderung aller Artikelbilder ist dem User "zu aufwendig".
Habe zunächst (auf reine Vermutung hin) im Filebrowser "JPG" hinzugefügt - was allerdings Quatsch war, denn der Shop verwendet eine eigene (hartkodierte) Liste der erlaubten Dateiendungen:
https://github.com/cmsimple-xh/xhshop/b ... r.php#L350
Also habe ich dort "JPG" hinzugefügt - der User kann weiterarbeiten.
Meine Frage:
Könnten mit der Großschreibung Probleme auftauchen?
Könnte der Shop nicht die Filebrowser-Konfiguration verwenden, damit es konfigurierbar wird?
Re: xhShop - Bilder Formate
Ich glaube nicht, dass es damit Probleme geben wird.
Das kann ich nicht beantworten.
Man müsste vielleicht auch nur diese Zeile anpassen.
In Richtung not case sensitive.
Gruß Olaf, Plugins for CMSimple_XH
Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.
Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.
Re: xhShop - Bilder Formate
Na ja. Irgendwie hatte ich im Hinterkopf, dass es mit der Großschreibung Probleme geben könnte.
Aber prinzipiell hast du recht: falls es da keine Probleme gibt, könnte XH selbst auch auf die Kleinschreibung verzichten.
Und vielleicht könnte der Shop tatsächlich die Einstellungen des Filebrowsers übernehmen - damit dort nicht noch eine Konfig-Option hinzukommen muss. Hartkodiert scheint mir irgendwie nicht zeitgemäß.
Re: xhShop - Bilder Formate
Ich sehe zwei mögliche Problemchen. Zum einen könnte der User bild.jpg und bild.JPG hochladen, was ihn selbst verwirren könnte, und wenn er diese Bilder später auf sein Windowssystem herunterlädt, dann wird das nicht ohne weiteres möglich sein. Und zum anderen könnte der Webserver so konfiguriert sein, dass er nur *.jpg als image/jpeg ausliefert, aber nicht *.JPG.
Der Filebrowser ist ja als austauschbares Modul gedacht, so dass ein Rückgriff auf die Filebrowserkonfiguration nicht sinnvoll ist, wenn der User eigentlich mit einem alternativen Filemanager arbeitet.
Die fest vorgegebene Liste ist eigentlich suboptimal, und hier auch nicht nötig, da der Shop ja nichts mit den Bildern macht. Könnte also in eine Konfigurationsoption des XHShop ausgelagert werden.
Aber auf jeden Fall ist es sinnvoll, hier case-insensitive zu vergleichen (ansonsten müsste man zumindest theoretisch auch .Jpg usw. erlauben). Konkret also vielleicht:
Code: Select all
if (in_array(strtolower($extension), $extensions)) {
Christoph M. Becker – Plugins for CMSimple_XH
Re: xhShop - Bilder Formate
Na, mir fällt da nur das Problem ein, welches du gerade hast/hattest
Es scheint eine Macke einiger MS-Tools zu sein, die Erweiterung immer in Großbuchstaben vorzugeben.
Von daher sehe ich den "Fehler" eher dort.
Gruß Olaf, Plugins for CMSimple_XH
Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.
Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.
Re: xhShop - Bilder Formate
Ha, dann müsste ich mal an die Firma "KleinWeich" schreiben ...
Is wahr! Gerade bei solchen Usern, die sowieso nicht besonders fit in solchen Dingen sind.cmb wrote: ↑Tue Dec 11, 2018 10:01 amIch sehe zwei mögliche Problemchen. Zum einen könnte der User bild.jpg und bild.JPG hochladen, was ihn selbst verwirren könnte, und wenn er diese Bilder später auf sein Windowssystem herunterlädt, dann wird das nicht ohne weiteres möglich sein. Und zum anderen könnte der Webserver so konfiguriert sein, dass er nur *.jpg als image/jpeg ausliefert, aber nicht *.JPG.
Bei den issues als "enhancement" eintragen?
Re: xhShop - Bilder Formate
Quick-Fix: du könntest ihm eine kleine Batch-Datei (rename-JPG-to-jpg.bat) geben mit dem Inhalt
Code: Select all
ren *.JPG *.jpg
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“ Ludwig's XH-Templates for MultiPage & OnePage
Re: xhShop - Bilder Formate
Ja. Auch 'ne gute Idee.
Aber - es gibt User, denen man auch das nicht zutrauen kann.
Wie gesagt, ich habe ja eine praktikable Lösung.
(Das Leben ist voller schräger Kompromisse.)
Re: xhShop - Bilder Formate
Das wird auch nicht funktionieren. Ich vermute Windows wird diese Änderung, die für Windows eben keine ist, ignorieren.lck wrote: ↑Tue Dec 11, 2018 11:19 amQuick-Fix: du könntest ihm eine kleine Batch-Datei (rename-JPG-to-jpg.bat) geben mit dem InhaltDie legt er in den Ordner mit den Bildern und startet den Batchvorgang per DoppelklickCode: Select all
ren *.JPG *.jpg
Gruß Olaf, Plugins for CMSimple_XH
Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.
Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.
Re: xhShop - Bilder Formate
Gerade mal getestet: funktioniert. (Win8.1 - nur die Firewall meckert)