Merge pull request 'fix: refs #7213 componentLack y setRisk' (!2820) from 7213-Hotfix-calculo-problemas into master

Reviewed-on: #2820
Reviewed-by: Guillermo Bonet <guillermo@verdnatura.es>

db/routines/vn/procedures/sale_setProblemComponentLack.sql

@@ -14,7 +14,7 @@ BEGIN
 		ENGINE = MEMORY
 		SELECT vSelf saleFk, 
 				sale_hasComponentLack(vSelf) hasProblem,
-				ticket_isProblemCalcNeeded(ticketFk) isProblemCalcNeeded
+				(ticket_isProblemCalcNeeded(ticketFk) AND quantity > 0) isProblemCalcNeeded
 			FROM sale 
 			WHERE id = vSelf;
 

db/routines/vn/procedures/sale_setProblemComponentLackByComponent.sql

@@ -14,9 +14,9 @@ BEGIN
 		ENGINE = MEMORY
 		SELECT saleFk, 
 				sale_hasComponentLack(saleFk) hasProblem, 
-				ticket_isProblemCalcNeeded(ticketFk) isProblemCalcNeeded
+				(ticket_isProblemCalcNeeded(ticketFk) AND quantity > 0) isProblemCalcNeeded
 			FROM (
-				SELECT s.id saleFk, s.ticketFk
+				SELECT s.id saleFk, s.ticketFk, s.quantity
 					FROM ticket t
 						JOIN sale s ON s.ticketFk = t.id
 						LEFT JOIN saleComponent sc ON sc.saleFk = s.id 

db/routines/vn/procedures/ticket_setRisk.sql

@@ -1,40 +1,30 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`ticket_setRisk`(
-	vClientFk INT)
+	vClientFk INT
+)
 BEGIN
 /**
- * Update the risk for a client with pending tickets
+ * Update the risk for a client with pending tickets.
  *
  * @param vClientFk Id cliente
  */
-	DECLARE vHasDebt BOOL;
+	IF (SELECT COUNT(*) FROM client WHERE id = vClientFk AND typeFk = 'normal') THEN
-	DECLARE vStarted DATETIME;
-
-	SELECT COUNT(*) INTO vHasDebt
-		FROM `client`
-		WHERE id = vClientFk 
-			AND typeFk = 'normal';
-
-	IF vHasDebt THEN
-
-		SELECT util.VN_CURDATE() - INTERVAL riskScope MONTH INTO vStarted
-			FROM clientConfig;
-
 		CREATE OR REPLACE TEMPORARY TABLE tTicketRisk
-			(KEY (ticketFk))
+			(PRIMARY KEY (ticketFk))
 			ENGINE = MEMORY
 			WITH ticket AS (
-		SELECT id ticketFk, 
+				SELECT t.id ticketFk,
-				companyFk, 
+						t.companyFk,
-				DATE(shipped) dated, 
+						DATE(t.shipped) dated,
-				totalWithVat,
+						t.totalWithVat,
-				ticket_isProblemCalcNeeded(id) isProblemCalcNeeded 
+						ticket_isProblemCalcNeeded(t.id) isProblemCalcNeeded
-			FROM vn.ticket 
+					FROM vn.ticket t
-			WHERE clientFk = vClientFk
+						JOIN vn.clientConfig cc
-				AND refFk IS NULL
+					WHERE t.clientFk = vClientFk
-				AND NOT isDeleted 
+						AND t.refFk IS NULL
-				AND IFNULL(totalWithVat, 0) <> 0
+						AND NOT t.isDeleted
-				AND shipped > vStarted	
+						AND IFNULL(t.totalWithVat, 0) <> 0
+						AND t.shipped > (util.VN_CURDATE() - INTERVAL cc.riskScope MONTH)
 				), balance AS (
 					SELECT SUM(amount)amount, companyFk
 						FROM (
@@ -47,7 +37,7 @@ BEGIN
 									JOIN hedera.tpvMerchant tm ON t.id = t.merchantFk
 								WHERE clientFk = vClientFk
 									AND receiptFk IS NULL
-						AND status = 'ok'
+									AND `status` = 'ok'
 						) sub
 						WHERE companyFk
 						GROUP BY companyFk
@@ -69,24 +59,18 @@ BEGIN
 								SUM(IFNULL(r.amount, 0)) amount
 						FROM balance b
 							JOIN uninvoiced ui ON ui.companyFk = b.companyFk
-				LEFT JOIN receipt r ON r.dated > ui.dated AND r.companyFk = ui.companyFk
+							LEFT JOIN receipt r ON r.dated > ui.dated
+								AND r.companyFk = ui.companyFk
 						GROUP BY b.companyFk, ui.dated
 				)
 				SELECT ti.ticketFk, r.amount, ti.isProblemCalcNeeded
 					FROM ticket ti 
-				JOIN risk r ON r.dated = ti.dated AND r.companyFk = ti.companyFk;
+						JOIN risk r ON r.dated = ti.dated
+							AND r.companyFk = ti.companyFk;
 
 		UPDATE ticket t 
 				JOIN tTicketRisk tr ON tr.ticketFk = t.id
-			SET t.risk = tr.amount
+			SET t.risk = IF(tr.isProblemCalcNeeded, tr.amount, NULL);
-			WHERE tr.isProblemCalcNeeded
-			ORDER BY t.id;
-
-		UPDATE ticket t 
-				JOIN tTicketRisk tr ON tr.ticketFk = t.id
-			SET t.risk = NULL
-			WHERE tr.isProblemCalcNeeded
-			ORDER BY t.id;	
 
 		DROP TEMPORARY TABLE tTicketRisk;
 	END IF;