Falsche Darstellung der Lieferstatus

Kein System ist vollkommen, findet Ihr Fehler im System, könnt ihr diese hier gerne Posten. Wir werden auch kurzfistige Lösungen und Fixes hier schreiben.

Moderatoren: Magnus, mmaass, ablass

Falsche Darstellung der Lieferstatus

Beitragvon apotheker » Mo 2. Mär 2015, 19:20

Hallo,

ich habe festgestellt, dass unter bestimmten Bedingungen der Lieferstatus nicht richtig angezeigt wird,
und zwar wenn man im Template der Kategorieübersicht nicht den 'lieferstatus_imagestring', sondern
'lieferstatus_smallimagestring' oder 'lieferstatus_name' verwenden möchte. Das Problem liegt in der Funktion
'GetArtikelDataArrayPA' aus 'functions.artikel.inc.php': Nur der 'lieferstatus_imagestring' wird je nach
Bestand angepasst, aber nicht 'lieferstatus_smallimagestring' und 'lieferstatus_name'. Dadurch zeigen
diese drei Elemente unter Umständen einen unterschiedlichen Lieferstatus an.

Mein Vorschlag zur Anpassung:

*** Zeile 2731 ff. ***
// auskommentiert jb 2015-03-02 - sollte hier noch nicht gesetzt werden
// $ArtikelDataArray[$ArtikelCounter]["lieferstatus"] = $ArtikelRowArray["lieferstatus_id"];
// $ArtikelDataArray[$ArtikelCounter]["lieferstatusname"] = $ArtikelRowArray["lieferstatusname"];

*** Zeile 2909 ff. ***
// auskommentiert jb 2015-03-02
// kleines Lieferstatusbild - sollte hier noch nicht gesetzt werden
//if ($ArtikelRowArray["lieferstatus_imagesmall"] && file_exists(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["lieferstatus_imagesmall"])) {
// $ImageSizeArray = getimagesize(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["lieferstatus_imagesmall"]);
// $ArtikelDataArray[$ArtikelCounter]["lieferstatus_smallimagestring"] = "<img src=\"" . IMAGEPFAD . "dbimages/" . $ArtikelRowArray["lieferstatus_imagesmall"] . "\" width=\"" . $ImageSizeArray[0] . "\" height=\"" . $ImageSizeArray[1] . "\" alt=\"" . $ArtikelRowArray["lieferstatusname"] . "\" />";
//}

// grosses Lieferstatusbild
if ($ArtikelRowArray["lager_bestellungen"] > 0 || !$ArtikelRowArray["nulllieferstatusid"]) {

if ($ArtikelRowArray["lieferstatus_image"] && file_exists(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["lieferstatus_image"])) {
$ImageSizeArray = getimagesize(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["lieferstatus_image"]);
$ArtikelDataArray[$ArtikelCounter]["lieferstatus_imagestring"] = "<img src=\"" . IMAGEPFAD . "dbimages/" . $ArtikelRowArray["lieferstatus_image"] . "\" width=\"" . $ImageSizeArray[0] . "\" height=\"" . $ImageSizeArray[1] . "\" alt=\"" . $ArtikelRowArray["lieferstatusname"] . "\" />";
} else {
$ArtikelDataArray[$ArtikelCounter]["lieferstatus_imagestring"] = $ArtikelRowArray["lieferstatus_name"];
}

// ergaenzt jb 2015-03-02 - kleines Lieferstatusbild und Lieferstatusname sollten HIER gesetzt werden
if ($ArtikelRowArray["lieferstatus_imagesmall"] && file_exists(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["lieferstatus_imagesmall"])) {
$ImageSizeArray = getimagesize(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["lieferstatus_imagesmall"]);
$ArtikelDataArray[$ArtikelCounter]["lieferstatus_smallimagestring"] = "<img src=\"" . IMAGEPFAD . "dbimages/" . $ArtikelRowArray["lieferstatus_imagesmall"] . "\" width=\"" . $ImageSizeArray[0] . "\" height=\"" . $ImageSizeArray[1] . "\" alt=\"" . $ArtikelRowArray["lieferstatusname"] . "\" />";
}
$ArtikelDataArray[$ArtikelCounter]["lieferstatus"] = $ArtikelRowArray["lieferstatus_id"];
$ArtikelDataArray[$ArtikelCounter]["lieferstatusname"] = $ArtikelRowArray["lieferstatusname"];
// Ende Ergaenzung

} else {

if ($ArtikelRowArray["nulllieferstatus_image"] && file_exists(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["nulllieferstatus_image"])) {
$ImageSizeArray = getimagesize(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["nulllieferstatus_image"]);
$ArtikelDataArray[$ArtikelCounter]["lieferstatus_imagestring"] = "<img src=\"" . IMAGEPFAD . "dbimages/" . $ArtikelRowArray["nulllieferstatus_image"] . "\" width=\"" . $ImageSizeArray[0] . "\" height=\"" . $ImageSizeArray[1] . "\" alt=\"" . $ArtikelRowArray["lieferstatusname"] . "\" />";
} else {
$ArtikelDataArray[$ArtikelCounter]["lieferstatus_imagestring"] = $ArtikelRowArray["nulllieferstatus_name"];
}

// ergaenzt jb 2015-03-02 - auch kleines Lieferstatusbild und Lieferstatusname sollten angepasst werden !
if ($ArtikelRowArray["lieferstatus_imagesmall"] && file_exists(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["nulllieferstatus_imagesmall"])) {
$ImageSizeArray = getimagesize(DATEIPFAD . "/images/dbimages/" . $ArtikelRowArray["nulllieferstatus_imagesmall"]);
$ArtikelDataArray[$ArtikelCounter]["lieferstatus_smallimagestring"] = "<img src=\"" . IMAGEPFAD . "dbimages/" . $ArtikelRowArray["nulllieferstatus_imagesmall"] . "\" width=\"" . $ImageSizeArray[0] . "\" height=\"" . $ImageSizeArray[1] . "\" alt=\"" . $ArtikelRowArray["nulllieferstatus_name"] . "\" />";
}
$ArtikelDataArray[$ArtikelCounter]["lieferstatus"] = $ArtikelRowArray["nulllieferstatusid"];
$ArtikelDataArray[$ArtikelCounter]["lieferstatusname"] = $ArtikelRowArray["nulllieferstatus_name"];
// Ende Ergaenzung
}

Kann man Euch für größere Änderungsvorschläge eigentlich Patches schicken ???
apotheker
 
Beiträge: 3
Registriert: Mi 5. Nov 2014, 18:18

Re: Falsche Darstellung der Lieferstatus

Beitragvon ablass » Di 9. Jun 2015, 15:29

Danke dir, habe ich in die v2.4 so übernommen.

Du kannst uns gerne auch patch-files hier dranhängen, das erleichtert uns auf jeden Fall die Arbeit.
Ansonsten wenn du hier Code postest machen sich Code Tags drumrum auch immer ganz gut. ;)
ablass
 
Beiträge: 126
Registriert: Mi 22. Jul 2009, 07:56


Zurück zu Bugmeldungen

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 8 Gäste