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

#5858 - zoneCollisions #1853

Merged
jsegarra merged 78 commits from 5858-zone_Collisions into dev 2024-03-15 12:36:38 +00:00
Conversation 0 Commits 78 Files Changed 16 +4 -4
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 4994a836c7 - Show all commits

1
db/versions/10881-greenHydrangea/00-alterTableNotification.sql Normal file
View File

@ -0,0 +1 @@
ALTER TABLE util.notification MODIFY COLUMN id int(11) auto_increment NOT NULL;

7
db/versions/10881-greenHydrangea/01-notification.vn.sql
View File

@ -1,8 +1,7 @@
SET @MaxId = (SELECT MAX(id)+1 from util.notification n);
INSERT IGNORE INTO util.notification (`id`, `name`,`description`)
INSERT IGNORE INTO util.notification ( `name`,`description`)
jgallego marked this conversation as resolved Outdated
jgallego commented 2024-03-06 08:37:13 +00:00
Outdated
Review
Copy Link

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.

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.
jsegarra commented 2024-03-06 09:41:30 +00:00
Outdated
Review
Copy Link

No hace falta ir tan lejos...tonto de mi por no haber revisado el resto del proyecto.

Basta con usar LAST_INSERT_ID()

No hace falta ir tan lejos...tonto de mi por no haber revisado el resto del proyecto. Basta con usar LAST_INSERT_ID()
jsegarra commented 2024-03-06 09:49:55 +00:00
Outdated
Review
Copy Link

Revisamos porque esta tabla empieza por 0.
Devuelve 0 LAST_INSERT_ID();

Revisamos porque esta tabla empieza por 0. Devuelve 0 LAST_INSERT_ID();
jsegarra commented 2024-03-09 10:29:25 +00:00
Outdated
Review
Copy Link

Solved, he modificado la propiedad auto_increment de la columna id de util.notification

Solved, he modificado la propiedad auto_increment de la columna id de util.notification
VALUES
(@MaxId, 'zone-included','An email to notify zoneCollisions');
( 'zone-included','An email to notify zoneCollisions');
SET @MaxId = LAST_INSERT_ID();
jgallego marked this conversation as resolved Outdated
jgallego commented 2024-03-08 14:30:19 +00:00
Outdated
Review
Copy Link

pongo esto para descartar y que cuando tengas el nuevo pr me puedas pedir revision de nuevo

pongo esto para descartar y que cuando tengas el nuevo pr me puedas pedir revision de nuevo
jsegarra commented 2024-03-12 08:43:09 +00:00
Outdated
Review
Copy Link

Corregido: 4994a836c7

Corregido: 4994a836c7119a6c894e3600a78ba71438ecb410
INSERT IGNORE INTO util.notificationSubscription (notificationFk,userFk)
VALUES(
@MaxId, (SELECT id from `account`.`user` where name = "system"));
jgallego marked this conversation as resolved Outdated
jgallego commented 2024-03-12 06:12:36 +00:00
Outdated
Review
Copy Link

porque se la añades al usuario system? deberia de ser el que solicita la tarea no?

porque se la añades al usuario system? deberia de ser el que solicita la tarea no?
jsegarra commented 2024-03-12 08:49:01 +00:00
Outdated
Review
Copy Link

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?

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?
jsegarra commented 2024-03-12 09:22:38 +00:00
Outdated
Review
Copy Link

@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

@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