[SOLUCIONADO] Codigo para subir imagen

tiobavie · #1 (**permalink**) 23/11/2015, 10:38

Hola. Estoy intentando hacer un formulario para añadir datos desde la web a la base de datos. Hay un campo que guardará el enlace a una imagen (he leído que es mejor guardar las imágenes en una carpeta del servidor y el enlace en la BD, ya que es más sencillo acceder así a esas imágenes)

El caso es que he adaptado uno de tantos códigos que he ido encontrando y que en principio todo funcionaba... Hasta que me ha dado por mirar esos registros en la BD y en la carpeta, y no están.

¿Podríais indicarme que errores hay en el código que uso?
En principio, me gustaría que estuviese todo (HTML y PHP) en la misma página... Si es posible, aunque también he probado ese código en dos paginas separadas y tampoco guarda ni los datos ni la imagen.

Saludos.

Código HTML:

Ver original<?php include ("Connections/miriamweb.php");?>
<!doctype html>
<html>
<head>
<meta charset="utf-8">
<title>Documento sin título</title>
</head>
<body>
<?php 
if ($_FILES["imagen"]["error"] > 0){
    echo "ha ocurrido un error";
    //ahora vamos a verificar si el tipo de archivo es un tipo de imagen permitido.
    //y que el tamano del archivo no exceda los 100kb
    $permitidos = array("image/jpg", "image/jpeg", "image/gif", "image/png");
    $limite_kb = 200;
    if (in_array($_FILES['imagen']['type'], $permitidos) && $_FILES['imagen']['size'] <= $limite_kb * 1024){
        //esta es la ruta donde copiaremos la imagen
        //recuerden que deben crear un directorio con este mismo nombre
        //en el mismo lugar donde se encuentra el archivo subir.php
        $ruta = "album/" . $_FILES['imagen']['name'];
        //comprovamos si este archivo existe para no volverlo a copiar.
        //pero si quieren pueden obviar esto si no es necesario.
        //o pueden darle otro nombre para que no sobreescriba el actual.
        if (!file_exists($ruta)){
            //aqui movemos el archivo desde la ruta temporal a nuestra ruta
            //usamos la variable $resultado para almacenar el resultado del proceso de mover el archivo
            //almacenara true o false
            $resultado = move_uploaded_file($_FILES["imagen"]["tmp_name"], $ruta);
            if ($resultado){///hasta aqui es para guardar la imagen en carpeta album/... Una vez hecho esto, se añaden los campos a la base de datos.
                $nombre = $_FILES['imagen']['name'];
                $GModo = $_POST['GModo'];
                $GSERIE = $_POST['GSerie']; 
                $GTitulo = $_POST['GTitulo'];
                $GTecnica = $_POST['GTecnica'];
                $GFecha = $_POST['GFecha'];
                $GMedidas = $_POST['GMedidas'];
                $GDisponibilidad =  $_POST['GDisponibilidad'];      
                $Ganadir = "INSERT INTO `miriamweb`.`obras` (`ID`, `Medio`, `Serie`, `Titulo`, `Tecnica`, `Fecha`, `Medidas`, `Disponibilidad`, `Imagen`) VALUES (NULL, '$GModo', '$GSERIE', '$GTitulo', '$GTecnica', '$GFecha', '$GMedidas', '$GDisponibilidad1', '$nombre');";
                
                echo "el archivo ha sido movido exitosamente";
            } else {
                echo "ocurrio un error al mover el archivo.";
            }
        } else {
            echo $_FILES['imagen']['name'] . ", este archivo existe";
        }
    } else {
        echo "archivo no permitido, es tipo de archivo prohibido o excede el tamano de $limite_kb Kilobytes";
    }
}
?>
 
<form action="" method="POST" enctype="multipart/form-data">
    <label for="imagen">Medio:</label>
    <input type="text" name="GModo"  required/>
    <label for="imagen">Serie:</label>
    <input type="text" name="GSerie"  required/>
    <label for="imagen">Titulo:</label>
    <input type="text" name="GTitulo"  required/>
    <label for="imagen">Tecnica:</label>
    <input type="text" name="GTecnica"  required/>
    <label for="imagen">Fecha:</label>
    <input type="text" name="GFecha"  required/>    
    <label for="imagen">Tamaño:</label>
    <input type="text" name="GMedidas"  required/>
    <label for="imagen">Disponibilidad:</label>
    <input type="text" name="GDisponibilidad"  required/>
    <label for="imagen">Imagen:</label>
    <input type="file" name="imagen" id="imagen" required/>
    
    <input type="submit" name="subir" value="Añadir"/>
</form>
</body>
</html>

chronos682 · #2 (**permalink**) 23/11/2015, 13:28

Tienes que ir depurando línea por línea tu código hasta encontrar en dónde falla, porque así por encima no veo en dónde pueda estar el error, aparentemente se ve bien el código.

Edito: ya vi donde falla tu código. Todo el código para subir la imagen al directorio y a la bd lo tienes en este if

