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

feat: refs #7335 Added call ticket_afterUpdate #2415

Merged
guillermo merged 7 commits from 7335-modifyCmr into dev 2024-05-08 11:54:03 +00:00
Conversation 0 Commits 7 Files Changed 2 +6 -11
2 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
db/routines/vn/procedures/ticket_doCmr.sql
View File

@ -7,7 +7,7 @@ BEGIN
*
* @param vSelf El id del ticket
*/
DECLARE vCmrFk, vPreviousCmrFk, vCurrentCmrFk INT;
DECLARE vCmrFk INT;
SELECT cmrFk INTO vCmrFk
FROM ticket
WHERE id = vSelf;
@ -44,8 +44,6 @@ BEGIN
AND t.id = vSelf
GROUP BY t.id;
START TRANSACTION;
IF vCmrFk THEN
UPDATE cmr c
JOIN tTicket t
@ -58,8 +56,6 @@ BEGIN
c.ead = t.landed
WHERE id = vCmrFk;
ELSE
SELECT MAX(id) INTO vPreviousCmrFk FROM cmr;
INSERT INTO cmr (
senderInstruccions,
truckPlate,
@ -71,16 +67,13 @@ BEGIN
)
SELECT * FROM tTicket;
SELECT MAX(id) INTO vCurrentCmrFk FROM cmr;
IF vPreviousCmrFk <> vCurrentCmrFk THEN
IF (SELECT EXISTS(SELECT * FROM tTicket)) THEN
UPDATE ticket
SET cmrFk = vCurrentCmrFk
SET cmrFk = LAST_INSERT_ID()
guillermo marked this conversation as resolved Outdated
jgallego commented 2024-05-07 13:15:52 +00:00
Outdated
Review
Copy Link

SQL Error [1054] [42S22]: Unknown column 'vCurrentCmrFk' in 'field list'

SQL Error [1054] [42S22]: Unknown column 'vCurrentCmrFk' in 'field list'
WHERE id = vSelf;
END IF;
END IF;
COMMIT;
DROP TEMPORARY TABLE tTicket;
END$$
DELIMITER ;

2
db/routines/vn/triggers/ticket_afterUpdate.sql
View File

@ -7,6 +7,8 @@ BEGIN
UPDATE expedition
SET hasNewRoute = TRUE
WHERE ticketFk = NEW.id;
CALL ticket_doCmr(NEW.id);
guillermo marked this conversation as resolved
jgallego commented 2024-05-07 13:17:52 +00:00
Review
Copy Link

tal com esta el if de dalt, si es posa una ruta a null o una ruta te null y se li asigna valor, en entra en este if...valora si això es correcte.

tal com esta el if de dalt, si es posa una ruta a null o una ruta te null y se li asigna valor, en entra en este if...valora si això es correcte.
guillermo commented 2024-05-08 08:28:16 +00:00
Review
Copy Link

Es correcto, ya que si es uno de los dos (NEW.id o OLD.id) es null, no queremos que entre.

Ya que el CMR se genera cuando se le cambia el estado al ticket.

Es correcto, ya que si es uno de los dos (NEW.id o OLD.id) es null, no queremos que entre. Ya que el CMR se genera cuando se le cambia el estado al ticket.
END IF;
END$$
DELIMITER ;