Projekt

Allgemein

Profil

Aktionen

GX-Abgewiesen #46348

geschlossen

Inkonsistente Verwendung der Konstante STOCK_CHECK

Von Tobias Schindler vor fast 8 Jahren hinzugefügt. Vor mehr als 2 Jahren aktualisiert.

Status:
Abgewiesen
Priorität:
Normal
Zugewiesen an:
Tobias Schindler
Kategorie:
Core
Zielversion:
-
Beginn:
Abgabedatum:
% erledigt:

0%

Geschätzter Aufwand:
Steps to reproduce:
Release Notes Langtext:

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.


Aktionen

Auch abrufbar als: Atom PDF