id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking
2554	xtc_check_stock überflüssig/ändern	info@…	somebody	"Die Funktion xtc_check_stock() wird nur in folgenden beiden Dateien verwendet:
includes/moduels/order_details_cart.php
includes/checkout_requirements.php

die xtc_check_stock() verwendet wiederum die xtc_get_products_stock() um den aktuellen Artikelbestand in der Datenbank abzufragen.
Diese Datenbankabfrage ist aber nicht notwendig, da der aktuelle Bestand schon im products-array steht, den die Klasse shoppingCart anhand der get_products()-Methode bildet.

Ich schlage daher folgende Änderungen vor:

inc/xtc_check_stock.inc.php
{{{
function xtc_check_stock($products_db_stock, $products_quantity) {
    $stock_left = products_db_stock - $products_quantity;

    $out_of_stock = '';
    if ($stock_left < 0) {
      $out_of_stock = '<span class=""markProductOutOfStock"">' . STOCK_MARK_PRODUCT_OUT_OF_STOCK . '</span>';
    }

    return $out_of_stock;
  }
}}}
(und require_once.... streichen)

includes/modules/order_details_cart.php Zeile 55
{{{
$mark_stock = xtc_check_stock($products[$i]['stock'], $products[$i]['quantity']);
}}}


includes/checkout_requirements.php Zeile 70
{{{
if (xtc_check_stock($products[$i]['stock'], $products[$i]['quantity'])) {
}}}

Dadurch spart man ein paar Datenbankabfragen ein und - der Grund warum ich mir das angeschaut habe: man kann die Anzahl über eine Klassenerweiterung der shoppingCart manipulieren.

Meine Kunden haben lassen grundsätzlich den Bestand im Warenkorb prüfen. Bei einigen wenigen Artikeln soll die Prüfung jedoch ausgesetzt sein. Da jedoch eine eine Wawi angebunden ist, kann nicht einfach mit einem Millionenbestand für diese Artikel gearbeitet werden, sodass die Bestandsprüfung anderweitig abgeschaltet werden muss.

Grüße
AGI
"	Neues Feature	closed	normal	modified-shop-2.0.8.0	Shop	trunk	fixed