Resultado null de una select en un procedimiento almacenado

[gnzsoloyo]

Bien, finalmente encontré los problemas que generaban el error:
1) La variable "done" estaba creada, pero no inicializada, por lo que su valor real era NULL y en ese caso todo el proceso del IF falla. No se puede hacer un IF contra un NULL, sino contra una función para NULLs.
2) Resuelto eso, la cantidad de valores a ingresar en esa tabla no se corresponde con la cantidad de columnas de la tabla. En ese caso lo que corresponde hacer es o bien poner ,todos los valores necesarios, o indicar los valores que se usarán.
Quedó finalmente así:

Código MySQL:

DELIMITER $$
 
DROP PROCEDURE IF EXISTS `test`.`CUR` $$
CREATE DEFINER=`root`@`%` PROCEDURE `CUR`()
BEGIN
  DECLARE done INT DEFAULT 0;
  DECLARE VANYTRA int(4);
  DECLARE VMESTRA int(2);
  DECLARE VNOMMES varchar(15);
  DECLARE VNCUENTA varchar(30);
  DECLARE VNOMCLIENTE varchar(255);
  DECLARE VCODREPRE varchar(20);
  DECLARE VNOMREPRE varchar(50);
  DECLARE VMES_ACTUAL_IMP float(15,2);
  DECLARE VMES_ACTUAL_CANT float(15,2);
  DECLARE VIMPBRU float(11,2);
  DECLARE VDTOFAM float(11,2);
  DECLARE VDTOART float(11,2);
  DECLARE VDTOCAB float(11,2);
  DECLARE VDTOOTR float(11,2);
  DECLARE VPRECIO float(11,2);
  DECLARE VPRECOS float(11,2);
  DECLARE VAR_MES_ANT_IMP float(15,2);
  DECLARE VAR_MES_ANT_IMP_PCT float(10,2);
  DECLARE VAR_MES_ANT_CANT float(15,2);
  DECLARE VAR_MES_ANT_CANT_PCT float(10,2);
  DECLARE VAR_FIN_ANY_IMP float(15,2);
  DECLARE VAR_FIN_ANY_IMP_PCT float(10,2);
  DECLARE VAR_FIN_ANY_CANT float(15,2);
  DECLARE VAR_FIN_ANY_CANT_PCT float(10,2);
  DECLARE VAR_INTERANUAL_IMP float(15,2);
  DECLARE VAR_INTERANUAL_IMP_PCT float(10,2);
  DECLARE VAR_INTERANUAL_CANT float(15,2);
  DECLARE VAR_INTERANUAL_CANT_PCT float(10,2);
  DECLARE VAR_PRESUPUESTO_IMP float(15,2);
  DECLARE VAR_PRESUPUESTO_IMP_PCT float(10,2);
  DECLARE VAR_PRESUPUESTO_CANT float(15,2);
  DECLARE VAR_PRESUPUESTO_CANT_PCT float(10,2);
  /* ------------------------------------------------------------------------------- */
  DECLARE CUR_VENTAS_CLIENTE_REPRE CURSOR FOR
          SELECT ANYTRA,MESTRA,InformarNombreMes(MESTRA) AS NOMMES,NCUENTA,NOMCLIENTE,REPRE AS CODREPRE,NOMREPRE,SUM(IMPNET) AS MES_ACTUAL_IMP,COUNT(*) AS MES_ACTUAL_CANT,
                 SUM(IMPBRU) AS IMPBRU,SUM(DTOFAM) AS DTOFAM,SUM(DTOART) AS DTOART,SUM(DTOCAB) AS DTOCAB,SUM(DTOOTR) AS DTOOTR,SUM(PRECIO)/COUNT(*) AS PRECIO,SUM(PRECOS) AS PRECOS
          FROM DADESVENDES
          WHERE NCUENTA IS NOT NULL
          GROUP BY ANYTRA,MESTRA,NCUENTA,NOMCLIENTE,CODREPRE,NOMREPRE;
  /* ------------------------------------------------------------------------------- */
  DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
  /* ------------------------------------------------------------------------------- */
  OPEN CUR_VENTAS_CLIENTE_REPRE;
  /* ------------------------------------------------------------------------------- */
  REPEAT
    FETCH CUR_VENTAS_CLIENTE_REPRE INTO
            VANYTRA,
            VMESTRA,
            VNOMMES,
            VNCUENTA,
            VNOMCLIENTE,
            VCODREPRE,
            VNOMREPRE,
            VMES_ACTUAL_IMP,
            VMES_ACTUAL_CANT,
            VIMPBRU,
            VDTOFAM,
            VDTOART,
            VDTOCAB,
            VDTOOTR,
            VPRECIO,
            VPRECOS;
    IF NOT done THEN
      /* ------------------------------------------------------------------------------- */
      /* Inserción de los registros de referencia agrupados por CLIENTE y REPRE */
      /* ------------------------------------------------------------------------------- */
      INSERT INTO VENTAS_CLIENTE_REPRE_BACKUP(
        ANYTRA,
        MESTRA,
        NOMMES,
        NCUENTA,
        NOMCLIENTE,
        CODREPRE,
        NOMREPRE,
        MES_ACTUAL_IMP,
        MES_ACTUAL_CANT,
        IMPBRU,
        DTOFAM,
        DTOART,
        DTOCAB,
        DTOOTR,
        PRECIO,
        PRECOS)
    VALUES
        (VANYTRA,
        VMESTRA,
        VNOMMES,
        VNCUENTA,
        VNOMCLIENTE,
        VCODREPRE,
        VNOMREPRE,
        VMES_ACTUAL_IMP,
        VMES_ACTUAL_CANT,
        VIMPBRU,
        VDTOFAM,
        VDTOART,
        VDTOCAB,
        VDTOOTR,
        VPRECIO,
        VPRECOS);
    END IF;
  UNTIL done END REPEAT;
  /* ------------------------------------------------------------------------------- */
  CLOSE CUR_VENTAS_CLIENTE_REPRE;
  /* ------------------------------------------------------------------------------- */
  SELECT MAX(ANYTRA), MAX(MESTRA), NOMMES FROM VENTAS_CLIENTE_REPRE_BACKUP LIMIT 1
  INTO VANYTRA, VMESTRA, VNOMMES;
  /* ------------------------------------------------------------------------------- */
  SELECT VANYTRA, VMESTRA, VNOMMES;