if ($_FILES["imagen"]["error"] > 0){

Por lo cual sólo se va a ejecutar si hay un error. Arregla eso cerrando el if luego de moestrar el mensaje de error y colocando un else con el resto del código.

tiobavie · #3 (**permalink**) 23/11/2015, 16:24

Ok. Gracias. Con ese cambio ya se guarda la imagen en la carpeta al menos.
Tambien habia que incluir un par de filas para el intro en la BD, que revisando poco a poco, no ha sido más problema que detectarlo.

el código final queda así:

Código HTML:

Ver original<?php include ("Connections/miriamweb.php");?>
<?php
 
if ($_FILES["imagen"]["error"] > 0){
    echo "ha ocurrido un error";
} else {
    //ahora vamos a verificar si el tipo de archivo es un tipo de imagen permitido.
    //y que el tamano del archivo no exceda los 100kb
    $permitidos = array("image/jpg", "image/jpeg", "image/gif", "image/png");
    $limite_kb = 2000;
    if (in_array($_FILES['imagen']['type'], $permitidos) && $_FILES['imagen']['size'] <= $limite_kb * 1024){
        //esta es la ruta donde copiaremos la imagen
        //recuerden que deben crear un directorio con este mismo nombre
        //en el mismo lugar donde se encuentra el archivo subir.php
        $ruta = "album/" . $_FILES['imagen']['name'];
        //comprovamos si este archivo existe para no volverlo a copiar.
        //pero si quieren pueden obviar esto si no es necesario.
        //o pueden darle otro nombre para que no sobreescriba el actual.
        if (!file_exists($ruta)){
            //aqui movemos el archivo desde la ruta temporal a nuestra ruta
            //usamos la variable $resultado para almacenar el resultado del proceso de mover el archivo
            //almacenara true o false
            $resultado = @move_uploaded_file($_FILES["imagen"]["tmp_name"], $ruta);
            if ($resultado){///hasta aqui es para guardar la imagen en carpeta album/... Una vez hecho esto, se añaden los campos a la base de datos.
                $nombre = $_FILES['imagen']['name'];
                $GModo = $_POST['GModo'];
                $GSERIE = $_POST['GSerie']; 
                $GTitulo = $_POST['GTitulo'];
                $GTecnica = $_POST['GTecnica'];
                $GFecha = $_POST['GFecha'];
                $GMedidas = $_POST['GMedidas'];
                $GDisponibilidad =  $_POST['GDisponibilidad'];      
                $Ganadir = "INSERT INTO obras (Medio, Serie, Titulo, Tecnica, Fecha, Medidas, Disponibilidad, Imagen) VALUES ('$GModo', '$GSERIE', '$GTitulo', '$GTecnica', '$GFecha', '$GMedidas', '$GDisponibilidad', '$nombre');";
                mysql_select_db($database_miriamweb, $miriamweb);
  $Result1 = mysql_query($Ganadir, $miriamweb) or die(mysql_error());
                
                echo "el archivo ha sido movido exitosamente";
            } else {
                echo "ocurrio un error al mover el archivo.";
            }
        } else {
            echo $_FILES['imagen']['name'] . ", este archivo existe";
        }
    } else {
        echo "archivo no permitido, es tipo de archivo prohibido o excede el tamano de $limite_kb Kilobytes";
    }
}
?>
 
<form action="" method="POST" enctype="multipart/form-data">
    Medio:<input type="text" name="GModo" /><br />
    Serie:<input type="text" name="GSerie" /><br />
    Titulo:<input type="text" name="GTitulo" /><br />
    Tecnica:<input type="text" name="GTecnica" /><br />
    Fecha:<input type="text" name="GFecha" /><br />    
    Tamaño:<input type="text" name="GMedidas" /><br />
    Disponibilidad:<input type="text" name="GDisponibilidad" /><br />
    Imagen:<input type="file" name="imagen" id="imagen" /><br />
    
    <input type="submit" name="subir" value="Añadir"/>
</form>

El único "problema" es el aviso de variables no definidas al lanzar la página

(Notice: Undefined index: imagen in C:\xampp\htdocs\miriam3\anadir.php on line 4

Notice: Undefined index: imagen in C:\xampp\htdocs\miriam3\anadir.php on line 11
archivo no permitido, es tipo de archivo prohibido o excede el tamano de 2000 Kilobytes) pero se soluciona colocando todo el php en otro archivo.

Muchas gracias.

chronos682 · #4 (**permalink**) 23/11/2015, 16:34

Has var_dump de la variable $_FILES para ver que índices trae porque imagen no está definido.

tiobavie · #5 (**permalink**) 24/11/2015, 02:01

El resultado del var_dump era "null". Lo que es lógico, ya que aun no habias sido definida la variable imagen desde el formulario. Leyendo otros temas, concretamente en [URL="http://www.forosdelweb.com/f18/llamar-funcion-desde-form-986752/"]http://www.forosdelweb.com/f18/llamar-funcion-desde-form-986752/,[/URL] he comprobado que añadiendo if(isset) al inicio, no se ejecuta el php sin haber enviado el formulario. Con esto quiero decir que ya no me salen esos avisos.

Gracias de nuevo, y ahora con más motivo, ya que si no llegas a continuar con el tema, me habria resignado a colocar el php en otro archivo.

Y de paso, os agradezco a todos los miembros del foro todos y cada uno de los mensajes que escribís, es increíble la cantidad de conocimiento que hay en esta web. GRACIAS.