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

fixes #4951 Comprobar longitud de cuentas contables al generar asientos #1238

Merged
alexandre merged 5 commits from 4951-longitud-cuentaContables into dev 2023-01-17 07:30:37 +00:00
Conversation 0 Commits 5 Files Changed 1 +83
1 changed files with 83 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f499311a16 - Show all commits

83
db/changes/230201/00-triggersXDiario.sql Normal file
View File

@ -0,0 +1,83 @@
DROP TRIGGER IF EXISTS vn.XDiario_beforeUpdate;
USE vn;
DELIMITER $$
$$
CREATE DEFINER=`root`@`localhost` TRIGGER `vn`.`XDiario_beforeUpdate`
BEFORE UPDATE ON `XDiario`
FOR EACH ROW
BEGIN
IF NOT NEW.SUBCTA <=> OLD.SUBCTA THEN
IF NOT util.checkLength(NEW.SUBCTA, 10) THEN
jgallego commented 2023-01-10 06:40:41 +00:00
Review
Copy Link

Este if, sobraria, perque si li pases '' la funció length ja et trau el fallo

Este if, sobraria, perque si li pases '' la funció length ja et trau el fallo
alexandre commented 2023-01-10 07:00:44 +00:00
Review
Copy Link

Ho gaste per a transformar '' en NULL. Me digué Carlos que ho posara aixina perque lo correcte seria que no se pogueren tindre camps buits (que es el que passa si borres directament el camp en la base de dades), sinó camps en NULL.

Ho gaste per a transformar '' en NULL. Me digué Carlos que ho posara aixina perque lo correcte seria que no se pogueren tindre camps buits (que es el que passa si borres directament el camp en la base de dades), sinó camps en NULL.
jgallego commented 2023-01-11 10:32:04 +00:00
Review
Copy Link

Ho entenc, pero si just baix valides la longitud, en cap cas insertes '' ni null, sino que trau fallo.

Ho entenc, pero si just baix valides la longitud, en cap cas insertes '' ni null, sino que trau fallo.
alexandre commented 2023-01-11 10:39:54 +00:00
Review
Copy Link

En el casos que siga '' o NULL no trau fallo, només trau fallo si la cadena no és null i la longitud no és 10

En el casos que siga '' o NULL no trau fallo, només trau fallo si la cadena no és null i la longitud no és 10
CALL util.throw('INVALID_LENGTH');
END IF;
END IF;
jgallego commented 2023-01-10 06:38:53 +00:00
Outdated
Review
Copy Link

has valorat posar directament LENGTH(vString) <=> 10 en el if?
i així no creem la funcio?

has valorat posar directament LENGTH(vString) <=> 10 en el if? i així no creem la funcio?
alexandre commented 2023-01-10 07:01:44 +00:00
Outdated
Review
Copy Link

Me va dir Carlos que creara la funció i la posara en util perque aixina se pot gastar en més llocs

Me va dir Carlos que creara la funció i la posara en util perque aixina se pot gastar en més llocs
jgallego commented 2023-01-11 07:46:07 +00:00
Outdated
Review
Copy Link

He parlat en Juan i ell tampoc crearia una funció, ni aporta molta legibilitat, ni lleva linies de codi.
No es justifica.

He parlat en Juan i ell tampoc crearia una funció, ni aporta molta legibilitat, ni lleva linies de codi. No es justifica.
IF NOT NEW.CONTRA <=> OLD.CONTRA THEN
IF NOT util.checkLength(NEW.CONTRA, 10) THEN
CALL util.throw('INVALID_LENGTH');
END IF;
END IF;
IF NOT NEW.FECHA <=> OLD.FECHA THEN
CALL XDiario_checkDate(NEW.FECHA);
END IF;
IF NOT NEW.FECHA_EX <=> OLD.FECHA_EX THEN
CALL XDiario_checkDate(NEW.FECHA_EX);
END IF;
IF NOT NEW.FECHA_OP <=> OLD.FECHA_OP THEN
CALL XDiario_checkDate(NEW.FECHA_OP);
END IF;
IF NOT NEW.FECHA_RT <=> OLD.FECHA_RT THEN
CALL XDiario_checkDate(NEW.FECHA_RT);
END IF;
IF NOT NEW.FECREGCON <=> OLD.FECREGCON THEN
CALL XDiario_checkDate(NEW.FECREGCON);
END IF;
END$$
DELIMITER ;
DROP TRIGGER IF EXISTS vn.XDiario_beforeInsert;
USE vn;
DELIMITER $$
$$
CREATE DEFINER=`root`@`localhost` TRIGGER `vn`.`XDiario_beforeInsert`
BEFORE INSERT ON `XDiario`
FOR EACH ROW
BEGIN
IF NOT util.checkLength(NEW.SUBCTA, 10) THEN
CALL util.throw('INVALID_LENGTH');
END IF;
IF NOT util.checkLength(NEW.CONTRA, 10) THEN
CALL util.throw('INVALID_LENGTH');
END IF;
CALL XDiario_checkDate(NEW.FECHA);
CALL XDiario_checkDate(NEW.FECHA_EX);
CALL XDiario_checkDate(NEW.FECHA_OP);
CALL XDiario_checkDate(NEW.FECHA_RT);
CALL XDiario_checkDate(NEW.FECREGCON);
END$$
DELIMITER ;
DROP FUNCTION IF EXISTS `util`.`checkLength`;
DELIMITER $$
$$
CREATE DEFINER=`root`@`localhost` FUNCTION `util`.`checkLength`(vString VARCHAR(10), vLength INT(11)) RETURNS tinyint(1)
DETERMINISTIC
BEGIN
/**
* Comprueba la longitud de un string
*
* @param vString String a comprobar
* @param vLength Longitud que debe tener
* @return Devuelve TRUE/FALSE en caso de que tenga la longitud correcta o no
*/
IF LENGTH(vString) <=> vLength THEN
RETURN TRUE;
END IF;
RETURN FALSE;
END$$
DELIMITER ;