verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

fix: refs #7404 restore stockBuyed old proc #3183

Merged
pablone merged 2 commits from 7404-recoverStockBuyedProcs into master 2024-11-12 08:13:09 +00:00
Conversation 0 Commits 2 Files Changed 3 +146
pablone commented 2024-11-12 07:51:49 +00:00
Member
Copy Link
No description provided.
pablone added 1 commit 2024-11-12 07:51:49 +00:00
gitea/salix/pipeline/pr-master Build queued... Details
63425d31ae fix: refs #7404 restore stockBuyed old proc
pablone added 1 commit 2024-11-12 07:52:00 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
69999125c7 Merge branch 'master' into 7404-recoverStockBuyedProcs
pablone requested review from jgallego 2024-11-12 07:55:56 +00:00
jgallego requested changes 2024-11-12 08:05:02 +00:00
Dismissed
db/routines/vn/procedures/stockBuyedByWorker.sql
@ -0,0 +27,4 @@
INSERT INTO stockBuyed(userFk, buyed, `dated`, reserved, requested, description)
SELECT it.workerFk,
SUM((ti.quantity / b.packing) * buy_getVolume(b.id)) / vc.palletM3 / 1000000,
jgallego commented 2024-11-12 07:58:35 +00:00
Owner
Copy Link

sum sense gropu by?

sum sense gropu by?
db/routines/vn/procedures/stockBuyedByWorker.sql
@ -0,0 +34,4 @@
u.name
FROM itemType it
JOIN item i ON i.typeFk = it.id
LEFT JOIN tmp.item ti ON ti.itemFk = i.id
jgallego commented 2024-11-12 08:02:40 +00:00
Owner
Copy Link

si bases el calcul en la ti.quantity, sino hi ha tmp.item no tindria sentit, per tant, es posible que ací necesites un JOIN. sense left

si bases el calcul en la ti.quantity, sino hi ha tmp.item no tindria sentit, per tant, es posible que ací necesites un JOIN. sense left
db/routines/vn/procedures/stockBuyedByWorker.sql
@ -0,0 +52,4 @@
ti.quantity Cantidad,
(ac.conversionCoefficient * (ti.quantity / b.packing) * buy_getVolume(b.id))
/ (vc.trolleyM3 * 1000000) buyed,
b.packagingFk id_cubo,
jgallego commented 2024-11-12 08:03:29 +00:00
Owner
Copy Link

castellà no.

castellà no.
jgallego approved these changes 2024-11-12 08:06:36 +00:00
pablone merged commit 307179cb9d into master 2024-11-12 08:13:09 +00:00
pablone deleted branch 7404-recoverStockBuyedProcs 2024-11-12 08:13:09 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#3183
No description provided.
Delete Branch "7404-recoverStockBuyedProcs"

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?