GX-Abgewiesen #46348
geschlossenInkonsistente Verwendung der Konstante STOCK_CHECK
0%
Beschreibung
Die Funktion xtc_remove_order benutzt die Konstante STOCK_CHECK zur Überprüfung, ob der Lagerbestand aufgefüllt werden soll. In den Lagerverwaltungsoptionen lautet die Einstellungsmöglichkeit des Wertes STOCK_CHECK "Überprüfen des Artikelbestandes" , es gibt aber auch eine Einstellung namens "Artikelmenge abziehen" mit den Wert STOCK_LIMITED.
Ich denke dass der Einstellungswert inkonsistent ist.
Zusätzlich vermute ich, dass die Logik der Funktion nicht richtig ist. Der oben genannte Wert STOCK_CHECK wird öfters zur Überprüfung genutzt und wenn diese richtig (true) ist, wird ein SQL Befehl ausgeführt (z.B. generals.php Z.1471). Wenn wir annehmen, dass der Wert wird in den Lagerverwaltungsoptionen deaktiviert wird/ist, dann würde die Bedingung niemals true sein und der SQL-Befehl innerhalb der Bedingung der oben genannten Zeile auch nicht ausgeführt werden.
Das heißt, dass in diesen Edge-Case das Lager nicht aufgestockt werden würde, obwohl man vorher im Modal beim Löschen der Bestellung den Wert "Lagerbestand auffüllen" gesetzt hätte.
Ich denke zu den Thema sollte man sich noch einmal mit mehreren Leuten Gedanken machen.