[SOLUCIONADO] Duda/consulta triggers

[Begotten]

Hola, me estoy complicando un poco trabajando con triggers, necesito saber como puedo comparar un valor de tipo date con la fecha actual dentro de un "IF". Después de modificar el trigger un poco logre que phpmyadmin lo aceptara pero ahora no puedo agregar datos a la tabla "cuenta", creo que el problema esta en la comparación del if.

Código MySQL:

BEGIN
    IF NEW.fecha <= date_format(curdate(),'%Y%m%d') THEN
        UPDATE cuenta SET 
            `saldo_manana_caja`= saldos.saldo_manana_caja + (NEW.ingreso_caja - NEW.egreso_caja),
            `saldo_manana_bci`= saldos.saldo_manana_bci + (NEW.ingreso_bci - NEW.egreso_bci),
            `saldo_manana_chi`= saldos.saldo_manana_chi + (NEW.ingreso_chile - NEW.egreso_chile),
            `saldo_manana_sta1`= saldos.saldo_manana_sta1 + (NEW.ingreso_santa_uno - NEW.egreso_santa_uno),
            `saldo_manana_sta2`= saldos.saldo_manana_sta2 + (NEW.ingreso_santa_dos - NEW.egreso_santa_dos),
            `saldo_manana_prove`= saldos.saldo_manana_prove + (NEW.ingreso_prove - NEW.egreso_prove),
            `saldo_manana_cliente`= saldos.saldo_manana_cliente + (NEW.ingreso_clte - NEW.egreso_clte),
            `saldo_manana_acre`= saldos.saldo_manana_acre + (NEW.ingreso_acre - NEW.egreso_acre),
            `saldo_manana_deud`= saldos.saldo_manana_deud + (NEW.ingreso_deud - NEW.egreso_deud),
            `saldo_manana_edo`= saldos.saldo_manana_edo + (NEW.ingreso_eduardo - NEW.egreso_eduardo),
            `saldo_manana_edu`= saldos.saldo_manana_edu + (NEW.ingreso_eduar - NEW.egreso_eduar),
            `caja_bancos`= saldos.saldo_manana_caja + saldos.saldo_manana_bci + saldos.saldo_manana_chi + saldos.saldo_manana_sta1 + saldos.saldo_manana_sta2,
            `saldo_general`= saldos.caja_bancos + saldos.saldo_manana_acre + saldos.saldo_manana_deud
        WHERE saldos.codigo = NEW.saldos_codigo
    END IF;
END
 

[gnzsoloyo]

Si el campo "fecha" de la tabla del trigger es un DATE, ¿para qué usas DATE_FORMAT()?
Y si no es undate, entonces tienes datos basura...

[Begotten]

Iniciado por gnzsoloyo

Si el campo "fecha" de la tabla del trigger es un DATE, ¿para qué usas DATE_FORMAT()?
Y si no es undate, entonces tienes datos basura...

El date_format se lo estoy dando a la curdate() que segun entiendo viene con horas, minutos y segundos incluidos, no al valor NEW.fecha.
Ademas si es que hago ese cambio, el error aun se presenta:
#1054 - Champ 'saldos.codigo' inconnu dans where clause
Este atributo es la clave primaria de saldos.

PD: no es un Update, es un Insert

[gnzsoloyo]

POr un lado, no, CURDATE() sólo devuelve una fecha, no las horas. El que devuelve todo el DATETIME es NOW().

#1054 - Champ 'saldos.codigo' inconnu dans where clause

Eso es otra cosa que no está relacionada con la fecha: Esa tabla no tiene ningún campo denominado "codigo", por lo que o es incorrecto, o no pertenece a la tabla indicada.
En realidad el problema es que estás indicando la tabla SALDOS, pero el UPDATE es para la tabla CUENTA...

¿Lo ves?

[Begotten]

Iniciado por gnzsoloyo

POr un lado, no, CURDATE() sólo devuelve una fecha, no las horas. El que devuelve todo el DATETIME es NOW().


Eso es otra cosa que no está relacionada con la fecha: Esa tabla no tiene ningún campo denominado "codigo", por lo que o es incorrecto, o no pertenece a la tabla indicada.
En realidad el problema es que estás indicando la tabla SALDOS, pero el UPDATE es para la tabla CUENTA...

¿Lo ves?

si, si entiendo pero si borro el trigger extrañamente no me sale ese error, ademas el trigger esta hecho para cuando se agreguen datos a "cuenta" se actualicen los valores de "saldos"

[gnzsoloyo]