END $$
 
DELIMITER ;
 

La última parte es una adaptación de lo que pusiste para que devuelva algo.

Queda pendiente resolver lo de las PK de las tablas, ya que no se está controlando la unicidad de valores.
Tampoco hay un buen uso de la programación de SP y SF en algunos casos, en los que se están haciendo operaciones innecesariamente complicadas, como por ejemplo en la función `InformarNombreMes`, que yo escribiría así:

Código MySQL:

DELIMITER $$
 
DROP FUNCTION IF EXISTS `InformarNombreMes` $$
CREATE FUNCTION `InformarNombreMes`(COD INT) RETURNS varchar(15) CHARSET latin1
BEGIN
  CASE COD
    WHEN 1 THEN
      RETURN 'ENE';
    WHEN 2 THEN
      RETURN 'FEB';
    WHEN 3 THEN
      RETURN 'MAR';
    WHEN 4 THEN
      RETURN 'ABR';
    WHEN 5 THEN
      RETURN 'MAY';
    WHEN 6 THEN
      RETURN 'JUN';
    WHEN 7 THEN
      RETURN 'JUL';
    WHEN 8 THEN
      RETURN 'AGO';
    WHEN 9 THEN
      RETURN 'SEP';
    WHEN 10 THEN
      RETURN 'OCT';
    WHEN 11 THEN
      RETURN 'NOV';
    WHEN 12 THEN
      RETURN 'DIC';
    ELSE
      RETURN 'NO INFORMADO';
  END CASE;
END $$
 
DELIMITER ;
 

Es un poco más limpio y al parser le resulta más eficiente.