fix: refs #7931 Immediately discount order lines from available #2925
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#2925
Loading…
Reference in New Issue
No description provided.
Delete Branch "7931-availableFix"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -0,0 +11,4 @@
SELECT id INTO vCalc FROM cache_calc
WHERE cacheName = 'available'
AND params = CONCAT_WS('/', vWarehouse, vDated)
AND last_refresh <= NOW();
¿ puede haber más de una cache con los mismos params ? Si es así, haría falta poner LIMIT, y si no, sobra la condición de last_refresh <= NOW()
No puede haber mas de una para el mismo día, elimino el filtro por
last_refresh
ya que siempre se va a cumplir.@ -0,0 +4,4 @@
`vWarehouse` INT,
`vDated` DATE,
`vQuantity` INT
)
comentaris
Comentarios añadidos
@ -0,0 +8,4 @@
BEGIN
DECLARE vCalc INT;
SELECT id INTO vCalc FROM cache_calc
bot de línea de FROM
@ -0,0 +15,4 @@
IF vCalc IS NOT NULL THEN
UPDATE available
SET available = available - vQuantity
per evitar els negatius ,es pot ficar una funció GREATEST(0, available - vQuantity)
No lo veo, el procedimiento también puede que se utilice para sumar algo a la cache, ademas de esa forma se estan actualizando los parámetros de entrada de forma oculta para el "llamante" del procedimiento.