GX-Abgewiesen #46348
geschlossen
Inkonsistente 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.
Von Till Tepelmann vor fast 8 Jahren aktualisiert
- Priorität wurde von Hoch zu Normal geändert
Von Mirko Janssen vor
Von Ulrich Wenk vor etwa 5 Jahren aktualisiert
- Zugewiesen an wurde auf Tobias Schindler gesetzt
Von Ulrich Wenk vor etwa 5 Jahren aktualisiert
- Status wurde von Gemeldet zu Feedback geändert
Von Jan-Olof Kratzke vor mehr als 4 Jahren aktualisiert
- Zugewiesen an wurde von Tobias Schindler zu Jan-Olof Kratzke geändert
Von Jan-Olof Kratzke vor mehr als 4 Jahren aktualisiert
- Status wurde von Feedback zu Abgewiesen geändert
Von Till Tepelmann vor mehr als 4 Jahren aktualisiert
- Tracker wurde von GX-Task zu GX-Abgewiesen geändert
- Zugewiesen an wurde von Jan-Olof Kratzke zu Tobias Schindler geändert
Von Moritz Bunjes vor