Creo que no me estás entendiendo...
Mira:

Código MySQL:

UPDATE cuenta SET
            ...
WHERE saldos.codigo = NEW.saldos_codigo
 

¿Ves la última línea?
El WHERE debe aplicar sobre la tabla CUENTA. No sobre SALDOS.
Lo que debe existir es una cuenta cuyo ID se corresponda con alguno de los campos que entran como NEW. DE lo contrario, ¿a que cuenta le haces el UPDATE? ¿Cómo la relacionas con la tabla que posee el trigger?

¿Se entiende mejor?

[Begotten]

acabo de probar y solo sale ese error cuando cumple la condición del "IF" del trigger, si no entra a la condición no hay ningún error

[Begotten]

Fue un fail gigante mio, el update iba a la tabla "saldos", no a "cuenta" ademas algunas variables estaban mal escritas, disculpa por las molestias pero es mi primer proyecto real de informática y me tiene bastante estresado.

Codigo Correcto:

Código MySQL:

CREATE TRIGGER `actualizarSaldo` AFTER INSERT ON `cuenta`
 FOR EACH ROW IF NEW.fecha <= date_format(curdate(),'%Y%m%d') THEN
        UPDATE saldos SET 
            `saldo_manana_caja`= saldos.saldo_manana_caja + (NEW.ingreso_caja - NEW.egreso_caja),
            `saldo_manana_bci`= saldos.saldo_manana_bci + (NEW.ingreso_bci - NEW.egreso_bci),
            `saldo_manana_chi`= saldos.saldo_manana_chi + (NEW.ingreso_chile - NEW.egreso_chile),
            `saldo_manana_sta1`= saldos.saldo_manana_sta1 + (NEW.ingreso_santa_uno - NEW.egreso_santa_uno),
            `saldo_manana_sta2`= saldos.saldo_manana_sta2 + (NEW.ingreso_santa_dos - NEW.egreso_santa_dos),
            `saldo_manana_prove`= saldos.saldo_manana_prove + (NEW.ingreso_prove - NEW.egreso_prove),
            `saldo_manana_clte`= saldos.saldo_manana_clte + (NEW.ingreso_cliente - NEW.egreso_cliente),
            `saldo_manana_acre`= saldos.saldo_manana_acre + (NEW.ingreso_acre - NEW.egreso_acre),
            `saldo_manana_deud`= saldos.saldo_manana_deud + (NEW.ingreso_deud - NEW.egreso_deud),
            `saldo_manana_edo`= saldos.saldo_manana_edo + (NEW.ingreso_eduardo - NEW.egreso_eduardo),
            `saldo_manana_edu`= saldos.saldo_manana_edu + (NEW.ingreso_eduar - NEW.egreso_eduar),
            `caja_bancos`= saldos.saldo_manana_caja + saldos.saldo_manana_bci + saldos.saldo_manana_chi + saldos.saldo_manana_sta1 + saldos.saldo_manana_sta2,
            `saldo_general`= saldos.caja_bancos + saldos.saldo_manana_acre + saldos.saldo_manana_deud
        WHERE saldos.codigo = NEW.saldos_codigo;
    END IF
 

[gnzsoloyo]

Insisto en que le quites esto:

Código MySQL:

NEW.fecha <= date_format(curdate(),'%Y%m%d') THEN
 

Si el campo "fecha" de SALDOS es un DATE o DATETIME, debería ser así:

Código SQL:

IF DATE(NEW.fecha) <= CURDATE() THEN
 

No es buena práctica realizar conversiones de datos innecesarias, y en ese caso lo estás obligando a MySQL a realizar una doble conversión, primero a caracteres (DATE_FORMAT()), y luego a DATE, por comparar la cadena de texto contra una fecha.
No lo hagas. Reduce performance.

[Begotten]

Iniciado por gnzsoloyo

Insisto en que le quites esto:

Código MySQL:

NEW.fecha <= date_format(curdate(),'%Y%m%d') THEN
 

Si el campo "fecha" de SALDOS es un DATE o ATETIME, debería ser así:

Código SQL:

IF DATE(NEW.fecha) <= CURDATE() THEN
 

No es buena práctica realizar conversiones de datos innecesarias, y en ese caso lo estás obligando a MySQL a realizar una doble conversión, primero a caracteres (DATE_FORMAT()), y luego a DATE, por comparar la cadena de texto contra una fecha.
No lo hagas. Reduce performance.

dicho y hecho, muchas gracias por la ayuda

[gnzsoloyo]