1

Tengo el siguiente Trigger comento que no tengo mucha experiencia en ello:

delimiter |

CREATE TRIGGER insertar_control AFTER UPDATE ON bienes
  FOR EACH ROW
  BEGIN

    INSERT INTO bienes_control SET Id_Bien = NEW.Id_Bien;
    INSERT INTO bienes_control SET Activo = NEW.Activo;
    INSERT INTO bienes_control SET SubActivo = NEW.SubActivo;
    INSERT INTO bienes_control SET Numero_Empleado = NEW.Numero_Empleado;
    INSERT INTO bienes_control SET Empleado_Movimiento = NEW.Empleado_Movimiento;
    INSERT INTO bienes_control SET Cotejado = NEW.Cotejado;
    INSERT INTO bienes_control SET Proceso = NEW.Proceso;

  END;
|

delimiter ;

Me base en Sintaxis Mysql , lo que pretendo realizar es lo siguiente, en cierto momento del sistema se actualizan dos campos de la tabla bienes, al momento que estos sean actualizados, se dispare el trigger y haga una copia de los campos que se han indicado en el trigger y estos sean insertados en la tabla bienes_control en un solo registro en una sola fila o renglón.

El trigger que se indica al inicio tal cual como esta codificado, hace su función, solo que inserta un valor en el campo de la tabla indicado, y luego rellena con ceros varios renglones, luego inserta otro campo y así se sigue hasta terminar de copiar los campos indicados generando una "tablota" de datos inservible.

Hice una modificación el día de ayer al Trigger dejándolo de esta manera:

delimiter |

CREATE TRIGGER insertar_control AFTER UPDATE ON bienes
  FOR EACH ROW
  BEGIN

     IF NEW.Cotejado == 1
     THEN

    INSERT INTO bienes_control SET Id_Bien = NEW.Id_Bien;
    INSERT INTO bienes_control SET Activo = NEW.Activo;
    INSERT INTO bienes_control SET SubActivo = NEW.SubActivo;
    INSERT INTO bienes_control SET Numero_Empleado = NEW.Numero_Empleado;
    INSERT INTO bienes_control SET Empleado_Movimiento = NEW.Empleado_Movimiento;
    INSERT INTO bienes_control SET Cotejado = NEW.Cotejado;
    INSERT INTO bienes_control SET Proceso = NEW.Proceso;

    END If;

  END;
|

delimiter ;

Y el resultado obtenido que se muestra desde phpmyadmin: Vista del resultado

Como se puede observar me genera ahora 7 registros o renglones, lo que necesito es que todos estos campos queden solo renglon en su respectivo campo.

5
  • Lo que te está pasando es la consecuencia de utilizar INSERT en vez de UPDATE: cada INSERT es un nuevo registro, que es lo que estás obteniendo.
    – David JP
    Commented el 22 may. 2020 a las 16:39
  • David, entonces si cambio los INSERT por UPDATE haría digamos un loop de update con cada campo hasta que termine y me lo dejaría todo en un registro..
    – Cesarin
    Commented el 22 may. 2020 a las 16:41
  • Tendrás que hacer un INSERT para el primero, pero el resto con UPDATE exacto, como en la respuesta que te he pasado
    – David JP
    Commented el 22 may. 2020 a las 16:42
  • Ok deja modificar en base a tus sugerencias. GRacias!!!!
    – Cesarin
    Commented el 22 may. 2020 a las 16:54
  • 1
    No, que voy a implementar lo que me comentas de hacer primero un INSERT y posteriormente los uPDATES basado en el trigger que amablemente me has compartido. Solo que primero tengo que hacer un reporte de mi trabajo :)
    – Cesarin
    Commented el 22 may. 2020 a las 17:01

3 respuestas 3

3

Lo que estás buscando parece ser algo así:

CREATE TRIGGER insertar_control AFTER UPDATE ON bienes
  FOR EACH ROW
  BEGIN
    IF NEW.Cotejado!=OLD.Cotejado AND NEW.Proceso!=OLD.Proceso THEN
      INSERT INTO bienes_control (Id_Bien) VALUES (NEW.Id_Bien);
      IF NEW.Activo!=OLD.Activo THEN
        UPDATE bienes_control SET Activo=NEW.Activo 
          WHERE Id_Bien=NEW.Id_Bien;
      END IF;
      IF NEW.SubActivo!=OLD.SubActivo THEN
        UPDATE bienes_control SET SubActivo=NEW.SubActivo 
          WHERE Id_Bien=NEW.Id_Bien;
      END IF;
      IF NEW.Numero_Empleado!=OLD.Numero_Empleado THEN
        UPDATE bienes_control SET Numero_Empleado=NEW.Numero_Empleado 
          WHERE Id_Bien=NEW.Id_Bien;
      END IF;
      IF NEW.Empleado_Movimiento!=OLD.Empleado_Movimiento THEN
        UPDATE bienes_control SET Empleado_Movimiento=NEW.Empleado_Movimiento 
          WHERE Id_Bien=NEW.Id_Bien;
      END IF;
      IF NEW.Cotejado!=OLD.Cotejado THEN
        UPDATE bienes_control SET Cotejado=NEW.Cotejado 
          WHERE Id_Bien=NEW.Id_Bien;
      END IF;
      IF NEW.Proceso!=OLD.Proceso THEN
        UPDATE bienes_control SET Proceso=NEW.Proceso 
          WHERE Id_Bien=NEW.Id_Bien;
      END IF;
    END IF;
  END;

Para el dataset de pruebas:

CREATE TABLE bienes(
  Id_Bien int AUTO_INCREMENT PRIMARY KEY,
  Activo varchar(63),
  SubActivo varchar(63),
  Numero_Empleado int,
  Empleado_Movimiento int,
  Cotejado bool,
  Proceso int
  );
CREATE TABLE bienes_control(
  Id_Bien int,
  Activo varchar(63),
  SubActivo varchar(63),
  Numero_Empleado int,
  Empleado_Movimiento int,
  Cotejado bool,
  Proceso int
  );
INSERT INTO bienes (
    Activo,SubActivo,Numero_Empleado,
    Empleado_Movimiento,Cotejado,Proceso
  ,Id_Bien) VALUES ('A','S',1,1,1,1);

Por cada UPDATE en bienes en los que se modifique simultáneamente Cotejado y Proceso, crea un nuevo registro en bienes_control con todos los campos a NULL excepto aquellos que también hayan sido modificados.

Como indicas, habrá un registro por cada cambio.

Espero que sea lo que buscabas. Déjame cualquier duda en los comentarios para ampliar la respuesta de la forma más ajustada a tu cuestión.

7
  • Buenos días, agradezco tu aportación , ya carge las tablas , y bueno ya no se si sea una falla de mi phpmyadmin, que de todo me envia errores, me indica que hay un error: #1064 - You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 4 y bueno de entrada a simple vista no le veo problema en la sintaxis pero habre de revisar
    – Cesarin
    Commented el 22 may. 2020 a las 14:18
  • Estuve revisando las sentencias, y estoy entendiendo que indican en general, que si encuentra un valor diferente en algún campo en bienes, entonces replica este cambio en bienes_control , pero solo copia el cambio hecho y lo demas queda null ?
    – Cesarin
    Commented el 22 may. 2020 a las 14:22
  • Exacto, ¿no era eso es lo que pedías?
    – David JP
    Commented el 22 may. 2020 a las 15:12
  • En este sentido creo que yo fui el que cometio una omision al redactar, el trigger que tengo indicado al inicio hace su funcion, si pero no del todo bien, lo que intente hacer era que al momento que los campos Cotejado y Proceso fueran actualizados simultaneamente , entonces el trigger hiciera una copia de ciertos campos de la tabla bienes a la de bienes control. pero que esto lo haga en un solo registro, y no escalonado que es como lo ejecuta ese trigeer
    – Cesarin
    Commented el 22 may. 2020 a las 15:12
  • Claro, es que, de momento en Stackoverflow, adivinos aún no somos... :) Modifica la pregunta con especificaciones concretas y modificaré la respuesta acorde a éstas. Por lo que comentas, no parece complicado. Ya me dices
    – David JP
    Commented el 22 may. 2020 a las 15:22
