verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 49 Releases Wiki Activity

4515-addBuyCheck #1974

Merged
pablone merged 7 commits from 4515-addBuyCheck into dev 2024-02-02 11:59:28 +00:00
Conversation 0 Commits 7 Files Changed 6 +1
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 3182bfa46c - Show all commits

1
db/routines/vn/procedures/buy_chekItem.sql
View File

@ -16,6 +16,7 @@ BEGIN
JOIN item i ON i.id = b.itemFk
JOIN tmp.buysToCheck bt ON bt.id = b.id
WHERE (i.weightByPiece IS NULL OR i.`size` IS NULL)
pablone marked this conversation as resolved
carlosap commented 2024-02-02 10:29:36 +00:00
Review
Copy Link

paréntesis se pueden quitar

paréntesis se pueden quitar
AND a.hasWeightVolumetric
pablone marked this conversation as resolved Outdated
jgallego commented 2024-01-31 08:44:24 +00:00
Outdated
Review
Copy Link

estas segur que funciona be?
si el primer registre coincidira que hasWeightVolumetric es false, no miraria la resta.
Estas asumint que totes compres que et pasen son del mateix travel, pero jo posaria el a.hasWeightVolumetric en el where per a asegurar.
Altra cosa, en procesos com l'inventari, açò fara que vaja molt lento? el buy_upsert li tinguerem que posar la variable dentorno per ixe motiu

estas segur que funciona be? si el primer registre coincidira que hasWeightVolumetric es false, no miraria la resta. Estas asumint que totes compres que et pasen son del mateix travel, pero jo posaria el a.hasWeightVolumetric en el where per a asegurar. Altra cosa, en procesos com l'inventari, açò fara que vaja molt lento? el buy_upsert li tinguerem que posar la variable dentorno per ixe motiu
pablone commented 2024-01-31 13:30:15 +00:00
Outdated
Review
Copy Link

bien visto, estaba realizando las pruebas con 1 sola buy

bien visto, estaba realizando las pruebas con 1 sola buy
LIMIT 1;
DROP TEMPORARY TABLE IF EXISTS tmp.buysToCheck;