fix: refs #7931 Immediately discount order lines from available #2925

Merged
juan merged 8 commits from 7931-availableFix into test 2024-09-06 11:11:36 +00:00
Owner
No description provided.
juan added 1 commit 2024-09-05 12:46:19 +00:00
juan requested review from pako 2024-09-05 12:46:27 +00:00
juan requested review from carlosap 2024-09-05 14:19:26 +00:00
pako approved these changes 2024-09-05 15:10:40 +00:00
pako reviewed 2024-09-05 15:11:35 +00:00
@ -0,0 +11,4 @@
SELECT id INTO vCalc FROM cache_calc
WHERE cacheName = 'available'
AND params = CONCAT_WS('/', vWarehouse, vDated)
AND last_refresh <= NOW();
Member

¿ 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()

¿ 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()
Author
Owner

No puede haber mas de una para el mismo día, elimino el filtro por last_refresh ya que siempre se va a cumplir.

No puede haber mas de una para el mismo día, elimino el filtro por `last_refresh` ya que siempre se va a cumplir.
juan marked this conversation as resolved
carlosap requested changes 2024-09-05 15:45:35 +00:00
Dismissed
@ -0,0 +4,4 @@
`vWarehouse` INT,
`vDated` DATE,
`vQuantity` INT
)
Member

comentaris

comentaris
Author
Owner

Comentarios añadidos

Comentarios añadidos
juan marked this conversation as resolved
@ -0,0 +8,4 @@
BEGIN
DECLARE vCalc INT;
SELECT id INTO vCalc FROM cache_calc
Member

bot de línea de FROM

bot de línea de FROM
juan marked this conversation as resolved
@ -0,0 +15,4 @@
IF vCalc IS NOT NULL THEN
UPDATE available
SET available = available - vQuantity
Member

per evitar els negatius ,es pot ficar una funció GREATEST(0, available - vQuantity)

per evitar els negatius ,es pot ficar una funció GREATEST(0, available - vQuantity)
Author
Owner

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.

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.
juan marked this conversation as resolved
juan added 1 commit 2024-09-06 07:06:08 +00:00
gitea/salix/pipeline/pr-test There was a failure building this commit Details
640b45ed99
fix: refs #7931 Apply PR change requests
juan added 1 commit 2024-09-06 07:14:11 +00:00
gitea/salix/pipeline/pr-test There was a failure building this commit Details
93330db88f
Merge branch 'test' into 7931-availableFix
juan added 2 commits 2024-09-06 07:16:26 +00:00
carlosap approved these changes 2024-09-06 08:01:08 +00:00
juan added 1 commit 2024-09-06 09:25:53 +00:00
gitea/salix/pipeline/pr-test There was a failure building this commit Details
d44dec3703
fix: refs #7931 Back test fix
juan added 1 commit 2024-09-06 09:30:49 +00:00
gitea/salix/pipeline/pr-test There was a failure building this commit Details
382e54b572
fix: refs #7931 Back test fix
juan added 1 commit 2024-09-06 10:58:01 +00:00
gitea/salix/pipeline/pr-test This commit looks good Details
e2c6a45756
fix: refs #7931 Available back test fix
juan merged commit 5700b95e22 into test 2024-09-06 11:11:36 +00:00
juan deleted branch 7931-availableFix 2024-09-06 11:11:36 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#2925
No description provided.