Trigger mysql

[herran1212]

Hola a todos , mi problema es el siguiente tengo el siguiente trigger:

Código MySQL:

BEGIN
 
SET @auto=(SELECT automatico FROM items_configuracion_codigo WHERE id=NEW.id_configuracion);
 
IF @auto = 'true' THEN
      
    SET @cuantos = (SELECT codigo FROM items_familia_grupo_subgrupo WHERE id_familia=NEW.id_familia AND id_grupo=NEW.id_grupo AND id_empresa=NEW.id_empresa AND activo=1  ORDER BY CAST(codigo AS SIGNED) DESC LIMIT 0,1);
    
    IF @cuantos >0 THEN
          SET NEW.codigo=RIGHT(CONCAT('0000000000',@cuantos+1),LENGTH(@cuantos));
    ELSE
          SET NEW.codigo = CONCAT(
                                                                    # LPAD('numero','cantidad de digitos','numero que se concatena para llenar el valor de digitos')
                                                                   LPAD(1,2,'0')
                                                           );
     END IF;
  
END IF;
END
 

lo utilizo para verificar si el codigo a insertar es automatico y si es que lo genere de forma automatica, el problema es que llegado al if de @cuantos no me toma el valor mayor y lo suma sino que es como si no hubiera ningun codigo es decir se salta al else ais haya otro codigo, ya relice el query y funciona, nose que pueda ser, si pudieran ayudarme os lo agradeceria

[gnzsoloyo]

Personalmente no le encuentro mucho el sentido de poner un VARCHAR para el código, si cada vez que necesitas crear un registro nuevo, debes convertirlo a un numero entero.
¿Por qué haces eso? ¿No sería más simple usar directamente un INT?

Fuera de eso, hay una condición que te estás olvidando: Las variables de usuario deben inicializarse siempre que se van a usar, o de lo contrario su valor es NULL. Y en MySQL toda operación de asignación, cálculo o concatenación entre cualquier valor y NULL da... NULL.

La cosa andaría mas o menos por acá:

Código MySQL:

delimiter $$
 
CREATE TRIGGER testref BEFORE INSERT ON test1
  FOR EACH ROW 
BEGIN
    
    # Las variables de usuario siempre se deben incializar.
    
    SET @auto:= FALSE;
    SET @cuantos := 0;
    
    # Es más eficiente poner un INTO.
    
    SELECT automatico 
    FROM items_configuracion_codigo 
    WHERE id=NEW.id_configuracion 
    INTO @auto;
    
    # El NULL es una condicion que se debe considerar y administrar.
    
    SET @auto := IF(@auto IS NULL, FALSE, TRUE);
    
    IF @auto = true THEN
        SELECT codigo 
        FROM items_familia_grupo_subgrupo 
        WHERE id_familia = NEW.id_familia 
            AND id_grupo = NEW.id_grupo 
            AND id_empresa = NEW.id_empresa 
            AND activo=1  
        ORDER BY CAST(codigo AS SIGNED) DESC 
        LIMIT 1
        INTO @cuantos;
 
        # El NULL es una condicion que se debe considerar y administrar.
 
        SET @cuantos := IF(@cuantos IS NULL, 0, @cuantos);
        
        IF @cuantos >0 THEN
            SET NEW.codigo=RIGHT(CONCAT('0000000000',@cuantos+1),LENGTH(@cuantos));
        ELSE
            SET NEW.codigo = CONCAT(LPAD(1,2,'0'));
        END IF;
    END IF;
END$$
 
delimiter ;