2

Utilizando una tabla de registros en mysql enlazada con una llave foranea a mi tabla imágenes deseo insertar a esta misma, varias imágenes a la vez en ese mismo registro. Esto lo intento hacer con un botón que envía el id del registro por metodo POST y se recibe para procesar una consulta. Las imágenes se cargan a la Carpeta del Servidor pero No a la BDD (ruta), que estoy haciendo mal?

 <?php
 function con(){
return new mysqli("localhost","root","","mydb");
}
function insert_img($folder, $image){
$con = con();
$con->query("insert into imagenes (id_registro, folder, src, 
created_at) value (\"$id_registro\",\"$folder\",\"$image\",NOW())");
}
<?

upload.php

<?php
include "db.php";
include "class.upload.php";

/// mostrar errores
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);
error_reporting(E_ALL);
$error = false;
$files = array();
foreach ($_FILES['imagenes'] as $k => $l) {
foreach ($l as $i => $v) {
if (!array_key_exists($i, $files))
$files[$i] = array();
$files[$i][$k] = $v;
}
}
foreach ($files as $file) {
$handle = new Upload($file);
if ($handle->uploaded) {
$handle->Process("imagenes/");
if ($handle->processed) {
    // usamos la funcion insert_img de la libreria db.php
    insert_img("imagenes/",$handle->file_dst_name);
 } else {
  $error = true;
  echo 'Error: ' . $handle->error;
 }
 } else {
    $error = true;
echo 'Error: ' . $handle->error;
}
unset($handle);
}   
if(!$error){
print "<h4>Exito!</h4>";
}
?>
8
  • 1
    Dentro de la función insert_img() la variable $id_registro no está definida. Esto seguramente genere un warning en PHP y un error en la consulta SQL si es campo no es nulable.
    – Xerif
    Commented el 5 nov. 2019 a las 10:45
  • Bien visto @Xerif ! Por otra parte, no hay necesidad de escribir la consulta así: "insert into imagenes (id_registro, folder, src, created_at) value (\"$id_registro\",\"$folder\",\"$image\",NOW())" se puede simplifcar: "insert into imagenes (id_registro, folder, src, created_at) value ($id_registro,'$folder','$image',NOW())" He asumido que la columna id_registro es numérica, por eso no la puede rodeada de comillas simples.
    – A. Cedano
    Commented el 5 nov. 2019 a las 11:17
  • 2
    @A.Cedano Creo que tanto tú como yo iríamos un poco más allá y utilizaríamos sentencias preparadas que sería lo más adecuado. Pero si, es más legible utilizando comillas simples que escapar las dobles.
    – Xerif
    Commented el 5 nov. 2019 a las 11:25
  • Claro @Xerif, así debería ser y no por gusto, sino por seguridad e incluso por rendimiento en el código.
    – A. Cedano
    Commented el 5 nov. 2019 a las 14:12
  • Agradezco mucho sus observaciones @Xerif y A. Cedano , soy nueva en PHP y espero poder tener la lógica que ustedes tiene, de momento he realizado los cambios que me hicieron, pero siguen sin guardarse las rutas de las imágenes en la tabla de imágenes y me manda el siguiente error: "Undefined index: imagenes in www" de la siguiente linea -> foreach ($_FILES['imagenes'] as $k => $l) { Commented el 6 nov. 2019 a las 16:40

0

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

Examina otras preguntas con la etiqueta o formula tu propia pregunta.