Probleme bei Uploads und Namenskonvention

[speedy]

Hallo,

bei Uploads gibt es immer wieder verschiedene Probleme, z.B.:
- Die gleichen Bilder können als .JPG oder .jpg existieren.
  Könnte man eventuell generell bei Uploads alles in Kleinbuchstaben umwandeln ?
- Auch Leerzeichen, Sonderzeichen, Umlaute oder Zeichen wie ß é werden gerne verwendet, obwohl das der gängigen Namenskonvention für Dateien im Internet widerspricht.

Das Ganze führt dann immer wieder zu Fehlern, nicht zwingend beim Upload, aber dann später im Shop.

Könnte man hier die Uploads bei Produkten, Kategorien und per Editor so umändern, dass nur noch folgende gängige Konvention gilt ? Den Rest ähnlich der Shopstat filtern oder ersetzen.

Zulässige Zeichen: a-z, 0-9, Unterstrich

Was auch noch interessant wäre, ist eine Begrenzung beim Upload.
Eine 6 MB Datei pro Bild muss man nun wirklich nicht hochladen.

Hat hier schon jemand Versuche unternommen ?



Linkback: https://www.modified-shop.org/forum/index.php?topic=7835.0

[hendrik]

Also bei Kategorien und Artikeln gibt's das Namensproblem eh nicht weil die Dateien nicht unter Originalnamen gespeichert werden. Bei Herstellern hat du Recht. Da ist noch ne Altlast.

Hab hier nen Korrekturpatch für die Herstellerbilder:

Datei admin/manufacturers.de, Zeile 41 ca.
Finde:

Code: PHP  [Auswählen]

