[SOLUCIONADO] Insertar multiples registros con checkbox

[dankko]

Hola, estoy desarollando una aplicación para una tienda online.

En la zona de admin, tengo una sección para subir productos...
Esos productos están relacionados con multiples tablas.

Tengo unos checkbox, que contienen varias opciones, las cuales pueden ir varias tildadas o sólo una, pero siempre tiene que llevar al menos una.

Mi problema es que no se como realizar ese insert. Lo estoy intentando de esta manera con un foreach, pero lo que hace es hacer como máximo 2 insert, y en la columna "compatible", en vez de añadir un valor distinto cada vez que añade un producto compatible, lo que hace es darle el mismo valor, es decir, si marco la casilla modelo 1, modelo 2 y modelo 3, Lo unico que me ingresa es:

Máximo de 2 insert
En la casilla compatible, a esos dos insert lo que hace es ponerle el mismo compatible (debería ser 1 y 2), es decir, sale 1 en los dos.

Código PHP:

<?php
//conexión con la base de datos
$conexion = conecta_base_datos();
if($_POST) { //echo "he recibido los datos y procederé a insertarlos";
    //primero se deben hacer algunas validaciones
if ($_POST["nombre_producto"]==""){
    echo "No he recibido el nombre del producto...";
}elseif($_POST["id_subgeneral"]==""){
    echo "No he recibido la categoría del equipo...";
}else{
    echo "He recibido datos correctos y procedo a insertarlos...";
 
$elec_array = $_POST['compatible'];
if ($elec_array=='')
{
    $query = "INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compatible) VALUES('" . $_POST["nombre_producto"] . "','" . $_POST["id_subgeneral"] . "','" . $_POST["id_fabricante"] . "','')";
    $result = mysql_query($query);
}
else
{
    foreach ($elec_array as $one_elec) 
    {
    $source .= $one_elec.", ";
    }
    $elec = substr($source, 0, -2);
    echo $elec; 
    $query = "INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compatible) VALUES('" . $_POST["nombre_producto"] . "','" . $_POST["id_subgeneral"] . "','" . $_POST["id_fabricante"] . "','" . $elec . "')";
    $result = mysql_query($query);
}
 
    //$ssql ="INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compatible)  values ('" . $_POST["nombre_producto"] . "','" . $_POST["id_subgeneral"] . "','" . $_POST["id_fabricante"] . "','" . $compatible . "')";
        if (mysql_query($query)){
            echo "<h4 class='alert_success'>¡Producto insertado correctamente!</h4>";
            //print "<meta http-equiv=Refresh content=\"2 ; url=./\">";
        }else{
            echo"<h4 class='alert_error'>Hubo un error al insertar los datos</h4>";
            echo mysql_error();
        }
    }
        }else{
    ?>
    
    <form action="<?php echo $_SERVER['PHP_SELF']?>" method="post"> 
        <fieldset>
            <label>Nombre producto: </label>
            <input type="text" name="nombre_producto">
        </fieldset>
        <fieldset>
            <label>Categoría:</label>
                <select name="id_subgeneral">
                    <?php
                    $ssql = "select * from cat3_subgeneral";
                    $rs = mysql_query ($ssql);
                    while ($fila = mysql_fetch_array($rs)){
    
                        echo "<option value=" . $fila["id_subgeneral"] . ">" .$fila["nombre_subgeneral"] . "</option>";
                    }
                    mysql_free_result ($rs);
                    ?>
                </select>
        </fieldset>
        <fieldset>
            <label>Fabricante:</label>
                <select name="id_fabricante">
                    <?php
                    $ssql = "select * from fabricante";
                    $rs = mysql_query ($ssql);
                    while ($fila = mysql_fetch_array($rs)){
    
                        echo "<option value=" . $fila["id_fabricante"] . ">" .$fila["nombre_fabricante"] . "</option>";
                    }
                    mysql_free_result ($rs);
                    ?>
                </select>
        </fieldset>
      
        <fieldset>
            <label>Coches compatibles:</label>
                <?php
                $conexion = conecta_base_datos();
                $ssql = "SELECT *
                FROM modelo
                INNER JOIN marcas ON marcas.id_marca = modelo.id_marca";
                $rs = mysql_query($ssql);
                while ($fila = mysql_fetch_array($rs)){
        
                    echo '<br><input type="checkbox" name="compatible[]" value="' . $fila["id_modelo"] . '">' . utf8_encode($fila["nombre_modelo"]) . ' - ' . utf8_encode($fila["nombre_marca"]) . '<br>';
        
                }
                ?>
        </fieldset>
        <fieldset>
            <input type="submit" value="Enviar">
        </fieldset>
    </form>
 
 <?php
}
 //Cerrar conexión con la base de datos, porque si no, al entrar demasiada gente, MYSQL se cae
 mysql_close($conexion);
 ?>
 

