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

fix: refs #7213 Hotfix-ticket_isTooLittle-group-by-addressFk #2895

Merged
carlosap merged 3 commits from Hotfix-ticket_isTooLittle-group-by-addressFk into master 2024-08-28 14:41:49 +00:00
Conversation 0 Commits 3 Files Changed 2 +7 -8
2 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 1ec9e5bf09 - Show all commits

9
db/routines/vn/functions/ticket_isTooLittle.sql
View File

@ -20,13 +20,12 @@ BEGIN
) )
SELECT (SUM(IFNULL(sv.litros, 0)) < vc.minTicketVolume SELECT (SUM(IFNULL(sv.litros, 0)) < vc.minTicketVolume
AND SUM(IFNULL(t.totalWithoutVat, 0)) < vc.minTicketValue) INTO vIsTooLittle AND SUM(IFNULL(t.totalWithoutVat, 0)) < vc.minTicketValue) INTO vIsTooLittle
FROM vn.ticket t FROM tickets ts
JOIN tickets ts ON ts.addressFk = t.addressFk JOIN vn.ticket t ON t.addressFk = ts.addressFk
JOIN vn.volumeConfig vc
LEFT JOIN vn.saleVolume sv ON sv.ticketFk = t.id LEFT JOIN vn.saleVolume sv ON sv.ticketFk = t.id
JOIN vn.volumeConfig vc
WHERE t.shipped BETWEEN ts.dated AND util.dayEnd(ts.dated) WHERE t.shipped BETWEEN ts.dated AND util.dayEnd(ts.dated)
AND sv.litros > 0 AND ticket_isProblemCalcNeeded(t.id);
AND t.totalWithoutVat > 0;
RETURN vIsTooLittle; RETURN vIsTooLittle;
END$$ END$$

6
db/routines/vn/procedures/ticket_setProblemTooLittle.sql
View File

@ -17,9 +17,9 @@ BEGIN
FROM vn.ticket FROM vn.ticket
WHERE id = vSelf WHERE id = vSelf
) )
SELECT vSelf ticketFk, SELECT t.id ticketFk,
carlosap marked this conversation as resolved Outdated
guillermo commented 2024-08-28 05:27:57 +00:00
Outdated
Review
Copy Link

Ací estás pasant per cada ticket per la funció vn.ticket_isTooLittle(), no et fa falta, jo ficaria el SELECT amb el ticket vSelf dalt en una variable, ja que tots els tickets te van a donar lo mateix.

Entenc tot lo demes, ja que si que te interesa cridar a la funció ticket_isProblemCalcNeeded per cada ticket.

Ací estás pasant per cada ticket per la funció `vn.ticket_isTooLittle()`, no et fa falta, jo ficaria el SELECT amb el ticket vSelf dalt en una variable, ja que tots els tickets te van a donar lo mateix. Entenc tot lo demes, ja que si que te interesa cridar a la funció `ticket_isProblemCalcNeeded` per cada ticket.
ticket_isTooLittle(vSelf) hasProblem, ticket_isTooLittle(t.id) hasProblem,
ticket_isProblemCalcNeeded(vSelf) isProblemCalcNeeded ticket_isProblemCalcNeeded(t.id) isProblemCalcNeeded
FROM vn.ticket t FROM vn.ticket t
JOIN tickets ts ON ts.addressFk = t.addressFk JOIN tickets ts ON ts.addressFk = t.addressFk
WHERE t.shipped BETWEEN ts.dated AND util.dayEnd(ts.dated); WHERE t.shipped BETWEEN ts.dated AND util.dayEnd(ts.dated);