xtc_db_query("update " . TABLE_MANUFACTURERS . " set
                         manufacturers_image ='manufacturers/".$manufacturers_image->filename . "'
                         where manufacturers_id = '" . xtc_db_input($manufacturers_id) . "'");
 

Ersetze gegen:

Code: PHP  [Auswählen]

// ---patch beginn-----------
function xtc_correct_filename( $text ) {
  $inp = array('ä','Ä','ü','Ü','ö','Ö','ß',' ','é', 'è', 'á', 'à');
  $out = array('ae','Ae','ue','Ue','oe','Oe','ss','_','e','e','a','a');
  $text = str_replace($inp,$out,$text);
  $text = preg_replace('~[^a-zA-Z0-9._-]*~i','',$text);
  $text = strtolower($text);
  return $text;
}

$sql = "select manufacturers_image from ".TABLE_MANUFACTURERS." where manufacturers_id='".xtc_db_input($manufacturers_id)."'";
$res = xtc_db_query($sql);
if( $data=xtc_db_fetch_array($res) ) {
  if( $data['manufacturers_image']!=$manufacturers_image->filename
      && $data['manufacturers_image']!='' ) {
     @unlink(DIR_FS_CATALOG_IMAGES.$data['manufacturers_image']);
  }
}
$manufacturers_new_image_filename = xtc_db_input($manufacturers_id).'_'.xtc_correct_filename($manufacturers_image->filename);
rename( $dir_manufacturers.'/'.$manufacturers_image->filename, $dir_manufacturers.'/'.$manufacturers_new_image_filename );
xtc_db_query("update " . TABLE_MANUFACTURERS . " set
                         manufacturers_image ='manufacturers/".$manufacturers_new_image_filename . "'
                         where manufacturers_id = '" . xtc_db_input($manufacturers_id) . "'");
// ---patch ende------------------------------
 

Vorherige Dateien werden gelöscht, Dateinamen korrigiert und mit ID versehen.

Mit der Volumensbegrenzung halt ich nicht für zwingend. Die Herstellerbilder werden nicht skaliert. Wer 6 MB Bilder hochlädt wird schon selber merken, daß das irgendwie besch... ist.

Gruß
Hen

[speedy]

Hallo Hendrik,

danke dir, das werde ich testen.
Bei Artikelbildern wird es jedoch noch nicht in Kleinbuchstaben umgewandelt.
Wenn man immer wieder einmal ein Backup zieht auf Windows, dann nervt das immer weil es dann .JPG und .jpg geben kann. Bei einer Rücksicherung muss es dann zwingend auch immer .JPG und .jpg sein. Besser wäre alle klein, dann kann man alles vermeiden.

Auch die Uploads durch den Editor sind betroffen und stressen immer wieder.

[hendrik]

Bei Artikelbildern wird es jedoch noch nicht in Kleinbuchstaben umgewandelt.

Ach so das. Kein Problem.

admin/includes/classes/categories.php, Zeile 526 ca.
Finde:

Code: PHP  [Auswählen]

  $pname_arr = explode('.', $products_image->filename);
  $nsuffix = array_pop($pname_arr);

Füge an:

Code: PHP  [Auswählen]

  $nsuffix = strtolower($nsuffix);

Gruß
Hen

[speedy]

Hallo Hendrik,

hat beides wunderbar funktioniert, nochmal vielen Dank

Wenn man jetzt noch die Uploads vom Editor umbenennt, dann müssten das alle Wege gewesen sein, über die der Anwender Unsinn machen kann.
Hat jemand schon den aktuellen Editor getestet, ob der eventuell die Uploads umbenennt ?

[hendrik]

Ich sehe es nicht als meinen Job (mögen andere anders sehen) den Anwender vor der eigenen Inkompetenz zu schützen. Wer das kleine 1x1 der Webadministration nicht beherrscht schlägt hier so oder so bald auf.

Ich sehe modified eCommerce Shopsoftwareod als ein Expertenprodukt. Ne saubere Grundinstallation und unter den Händen von Fachleuten oder Leuten die sich erfolgreich eingearbeitet haben wird daraus ein individuelles System. Für DAUs gibts andere xtcommerce-Forks die sich genau auf solche Klientel spezialisiert haben.

Gruß
Hen

[speedy]

Habe noch festgestellt, dass es in der Datei admin/includes/classes/categories.php die Zeile

Code: PHP  [Auswählen]

$nsuffix = array_pop($pname_arr);

4x gibt

Kann es sein, dass ich es 4x ändern muss.
Die zweite Stelle ist "Mo_Pics", der Rest irgendwas mit Duplikaten.

[franky_n]

Hi speedy,

mo_pics sind die zusätzlichen Bilder des Hauptprodukts...

und die anderen beiden sind die Duplicate vom Hauptbild und den zusätzlichen Bildern mit der zusätzlichen Endung des Dateinamens mit _0

Du solltest es also bei allen 4ren machen.

Viele Grüße

Franky

[speedy]

Hallo Franky,

ah, ok. Werd ich machen. Dankeschön

[speedy]

Könnte man diese Änderungen von hendrik nicht in den Trunk übernehmen ?
Es würde die Änderungen von Franky mit den gültigen Dateitypen sinnvoll ergänzen, ich hatte es einfach schon zu oft, dass der Shop User trotz mehrfacher Erklärung ungültige Zeichen im Namen verwendet hat - siehe 1. Post. Folge, Bilder wurden nicht angezeigt. Hendriks Änderungen vermeidet das Problem.

[Tomcraft]

Ich hätte nichts dagegen, was meinen die anderen?

Grüße

Torsten

[web28]

Da wir ja noch "Dateinamen bei Produktbildern erhalten" einbauen wollen, wird es eine neue Funktion xtc_validate_filename geben.

[Tomcraft]

Meinst du das: Dateinamen beim Bild-Upload erhalten
oder das: Bild-Upload mit xt:Commerce - Dateinamen erhalten

Auf der Roadmap haben wir das Modul von xtc-load.de

Grüße

Torsten

[web28]

Sind das nicht beides die Anleitungen von gunnart?

[Tomcraft]

lol... ja!

Grüße

Torsten