#5858 - zoneCollisions #1853
No reviewers
jgallego
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1853
Loading…
Reference in New Issue
No description provided.
Delete Branch "5858-zone_Collisions"
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: 5858-zoneCollisionsto WIP: #5858 - zoneCollisionsWIP: #5858 - zoneCollisionsto #5858 - zoneCollisions#5858 - zoneCollisionsto WIP: #5858 - zoneCollisionsWIP: #5858 - zoneCollisionsto #5858 - zoneCollisions@ -7,0 +7,4 @@-- Inserta en vn_schedules para ejecutar el procedure de 'zone_getCollisions'INSERT IGNORE INTO `util`.`vn_schedules`(`schema`, `procedure`)estamos generando una dependencia en util de vn.
Util no tiene que conocer vn.
Simplemente crea el schedule que se ejecute todas las noches en vn, ya està.
Para gestionar los métodos de notificaciones, sin embargo, esto, como ya no se gestiona mediante métodos, debería ir fuera. De hecho pone obsoleto porque pertenece al fixtures anterior.
Ya lo he eliminado
Le puse el campo id y el select, porque sin esto, me inserta como id 0.
Lo hice para que sea dinámico al insertar en la bd de prod.
Lo he movido a
838effadc0/db/versions/10881-greenHydrangea/01-notification.vnx.sqlDEPRECATED
DEPRECATED
@ -324,4 +324,5 @@ INSERT INTO mysql.roles_mapping (`User`, `Host`, `Role`, `Admin_option`)FROM mysql.roles_mappingWHERE `User` LIKE @prefixedLike AND `Host` = @genRoleHost;quitar salto
@ -9,2 +9,4 @@`changedModelId` = OLD.zoneFk,`userFk` = account.myUser_getId();no ponemos dos saltos de linea nunca
@ -0,0 +1,12 @@SET @MaxId = (SELECT MAX(id)+1 from util.notification n);dile a chatgpt que te cambie esta linea y que quieres usar el last_insert_id de forma que primero insertas y luego ya usas el id otorgado.
No hace falta ir tan lejos...tonto de mi por no haber revisado el resto del proyecto.
Basta con usar LAST_INSERT_ID()
Revisamos porque esta tabla empieza por 0.
Devuelve 0 LAST_INSERT_ID();
Solved, he modificado la propiedad auto_increment de la columna id de util.notification
@ -0,0 +5,4 @@INSERT IGNORE INTO util.notificationSubscription (notificationFk,userFk)VALUES(@MaxId, 100);que es este 100? el usuario 100 en prod es icef...no me cuadra.
Busca la forma de usar codes para no poner numeros.
INSERT IGNORE INTO util.notificationSubscription (notificationFk,userFk)
VALUES(
@MaxId, (SELECT id from
account.userwhere name = "system"));La solución estaba mas abajo
@ -0,0 +1,12 @@SET @MaxId = (SELECT MAX(id)+1 from util.notification n);INSERT IGNORE INTO util.notification (`id`, `name`,`description`)VALUES(@MaxId, 'zone-included','An email to notify zoneCollisions');pongo esto para descartar y que cuando tengas el nuevo pr me puedas pedir revision de nuevo
Corregido:
4994a836c7@ -0,0 +4,4 @@SET @MaxId = LAST_INSERT_ID();INSERT IGNORE INTO util.notificationSubscription (notificationFk,userFk)VALUES(@MaxId, (SELECT id from `account`.`user` where name = "system"));porque se la añades al usuario system? deberia de ser el que solicita la tarea no?
Es un evento el que dispara el procedure de zone_getCollisions, por tanto no hay usuario que haga la operación manual, si no que es el systema.
Me basé en los datos de producción util.notification where id=6
Si las especificaciones no son las correctas, hay algún sitio donde pueda tomar la referencia?
@jgallego Tras juntarme con Pablo, he cambiado el usuario de systema a root, porque por lo que me ha comentado, si el procedimiento se ejecuta en la BD es root y en salix es system