0

Hice algunas modificaciones y me quedo esto:

delimiter |

CREATE TRIGGER insertar_control AFTER UPDATE ON bienes FOR EACH ROW BEGIN

 IF NEW.Cotejado == 1
 THEN
 INSERT INTO bienes_control SET Id_Bien = NEW.Id_Bien;
 UPDATE bienes_control SET Activo = NEW.Activo;
 UPDATE bienes_control SET SubActivo = NEW.SubActivo;
 UPDATE bienes_control SET Numero_Empleado = NEW.Numero_Empleado;
 UPDATE bienes_control SET Empleado_Movimiento = NEW.Empleado_Movimiento;
 UPDATE bienes_control SET Cotejado = NEW.Cotejado;
 UPDATE bienes_control SET Proceso = NEW.Proceso
 WHERE Id_Bien = NEW.Id_Bien;

END If;

END; |

delimiter ;

Para un registro que se actualiza desde bienes funciona muy bien, pero si tengo otro update posterior el insert me mezcla datos de la tabla origen en la tabla destino.

Sigo revisando como mejorar esto.

Saludos

3
  • 1
    Necesitas un WHERE por cada UPDATE, de lo contrario cada UPDATE te modificará todos los registros de bienes_contral. Pasa igual que con el DELETE FROM youtube.com/watch?v=i_cVJgIz_Cs
    – David JP
    Commented el 22 may. 2020 a las 21:25
  • ok Gracias David, ya te estare comentando como va, muchas gracias por tu tiempo y consejos. Buen fin de Semana!!!
    – Cesarin
    Commented el 23 may. 2020 a las 2:12
  • Por favor, no agregues respuestas mas alla de la respuesta aceptada, ya que solo generan ruido. Mas si pones mas de una respuesta, no se entiende en realidad cual es la respuesta que ayuda!
    – gbianchi
    Commented el 2 jun. 2020 a las 18:32
0

finalmente asi quedo mi trigger por si a alguien le es de utilidad.

BEGIN

 IF NEW.Cotejado != 0 AND NEW.Proceso != 0
 THEN
 INSERT INTO bienes_control SET Id_Bien = NEW.Id_Bien;
 UPDATE bienes_control SET Activo = NEW.Activo
 WHERE Id_Bien = NEW.Id_Bien;
 END IF;
     
 IF NEW.Cotejado != 0 AND NEW.Proceso != 0
 THEN
 UPDATE bienes_control SET SubActivo = NEW.SubActivo
 WHERE Id_Bien = NEW.Id_Bien;
 END IF;
     
 IF NEW.Cotejado != 0 AND NEW.Proceso != 0
 THEN
 UPDATE bienes_control SET Numero_Empleado = NEW.Numero_Empleado
 WHERE Id_Bien = NEW.Id_Bien;
 END IF;
 
 IF NEW.Cotejado != 0 AND NEW.Proceso != 0
 THEN
 UPDATE bienes_control SET Empleado_Movimiento = NEW.Empleado_Movimiento
 WHERE Id_Bien = NEW.Id_Bien;
 END IF;
 
 IF NEW.Cotejado != 0 AND NEW.Proceso != 0
 THEN
 UPDATE bienes_control SET Cotejado = NEW.Cotejado
 WHERE Id_Bien = NEW.Id_Bien;
 END IF;
 
 IF NEW.Cotejado != 0 AND NEW.Proceso != 0
 THEN
 UPDATE bienes_control SET Proceso = NEW.Proceso
 WHERE Id_Bien = NEW.Id_Bien;
 END IF;
     
END

Gracias por sus aportaciones.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.