Este es el código.

También arroja otro error:

Notice: Undefined variable: source in C:\wamp\www\proyectos\...\admin\enviar-producto.php on line 26

Donde la variable indefinida es la variable que defino para usar el foreach:

$one_elec

La estructura de las tablas que aparecen:

modelo{id_modelo, nombre_modelo,id_marca}
marcas{id_marca,nombre_marca}
cat3_subcategoria{id_subcategoria,nombre_categoria }
productos{id_producto,nombre_producto,id_subcatego ria,compatible}

Donde la columna compatible, sería el valor de id_modelo.

Alguien sabe como podría solucionar esto?

[mortiprogramador]

<saludos>
Pues creo que te estás enredando al momento de insertar el tema del campo compatible, pues en principio los modelos están definidos en el formulario como un campo con nombre de array, y por ende php al recibirlo lo trata como tal, es decir,
que internamente dentro del insert actual que se tiene, se puede hacer otro foreach para recorrer los elementos de compatible (como un array cualquiera, solo que en la definición del foreach se usaría el elemento compatible de $_POST), y por cada elemento de estos si hacer el insert que ya se tiene (igual como entiendo el formulario va por producto, no es un masivo de productos a un mismo tiempo)

Igual si con este lazo que te enviè sigues enredado, te invito a que subas los scripts de tablas para recrear por completo ambiente de bd y aplicación para hacer los cambios pertinentes
</saludos>

[dankko]

Si, sigo enredado la verdad.

Dejo las tablas

Código MySQL:

