Falsche Darstellung der Lieferstatus
Verfasst: 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 ???
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 ???