[SOLUCIONADO] obtener resultado de procedimiento desde un disparador

[tanliz]

Hola! buen día... Les explico mi problema, estoy tratando de obtener el resultado que me arroja un procedimiento, con parametro de salida, llamandolo desde un disparador, pero me arroja un error en el insert. Les muestro el codigo del disparador, el procedimiento si me funciona, pues ya lo he ejecutado antes...
espero me puedan ayudar, de antemano GRACIAS!

Código:

DELIMITER $$
USE `proiplus`$$
DROP TRIGGER /*!50032 IF EXISTS */ `Actualiza_Cuentas_Por_Cobrar`$$

CREATE
    /*!50017 DEFINER = 'proi'@'localhost' */
    TRIGGER `Actualiza_Cuentas_Por_Cobrar` AFTER INSERT ON `detalle_cuentas_por_cobrar` 
    FOR EACH ROW BEGIN
	DECLARE nuevoAbono DECIMAL(10,2);
	DECLARE nuevoSaldo DECIMAL(10,2);
	DECLARE nuevaLlave VARCHAR(5);
	
	SET nuevoAbono =(SELECT Abono FROM cuentas_por_cobrar WHERE Cuenta_Por_Cobrar_Id=new.Cuenta_Por_Cobrar_Id)+new.Abono;
	SET nuevoSaldo = (SELECT Saldo FROM cuentas_por_cobrar WHERE Cuenta_Por_Cobrar_Id=new.Cuenta_Por_Cobrar_Id) - new.Abono;
	
	UPDATE cuentas_por_cobrar
	SET Abono=nuevoAbono, Saldo=nuevoSaldo
	WHERE Cuenta_Por_Cobrar_Id=new.Cuenta_Por_Cobrar_Id;
	
	IF (STRCMP(new.FormaPago, "Efectivo")==0) THEN
		CALL Crear_Llave_Caja(@llave);
		SET nuevaLlave=(SELECT @llave);
		INSERT INTO caja (Caja_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descipcion, Monto, Numero_Folio, Estatus_Registro)
		VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
	ELSE THEN
		CALL Crear_Llave_Bancos(@llave);
		SET nuevaLlave=(SELECT @llave);
		INSERT INTO bancos (Bancos_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descipcion, Monto, Numero_Folio, Estatus_Registro)
		VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
	END IF;
    END;
$$
DELIMITER ;

Y este es el error:

Error Code: 1064
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '==0) then
call Crear_Llave_Caja(@llave);
set nuevaLlave=(select @llave);
i' at line 16

[gnzsoloyo]

MySQL no admite llamar rutinas almacenadas (procedures, o functions), desde un TRIGGER.

[tanliz]

Gracias gnzsoloyo... pero ahora he cambiado la llamada del procedimiento por una simple consulta pero me sigue marcando error...
puse esto:

Código MySQL:

IF (STRCMP(new.FormaPago, "Efectivo")==0) THEN
        SET nuevaLlave=(SELECT IFNULL(MAX(Caja_Id),0)+1 FROM caja);
        INSERT INTO caja (Caja_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descripcion, Monto, Numero_Folio, Estatus_Registro)
        VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
    ELSE THEN
        SET nuevaLlave=(SELECT IFNULL(MAX(Bancos_Id),0)+1 FROM bancos);
        INSERT INTO bancos (Bancos_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descipcion, Monto, Numero_Folio, Estatus_Registro)
        VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
    END IF;
 

[gnzsoloyo]

Partamos de la base que no existen las pseudovariables NEW y OLD en los SP. Son exclusivamente de los TRIGGER, así que debes adaptar eso para comenzar.

Esto tampoco está bien:

Código MySQL:

ELSE THEN
 

Sólo va ELSE.

[tanliz]

Mmm ya quite el "then" en el "else" (persiste el error) y las pseudovariables las estoy utilizando en el trigger, no en el procedimento... o q te refieres?

[gnzsoloyo]

Por tu mensaje parecía entenderse que estabas cambiando de un TRIGGER a un SP.

Si es el SP, entonces pruébalo con las pseudovariables... Pero postea todo el mensaje de error y código del trigger. A veces el error está más adelante de lo que menciona.

[tanliz]

Este es el trigger:

Código MySQL:

DELIMITER $$
USE `proiplus`$$
DROP TRIGGER /*!50032 IF EXISTS */ `Actualiza_Cuentas_Por_Cobrar`$$
 
