#2687 - Fix Travel_cloneWithEntries #2021
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#2021
Loading…
Reference in New Issue
No description provided.
Delete Branch "2687_fix_travel_cloneWithEntries"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
WIP: #2687 - Fix Travel_cloneWithEntriesto #2687 - Fix Travel_cloneWithEntries@ -31,0 +30,4 @@
await models.Entry.destroyAll({
travelFk: newTravelId
}, options);
await models.Travel.destroyById(newTravelId, options);
Comentario de Alex: Aunque fasa rollback cal borrar les entrades i el travel???
Si, no hace rollback de las líneas clonadas
Falta pasar de "versions" a archivos
New commits pushed, approval review dismissed automatically according to repository settings
68e90f8607
tob9a624d9cd
Aquí algo
@ -0,0 +1,13 @@
DELIMITER $$
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `util`.`tx_commit`(IN tx BOOL)
El IN sobra, además, toda variable booleana tiene que tener el prefijo isXXX, hasXXXX (https://wiki.verdnatura.es/index.php/Convenciones_SQL#Booleano)
@ -0,0 +2,4 @@
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `util`.`tx_commit`(IN tx BOOL)
BEGIN
/**
* Procedimiento para confirmar los cambios asociados a una transacción
Lo de procedimiento es redundante, cambiar a: "Confirma los cambios asociados a una transacción."
Poner el punto al final del comentario, ya que se suele poner.
d9756a0f3f
@ -0,0 +4,4 @@
/**
* Procedimiento para confirmar los cambios asociados a una transacción
*
* @param tx BOOL es true si existe transacción asociada
Aquí sobra lo de BOOL y la primera letra después de la variable debe de estar en mayúscula (https://wiki.verdnatura.es/index.php/Convenciones_SQL#Documentaci%C3%B3n)
d9756a0f3f
d9756a0f3f
@ -0,0 +1,13 @@
DELIMITER $$
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `util`.`tx_rollback`(tx BOOL)
Ahora una duda general para todos los procedimientos creados, por que has puesto tx? Entiendo que es lo mismo quitando la variable y quitando el if, si quieres lo llamas y si no no.
Entiendo que hablas del if que hay definido dentro de cada procedure.
Se hace así para no tener que poner el IF desde cada llamada.
Si no he respondido a tu pregunta, nos juntamos
@ -0,0 +2,4 @@
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `util`.`tx_rollback`(tx BOOL)
BEGIN
/**
* Procedimiento para deshacer los cambios asociados a una transacción
Cambiar a: "Deshace los cambios asociados a una transacción."
d9756a0f3f
@ -0,0 +1,15 @@
Quitar esos 2 saltos de linea
d9756a0f3f
@ -0,0 +4,4 @@
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `util`.`tx_start`(tx BOOL)
BEGIN
/**
* Procedimiento para iniciar una transacción
Cambiar a: "Inicia una transacción."
d9756a0f3f
@guillermo El formato de los procedimientos está extraido de este comentario de Juan
https://redmine.verdnatura.es/issues/2687#note-14
New commits pushed, approval review dismissed automatically according to repository settings
Entonces supongo que habrá preferido ahorrar código en las llamadas al procedimiento que en el propio procedimiento, a la larga sí que es verdad que te ahorras código y a la hora de leer es mejor.