Anzeige des noimage.gif wenn kein Kategoriebild angegeben wurde

[astaller]

Hallo, die o.g. Funktion funktioniert nicht.
Es wird kein noimage.gif angezeigt.
Hab es soeben im Demoshop getestet.

MfG
Achim S.

[Torsten Riemer]

Das kann ich bestätigen. Das Kategoriebild wird derzeit nur durch das "noimage.gif" ersetzt, wenn in der Kategorie ein Bild hinterlegt ist, welches sich nicht mehr im Ordner "/images/categories/" befindet.

[Ronald Parcinski]

Fehler ist in default.php

Korrekturvorschlag:

if ($categories['categories_image'] != '') {
        $image = DIR_WS_IMAGES.'categories/'.$categories['categories_image'];
        if (!file_exists(DIR_FS_CATALOG.$image)) {
          if (CATEGORIES_IMAGE_SHOW_NO_IMAGE == 'true') {
            $image = DIR_WS_IMAGES.'categories/noimage.gif';
          } else {
            $image = '';
          }
        }
      } elseif(CATEGORIES_IMAGE_SHOW_NO_IMAGE == 'true') {
        $image = DIR_WS_IMAGES.'categories/noimage.gif';
      }

Wobei man wirklich überlegen sollte:
Platzhalterbild wenn kein Bild zugeordnet wurde
Fehlerbild wenn das Bild auf dem Server nicht gefunden wird.

[Hasenpriester]

Da ich auch gerade drauf gestoßen bin, den Korrekturvorschlag kann man noch leicht verkürzen was die Lesbarkeit des Quellcodes zugute kommt:

$image = '';
if($categories['categories_image'] != '')
  $image = DIR_WS_IMAGES.'categories/'.$categories['categories_image'];

if(!file_exists(DIR_FS_CATALOG.$image))
  $image = CATEGORIES_IMAGE_SHOW_NO_IMAGE == 'true' ? DIR_WS_IMAGES.'categories/noimage.gif' : '';

[Torsten Riemer]

Wenn ich das richtig sehe, dann haben wir das selbe Problem auch an anderen Stellen.
Suche nach "noimage.gif" ergibt alle Stellen, die nochmal überarbeitet werden sollten.

Logik:

• Bild in Datenbank, aber kein Bild vorhanden sollte immer das "noimage.gif" anzeigen
• Nur wenn "noimage.gif" Anzeige unter "Konfiguration" -> "Bild Optionen" auf "Ja" steht, sollte das "noimage.gif" auch immer dann als Platzhalter angezeigt werden, wenn kein Bild angegeben wurde.

[Gerhard Waldemair]

In 10782:

fix #1029

[Gerhard Waldemair]

In 10792:

fix #1029

[Gerhard Waldemair]

In 10794:

fix #1029