Manejo de fecha en procedimiento almacenados

[saumal20]

buen dia, espero ke me puedan ayudar. Tengo el siguiente PA:

Código SQL:

CREATE PROCEDURE PA_8_actualizar_compras(numerocomprobante INT(12), motivoanulacioncompras text)
    BEGIN
        DECLARE fehoanulacioncompra VARCHAR(19);
        SELECT CURRENT_TIMESTAMP() INTO fehoanulacioncompra;
        UPDATE compras
        SET anulacioncompras = 1, fehoanulacioncompra = @fehaanulacioncompra, motivoanulacioncompras = @motivoanulacioncompras;
        WHERE numerocomprobantecompras = @numerocomprobante;
    END;
 

en cual kiero actualizar una compra cambiar su estado a anulada, pero kiero asignarle la fecha de la base de datos, me muestra error...alguna ayuda...please

saludos

[huesos52]

Cual error?

[saumal20]

Iniciado por huesos52

Cual error?

un error de sintaxis, cerca a la linea 3

[diegonazho]

el into termina con ";" puede que sea eso

[saumal20]

Iniciado por diegonazho

el into termina con ";" puede que sea eso

???????????

[gnzsoloyo]

Estás usando incorrectamente el nombre de las variables locales. Acuérdate de que no estás programando para SQL Server, sino para mySQL.
Las variables locales no tiene prefijos, si estás recibiendo la variable numerocomprobante, no se le debe poner el prefijo @numerocomprobante, porque esa sería otra variable diferente.
Sólo las variables de usuario (de sesión) usan prefijo, pero no las locales de un procedimiento.

Lo correcto sería:

Código MySQL:

CREATE procedure PA_8_actualizar_compras(IN numerocomprobante INT, IN motivoanulacioncompras TEXT)
BEGIN
  DECLARE fehoanulacioncompra VARCHAR(19);
  SELECT CURRENT_timestamp() INTO fehoanulacioncompra;
  UPDATE compras
  SET anulacioncompras = 1, fehoanulacioncompra = fehaanulacioncompra, 
        motivoanulacioncompras = motivoanulacioncompras;
   WHERE numerocomprobantecompras = numerocomprobante;
END;
 

Tips:
- No uses nombres tan largos. Terminarás teniendo problemas de sintaxis al tipearlos.
- No uses VARCHAR para almacenar un TIMESTAMP. Usa los tipos de datos correctos. Un TIMESTAMP tiene una longitud real en memoria y disco de 8 bytes, mientras que el VARCHAR necesita 19 para hacer lo mismo. Y eso es simplemente porque los datos de fecha y hora se almacenan como binarios y no como caracteres, al igual que los números.

[saumal20]

Iniciado por gnzsoloyo

Estás usando incorrectamente el nombre de las variables locales. Acuérdate de que no estás programando para SQL Server, sino para mySQL.
Las variables locales no tiene prefijos, si estás recibiendo la variable numerocomprobante, no se le debe poner el prefijo @numerocomprobante, porque esa sería otra variable diferente.
Sólo las variables de usuario (de sesión) usan prefijo, pero no las locales de un procedimiento.

Lo correcto sería:

Código MySQL:

CREATE procedure PA_8_actualizar_compras(IN numerocomprobante INT, IN motivoanulacioncompras TEXT)
BEGIN
  DECLARE fehoanulacioncompra VARCHAR(19);
  SELECT CURRENT_timestamp() INTO fehoanulacioncompra;
  UPDATE compras
  SET anulacioncompras = 1, fehoanulacioncompra = fehaanulacioncompra, 
        motivoanulacioncompras = motivoanulacioncompras;
   WHERE numerocomprobantecompras = numerocomprobante;
END;
 

Tips:
- No uses nombres tan largos. Terminarás teniendo problemas de sintaxis al tipearlos.
- No uses VARCHAR para almacenar un TIMESTAMP. Usa los tipos de datos correctos. Un TIMESTAMP tiene una longitud real en memoria y disco de 8 bytes, mientras que el VARCHAR necesita 19 para hacer lo mismo. Y eso es simplemente porque los datos de fecha y hora se almacenan como binarios y no como caracteres, al igual que los números.

mira lo deje asi:

Código MySQL:

create procedure PA_8_actualizar_compras(numerocomprobantes int(12), motivoanulacioncompras text)
    BEGIN
        DECLARE fehoanulacioncompra timestamp;
        select CURRENT_timestamp() into @fehoanulacioncompra,
        UPDATE compras
        set anulacioncompras = 1, fehoanulacioncompra = @fehaanulacioncompra, motivoanulacioncompras = @motivoanulacioncompras;
        WHERE numerocomprobantes= @numerocomprobantes;
    END;
 

si tenia los @, solo ke me ekivoke al elegir el tipo de consulta, bueno igual me da error de sintaxis

[gnzsoloyo]

Quítale los"@" a las variables.
Estás comparando y asignando valores NULL en esos campos, porque @a y a no son la misma variable.
En segundo lugar, ¿qué estás usando para cargar esto? ¿phpMyAdmin, MySQL Query Browser, Navicat, qué?

[saumal20]

revisandolo nuevamente, me dije en el error de las comsas, pero lamentablmente sigue igual :(

Código MySQL:

create procedure PA_8_actualizar_compras(numerocomprobantes int(12), motivoanulacioncompras text)
    BEGIN
        --declare @fehoanulacioncompra timestamp;
        select CURRENT_timestamp() into @fehoanulacioncompra,
        UPDATE compras
        set anulacioncompras = 1 AND fehoanulacioncompra = @fehaanulacioncompra AND motivoanulacioncompras = @motivoanulacioncompras;
        WHERE numerocomprobantes= @numerocomprobantes;
    END;
 

[saumal20]

Iniciado por gnzsoloyo

Quítale los"@" a las variables.
Estás comparando y asignando valores NULL en esos campos, porque @a y a no son la misma variable.
En segundo lugar, ¿qué estás usando para cargar esto? ¿phpMyAdmin, MySQL Query Browser, Navicat, qué?

uso heidisql

[gnzsoloyo]

Código MySQL:

CREATE PROCEDURE PA_8_actualizar_compras(numerocomprobantes INT(12), motivoanulacioncompras TEXT)
BEGIN
    declare fehoanulacioncompra timestamp;
    SELECT CURRENT_TIMESTAMP() INTO fehoanulacioncompra;
    UPDATE CONTROLES
    SET anulacioncompras = 1,
    fehoanulacioncompra = fehaanulacioncompra,
    motivoanulacioncompras = motivoanulacioncompras
    WHERE numerocomprobantes= numerocomprobantes;
END;
 

1) Las asignaciones de un UPDATE se separan con comas y no con AND.
2) La sentencia termina únicamente con punto y coma (;), y sólo luego de la última cláusula. Habías puesto un cierre antes del WHERE.
3) No van las @ en este contexto. Lee el manual de referencia respecto al tema variables de usuario.
4) Habías cerrado una sentencia con coma y no punto yy coma.
5) Los comentarios con doble guion deben ser seguidos de un espacio para que los tome como tal.

[saumal20]

naa, lo he puesto como l pones tu, pero naa. con respecto a los del , en el update, no ha hexo problemas eso en los otros PA

[saumal20]

Iniciado por saumal20

naa, lo he puesto como l pones tu, pero naa. con respecto a los del , en el update, no ha hexo problemas eso en los otros PA

plops....

lo probe en navicat y si funciono, nose xq heidisql lo interpreta como si estuviera mal...en fin...la consulta seria asi:

Código MySQL:

create procedure PA_8_actualizar_compras(numerocomprobantes int(12), motivoanulacioncompras text)
    BEGIN
        DECLARE fehoanulacioncompra timestamp;
        select CURRENT_timestamp() into @fehoanulacioncompras;
        UPDATE compras
        SET anulacioncompras = 1 AND fehoanulacioncompra = @fehaanulacioncompra AND motivoanulacioncompras = @motivoanulacioncompras
        WHERE numerocomprobantes= @numerocomprobantes;
    END;
DELIMITER ;
 

alguna idea?

[saumal20]

re plops...ia salio...ke habra pasado? :S , para mi, seguro que el heidi sql es inestable aun...ojala lo arreglen, es bueno y liviano. saludos y gracias a todos los ke respondieron

[gnzsoloyo]

Me alegra que saliese.

para ora ocasión por favor, acuérdate que en un foro se debe escribir en español (castellano) y no en SMS...
Cuesta un poco entenderte.

[saumal20]

Iniciado por gnzsoloyo

Me alegra que saliese.

para ora ocasión por favor, acuérdate que en un foro se debe escribir en español (castellano) y no en SMS...
Cuesta un poco entenderte.

jajaja...ok. lo tendre presente