CREATE TABLE IF NOT EXISTS `cat3_subgeneral` (
  `id_subgeneral` int(7) NOT NULL AUTO_INCREMENT,
  `nombre_subgeneral` varchar(100) NOT NULL,
  `id_pregeneral` int(7) NOT NULL,
  `id_general` int(7) NOT NULL,
  PRIMARY KEY (`id_subgeneral`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=62 ;
 

Código MySQL:

CREATE TABLE IF NOT EXISTS `fabricante` (
  `id_fabricante` int(7) NOT NULL AUTO_INCREMENT,
  `nombre_fabricante` varchar(250) NOT NULL,
  PRIMARY KEY (`id_fabricante`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=77 ;
 

Código MySQL:

CREATE TABLE IF NOT EXISTS `marcas` (
  `id_marca` int(7) NOT NULL AUTO_INCREMENT,
  `nombre_marca` varchar(250) NOT NULL,
  `ruta` varchar(250) NOT NULL,
  PRIMARY KEY (`id_marca`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=99 ;
 

Código MySQL:

CREATE TABLE IF NOT EXISTS `modelo` (
  `id_modelo` int(7) NOT NULL AUTO_INCREMENT,
  `nombre_modelo` varchar(250) NOT NULL,
  `id_marca` int(11) NOT NULL,
  `ruta` varchar(250) NOT NULL,
  PRIMARY KEY (`id_modelo`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=3082 ;
 

Código MySQL:

CREATE TABLE IF NOT EXISTS `productos` (
  `id_producto` int(7) NOT NULL AUTO_INCREMENT,
  `nombre_producto` varchar(250) NOT NULL,
  `referencia` varchar(250) NOT NULL,
  `id_fabricante` int(7) NOT NULL,
  `id_subgeneral` int(7) NOT NULL,
  `compatible` int(20) DEFAULT NULL,
  PRIMARY KEY (`id_producto`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=322 ;
 

La relación con la tabla productos la hago con la tabla subcategoria, y modelo.
Donde compatible en la tabla productos, es el ID del modelo, con los que el producto es compatible.

Saludos.

[mortiprogramador]

<saludos>
Bueno, pues revisando el tema tengo lo siguiente:

En efecto, cuando por ej se tienen 4 marcas y se chequean todas,
al guardar solo realiza 2 inserciones, y en ambas guarda la misma marca compatible (es decir, el primer check)

Esto sucede por lo siguiente:
- Cuando se hace el ELSE se inserta un primer registro de este modo
INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compa tible) VALUES('producto1','64','77','3082, 3083, 3084, 3085')

Imagino yo que el último campo (compatible) que esta definido como int, al recibir semejante valor ('3082, 3083, 3084, 3085' que son los ids de los compatibles)
trunca el valor y solo deja el primero antes de la coma, en este caso el 3082

Y con esto ya tenemos un registro insertado en la tabla
Ahora bien, luego del ELSE se tiene un IF donde pregunta por un mysql_query y de nuevo usa el mismo query anterior, por esto es que siempre
registra 2 veces la misma data (que a todas estas es algo que no debería permitirse incluso por base de datos, es decir, que debería ser un registro único
la combinación de producto con marca compatible, pero que actualmente si lo permite)

La solución? Bueno, pues opino que una solución podría ser fusionar el código, es decir, que en el foreach se haga lo que tienes en el else y el siguiente if,
lo resumo así:

Código PHP:

<?php
 
$elec_array = $_POST['compatible'];
if ($elec_array=='')
{
    $query = "INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compatible) VALUES('" . $_POST["nombre_producto"] . "','" . $_POST["id_subgeneral"] . "','" . $_POST["id_fabricante"] . "','')";
    $result = mysql_query($query);
}
else
{
    foreach ($elec_array as $one_elec) 
    {
        $source .= $one_elec.", ";
        $query = "INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compatible) VALUES('" . $_POST["nombre_producto"] . "','" . $_POST["id_subgeneral"] . "','" . $_POST["id_fabricante"] . "','" . $one_elec . "')";
        if (mysql_query($query)){
            echo "<h4 class='alert_success'>¡Producto insertado correctamente!</h4>";
            //print "<meta http-equiv=Refresh content=\"2 ; url=./\">";
        }else{
            echo"<h4 class='alert_error'>Hubo un error al insertar los datos</h4>";
            echo mysql_error();
        }
    }
    
    $elec = substr($source, 0, -2);
    echo $elec; 
}
 
    //$ssql ="INSERT INTO productos (nombre_producto,id_subgeneral,id_fabricante,compatible)  values ('" . $_POST["nombre_producto"] . "','" . $_POST["id_subgeneral"] . "','" . $_POST["id_fabricante"] . "','" . $compatible . "')";
    }
        }else{
    ?>
    
    <form action="<?php echo $_SERVER['PHP_SELF']?>" method="post">
 

Ahora bien, finalmente me permito darte algunas sugerencias para tu modelo de bd y aplicación

1. La referencia entre las tablas marcas y modelo no cuenta con una llave foránea,
es decir, que si bien el query que hace el join funciona, a nivel de base de datos
no se tiene una normatividad consistente que en efecto se inserte un modelo
de una marca que si exista, y por esto puede que tengan distintos ids y en algún
momento algunos datos no se visualicen correctamente
Mi sugerencia es aplicarle un foreign key a la tabla modelo que apunte al campo id_marca de la tabla marcas

2. Creo que se podría mejorar el modelo aplicandole reglas de normalización a la tabla productos,
pues como actualmente funciona por cada modelo compatible inserta en la tabla, duplicando datos.
Sugiero que se haga lo siguiente:

2.1. Que la tabla de productos solo guarde el producto como tal una única vez
2.2. Crear una tabla productos_modelo donde se guarden id de producto y id de marca compatible
(Este último significaría un cambio en el foreach que acabamos de modificar, pero pues a nivel de data
sería lo mejor)

Sin embargo, son sugerencias que ya puedes decidir si las necesitas o no, por ahora prueba lo que
te publiqué y nos cuentas como te fue
</saludos>