fix: refs #7213 problem rounding #2903

Merged
carlosap merged 4 commits from 7213-Sale_setProblemRoundingByBuy into master 2024-09-02 15:48:27 +00:00
Member
No description provided.
carlosap added 1 commit 2024-08-30 11:09:16 +00:00
gitea/salix/pipeline/pr-master There was a failure building this commit Details
3da6a700ae
fix: refs #7213 problem rounding
carlosap requested review from jgallego 2024-08-30 11:09:24 +00:00
carlosap requested review from guillermo 2024-08-30 11:09:29 +00:00
carlosap added 1 commit 2024-08-30 11:14:48 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
beb54aedf8
fix: refs #7213 problem rounding
guillermo requested changes 2024-08-30 11:54:02 +00:00
Dismissed
@ -0,0 +4,4 @@
)
BEGIN
/**
* Update rounding problem for all sales related to a buy
Member
/**
 * Update rounding problem for all sales related to a buy.
 *
 * @param vBuyFk Buy id
 */
``` /** * Update rounding problem for all sales related to a buy. * * @param vBuyFk Buy id */ ```
carlosap marked this conversation as resolved
@ -0,0 +18,4 @@
SELECT b.itemFk, t.warehouseInFk
INTO vItemFk, vWareHouseFk
FROM vn.buy b
Member

No utilizar vn.

No utilizar `vn.`
carlosap marked this conversation as resolved
@ -0,0 +54,4 @@
END WHILE;
CREATE OR REPLACE TEMPORARY TABLE tmp.sale
(INDEX(saleFk, isProblemCalcNeeded))
Member

ENGINE = MEMORY

ENGINE = MEMORY
carlosap marked this conversation as resolved
@ -0,0 +55,4 @@
CREATE OR REPLACE TEMPORARY TABLE tmp.sale
(INDEX(saleFk, isProblemCalcNeeded))
SELECT s.id saleFk ,
Member

espacio entre el alias y la ,

espacio entre el alias y la ,
carlosap marked this conversation as resolved
@ -0,0 +62,4 @@
JOIN ticket t ON t.id = s.ticketFk
WHERE s.itemFk = vItemFk
AND s.quantity > 0
AND t.shipped BETWEEN vMinDated
Member

Per que no fiques aço en una linea? No he vist ningún lloc aon no se fique el BETWEEN seguit

Per que no fiques aço en una linea? No he vist ningún lloc aon no se fique el BETWEEN seguit
carlosap marked this conversation as resolved
guillermo requested changes 2024-08-30 11:58:07 +00:00
Dismissed
@ -0,0 +27,4 @@
INTO vMaxDated, vMinDated
FROM sale s
JOIN ticket t ON t.id = s.ticketFk
WHERE t.shipped >= CURDATE()
Member

util.VN_CURDATE()

util.VN_CURDATE()
carlosap marked this conversation as resolved
carlosap added 1 commit 2024-08-30 12:14:08 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
4ed5811042
fix: refs #7213 problem rounding
carlosap added 1 commit 2024-08-30 12:16:07 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
c88b069ac3
fix: refs #7213 problem rounding
carlosap requested review from guillermo 2024-08-30 12:17:54 +00:00
jgallego approved these changes 2024-08-30 13:38:47 +00:00
guillermo approved these changes 2024-09-02 05:06:58 +00:00
carlosap merged commit 41f0c8dc7f into master 2024-09-02 15:48:27 +00:00
carlosap deleted branch 7213-Sale_setProblemRoundingByBuy 2024-09-02 15:48:30 +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#2903
No description provided.