CREATE
    /*!50017 DEFINER = 'proi'@'localhost' */
    TRIGGER `Actualiza_Cuentas_Por_Cobrar` AFTER INSERT ON `detalle_cuentas_por_cobrar` 
    FOR EACH ROW BEGIN
    DECLARE nuevoAbono DECIMAL(10,2);
    DECLARE nuevoSaldo DECIMAL(10,2);
    DECLARE nuevaLlave VARCHAR(5);
    DECLARE FormaPago VARCHAR(20);
    
    SET nuevoAbono =(SELECT Abono FROM cuentas_por_cobrar WHERE Cuenta_Por_Cobrar_Id=new.Cuenta_Por_Cobrar_Id)+new.Abono;
    SET nuevoSaldo = (SELECT Saldo FROM cuentas_por_cobrar WHERE Cuenta_Por_Cobrar_Id=new.Cuenta_Por_Cobrar_Id) - new.Abono;
    
    UPDATE cuentas_por_cobrar
    SET Abono=nuevoAbono, Saldo=nuevoSaldo
    WHERE Cuenta_Por_Cobrar_Id=new.Cuenta_Por_Cobrar_Id;
    
    
    IF (STRCMP(new.Forma_Pago, "Efectivo")==0) THEN
        SET nuevaLlave=(SELECT IFNULL(MAX(Caja_Id),0)+1 FROM caja);
        INSERT INTO caja (Caja_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descripcion, Monto, Numero_Folio, Estatus_Registro)
        VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
    ELSE
        SET nuevaLlave=(SELECT IFNULL(MAX(Bancos_Id),0)+1 FROM bancos);
        INSERT INTO bancos (Bancos_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descipcion, Monto, Numero_Folio, Estatus_Registro)
        VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
    END IF;
    END;
$$
DELIMITER ;
 
 
y este es el error: 
 
 queries executed, 2 success, 1 errors, 0 warnings
 
Query: USE `proiplus`
 
Query: USE `proiplus`
 
0 row(s) affected
 
Execution Time : 0 sec
Transfer Time  : 0 sec
Total Time     : 0 sec
--------------------------------------------------
 
Query: DROP TRIGGER /*!50032 IF EXISTS */ `Actualiza_Cuentas_Por_Cobrar`
 
Query: DROP TRIGGER /*!50032 IF EXISTS */ `Actualiza_Cuentas_Por_Cobrar`
 
0 row(s) affected, 1 warning(s)
 
Execution Time : 0 sec
Transfer Time  : 0 sec
Total Time     : 0 sec
 
Note: To see complete list of warning(s), enable Tools -> Preferences -> General -> Show Warning(s) in Messages Tab
--------------------------------------------------
 
Query: CREATE /*!50017 DEFINER = 'proi'@'localhost' */ TRIGGER `Actualiza_Cuentas_Por_Cobrar` AFTER INSERT ON `detalle_cuentas_por_cobr...
 
Query: CREATE /*!50017 DEFINER = 'proi'@'localhost' */ TRIGGER `Actualiza_Cuentas_Por_Cobrar` AFTER INSERT ON `detalle_cuentas_por_cobr...
 
Error Code: 1064
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '==0) then
        set nuevaLlave=(SELECT IFNULL(MAX(Caja_Id),0)+1 FROM caja);
        insert' at line 18
 
Execution Time : 0 sec
Transfer Time  : 0 sec
Total Time     : 0 sec
 

[tanliz]

ya encontre el error: :P estaba poniendo un 0 de mas en la condicion

quedo asi:
IF (STRCMP(new.Forma_Pago, "Efectivo")=0) THEN...

muchas gracias por tu atencion!

[tanliz]

y si me funciono llamar al procedimiento en el trigger... por si a alguien le interesa...
quedo asi:

Código:

IF (STRCMP(new.Forma_Pago, "Efectivo")=0) THEN
		CALL Crear_Llave_Caja(@llave);
		SET nuevaLlave=(SELECT @llave);
		INSERT INTO caja (Caja_Id, Tipo_Id, Origen_Id, Ingreso_Id, Fecha, Descripcion, Monto, Numero_Folio, Estatus_Registro)
		VALUES (nuevaLlave, 'Ingreso', 'Cuentas_Por_Cobrar', new.Detalle_Cuentas_Por_Cobrar_Id, new.Fecha, 'Abono de cliente', new.Abono, new.Moneda, 'Activo');
	ELSE.....

[gnzsoloyo]

Veamos:
1) Las comparaciones en MySQL son con "=", no "==". Ten en cuenta siempre que los operadores lógicos no necesariamente son iguales entre diferentes DBMS.
2) Las comillas (") y los apóstrofos no son lo mismo. Las cadenas de texto se encierran entre apóstrofos (para usar comillas hay que modificar la configuracion del servidor).

Usa:

Código MySQL:

IF STRCMP(new.Forma_Pago, 'Efectivo') = 0 THEN
 

[tanliz]

si, tienes razon, he hecho me equivoque en mi respuesta, quise decir q habia escrito un "=" de as en mi condicion y en lugar de eso escribi un "0"...
pero ya me funciono, GRACIAS!