Dónde sube el archivo?

ommmm · #1 (**permalink**) 29/10/2015, 04:11

Saludos,

Me encuntro con la siguiente situación, he buscado la forma de subir archivos por aquí el foro y encontré unos scripts que me funcionaron bien.
Ahora bien cuando subo el archivo me dice que se ha incertado correctamente, pero no se dónde ha guardado el archivo...

Gracias de antemano!

CODIGOS USADOS:

Formulario

Código PHP:

  <HTML>  
  <BODY bgcolor="#E5E5E5"> 
    <form action="imatge5.php" method="POST" enctype="multipart/form-data"> 
      <input name="TAMANIO" type="hidden" value="600000000"> 
      Introduce el fichero a subir:  
      <input type="file" name="fichero"><br><br> 
      <input name="Enviar" type="submit" value="Subir"> 
             
      
</form> 
 
<td><a href="/scripts/premo/desc5.php"><input type="button" value="Bajar"></input></a></td> 
 
</BODY> 
</HTML>

Acción formulario

Código PHP:

  <?PHP 
     $fichero = $_FILES["fichero"]; 
     if ($_POST["TAMANIO"]>$_FILES["fichero"]["size"]) {  
       echo "<br><font color=\"#FF0000\" align=\"center\">El fichero se ha subido al servidor.</font>"; 
       copy ($_FILES["fichero"]["tmp_name"],$_FILES["fichero"]["name"]); 
     }  
         else { 
       echo "<br><p color=\"#FF0000\">        El fichero es demasiado grande</p>"; 
     } 
      ?>

chronos682 · #2 (**permalink**) 29/10/2015, 08:03

Pues por lo que veo en el código, no le estás asignando ninguna ruta entonces supongo lo está guardando en el mismo directorio donde está el archivo php.

También puede ser que ni siquiera se suba el fichero, porque igual el mensaje de fichero subido correctamente lo estás mostrando incluso antes de copiar el archivo temporal a su ubicación definitiva.

pateketrueke · #3 (**permalink**) 29/10/2015, 08:14

Además lo correcto es usar move_uploaded_file(), pues copy() puede ocasionar problemas.

ommmm, por favor lee el manual, ahí se explica cómo es que debes hacer las cosas: http://php.net/manual/es/features.file-upload.php

ommmm · #4 (**permalink**) 30/10/2015, 01:53

Hola,

Muchas gracias por los consejos, así lo haré, ya os contaré que tal!

ommmm · #5 (**permalink**) 30/10/2015, 03:05

Con el manual que me paso pateketrueke, me funcionó perfectamente.

Ahora quiero ir un poca más allá, me gustaria poder que cuando el usuario llene un formulario con sus datos, pueda subir un archivo.
Y que cuando se consulten las diferentes entradas hechas por diferentes usuarios (es decir diferentes filas), también se pudan descargar/ver los archivo subidos por cada usuario.

Es decir tengo la idea de que haya una columna llamada Archivos y que en esta haya un link en cada fila del listado, que cuando hagas click te muestre el archivo que habia subido el usuario correspondiente.

Gracias de nuevo!!

chronos682 · #6 (**permalink**) 30/10/2015, 05:22

Lo que tienes que hacer es agregar un nuevo campo varchar a la base de datos llamado Archivos y cuando envíes la infomación del formulario guardes en ese campo la ruta completa al archivo, luego al mostrar los registros usas la etiqueta <a href="... Y en href colocas la ruta que guardaste en la bse de datos para que puedas ver o descargar el archivo que se subió.

ommmm · #7 (**permalink**) 30/10/2015, 05:33

Gracias por la rapidez,

Pero no se hacer lo que dices de guardar la ruta completa al archivo cuando envio la información del formulario.
Te muestro mi codigo, haver si así ves más fácil dónde y que deberia poner para hacer lo que me comentas.

Muchas gracias

Este es mi codigo del formulario:

Código PHP:

Ver original<?php
           
           
            require_once ('../conexion.inc.php');
            $db1 = @mysql_connect($servidor,$usuario,$password) or die("Error al conectar al DBMS.");
            @mysql_select_db($basedatos) or die("Error al conectar a la BD");
            $IdRFQ = $_POST['txtrfq'];
            $Data = $_POST['txtdata'];
            $Suppliers = $_POST['txtsupplier'];            
            $Price = $_POST['txtprice'];   
            $Cry2 = $_POST['txtcry2']; 
            $MOQ = $_POST['txtmoq']; 
            $Volume = $_POST['txtvolume'];         
            $Investment = $_POST['txtinvestment']; 
            $Cry = $_POST['txtcry'];        
            $Timing = $_POST['txttiming']; 
             
            $Incoterms = $_POST['txtincoterms']; 
            $Comments = $_POST['txtcomments']; 
            $Customer = $_POST['txtcustomer']; 
             $Categoria = $_POST['txtcategoria']; 
            
            
 
 
function execute( $Data, $Suppliers, $Price, $Cry2, $MOQ, $Volume, $Investment, $Timing, $Cry, $Incoterms, $Customer, $Categoria) 
{ 
$q1= "INSERT INTO rfq (idrfq, data, suppliers, price, cry2, moq, volume, investment, timing, cry, incoterms, rm part nombre) 
values ('$IdRFQ', $Data', '$Suppliers', '$Price', '$Cry2', '$MOQ', '$Volume', '$Investment', '$Timing', '$Cry', '$Incoterms')"; 
$resultquery = mysql_query($q1); 
$errores = mysql_errno(); 
 
 
 
 
            
            
            mysql_close(); 
        ?>
        
      <?php}
// En versiones de PHP anteriores a la 4.1.0, debería utilizarse $HTTP_POST_FILES en lugar
// de $_FILES.
 
$dir_subida = 'C:\Archivos de programa\EasyPHP-DevServer-14.1VC9\data\localweb\subidas\S';
$fichero_subido = $dir_subida . basename($_FILES['fichero_usuario']['name']);
 
echo '<pre>';
if (move_uploaded_file($_FILES['fichero_usuario']['tmp_name'], $fichero_subido)) {
    echo "El fichero es válido y se subió con éxito.\n";
} else {
    echo "¡Posible ataque de subida de ficheros!\n";
}
 
echo 'Más información de depuración:';
print_r($_FILES);
 
print "</pre>";
 
?>

chronos682 · #8 (**permalink**) 30/10/2015, 05:48

La ruta completa del archivo la tienes en la variable $fichero_subido entonces estas son las cosas que tienes que hacer:

1. Determinar si el usuario va a poder subir uno o varios archivos.
1.1. Si son varios archivos debes crear una nueva tabla donde tengas un campo relacionado (puede ser idrfq) con la tabla rfq.
1.2. Si es 1 solo archivo bastaría con agregar un campo al final de la tabla rfq que se llame Archivo y sea de tipo varchar.
2. Guardar el contenido de la variable $fichero_subido en el campo Archivo. Para esto tienes dos opciones:
2.1. Ejecutas primero el código que se encarga de subir el archivo al servidor para que genere la variable $fichero_subido y luego la puedas poner en este insert:

Código PHP:

Ver original$q1= "INSERT INTO rfq (idrfq, data, suppliers, price, cry2, moq, volume, investment, timing, cry, incoterms, rm part nombre) 
values ('$IdRFQ', $Data', '$Suppliers', '$Price', '$Cry2', '$MOQ', '$Volume', '$Investment', '$Timing', '$Cry', '$Incoterms')";

2.2. O puedes dejar el código como está, cuando agregues el campo Archivo a la base de datos le indicas que acepte NULL, insertas los datos a la base de datos como lo estás haciendo hasta ahora y luego de que termines de subir la imagen y si todo salió bien colocas aquí:

Código __default:

Ver originalecho "El fichero es válido y se subió con éxito.\n";

Antes del echo un UPDATE al registro que acabaste de ingresar y actualizas el campo Archivo ingresando el valor de la variable $fichero_subido.

Analiza qué es lo que más te conviene y haz los cambios necesarios y cualquier cosa comentas.

ommmm · #9 (**permalink**) 30/10/2015, 06:15

Muchisimas gracias por la explicación paso a paso y tan detallada!

Pero no acabo de conseguir-lo me sale esto una vez envio el formulario:
'; if (move_uploaded_file($_FILES['fichero_usuario']['tmp_name'], $fichero_subido)) { echo "El fichero es válido y se subió con éxito.\n"; } else { echo "¡Posible ataque de subida de ficheros!\n"; } echo 'Más información de depuración:'; print_r($_FILES); print ""; ?>

Código formulario modificado

Código PHP:

Ver original<?php}
// En versiones de PHP anteriores a la 4.1.0, debería utilizarse $HTTP_POST_FILES en lugar
// de $_FILES.
 
$dir_subida = 'C:\Archivos de programa\EasyPHP-DevServer-14.1VC9\data\localweb\subidas\S';
$fichero_subido = $dir_subida . basename($_FILES['fichero_usuario']['name']);
 
echo '<pre>';
if (move_uploaded_file($_FILES['fichero_usuario']['tmp_name'], $fichero_subido)) {
    echo "El fichero es válido y se subió con éxito.\n";
} else {
    echo "¡Posible ataque de subida de ficheros!\n";
}
 
echo 'Más información de depuración:';
print_r($_FILES);
 
print "</pre>";
 
?>  
 
        <?php}
           
           
            require_once ('../conexion.inc.php');
            $db1 = @mysql_connect($servidor,$usuario,$password) or die("Error al conectar al DBMS.");
            @mysql_select_db($basedatos) or die("Error al conectar a la BD");
            $IdRFQ = $_POST['txtrfq'];
            $Data = $_POST['txtdata'];
            $Suppliers = $_POST['txtsupplier'];            
            $Price = $_POST['txtprice'];   
            $Cry2 = $_POST['txtcry2']; 
            $MOQ = $_POST['txtmoq']; 
            $Volume = $_POST['txtvolume'];         
            $Investment = $_POST['txtinvestment']; 
            $Cry = $_POST['txtcry'];        
            $Timing = $_POST['txttiming']; 
             
            $Incoterms = $_POST['txtincoterms']; 
            $Comments = $_POST['txtcomments']; 
             
           
            
            
 
 
function execute( $Data, $Suppliers, $Price, $Cry2, $MOQ, $Volume, $Investment, $Timing, $Cry, $Incoterms, $fichero_subido) 
{ 
$q1= "INSERT INTO rfq (idrfq, data, suppliers, price, cry2, moq, volume, investment, timing, cry, incoterms, archivo) 
values ('$IdRFQ', $Data', '$Suppliers', '$Price', '$Cry2', '$MOQ', '$Volume', '$Investment', '$Timing', '$Cry', '$Incoterms', $fichero_subido)"; 
$resultquery = mysql_query($q1); 
$errores = mysql_errno(); 
 
 
 
 
            
            
            mysql_close(); 
        ?>

chronos682 · #10 (**permalink**) 30/10/2015, 06:26

Primero que todo una pregunta: para qué le pones esa llave de cierre a la etiqueta php?

Código PHP:

Ver original<?php}

Quita ese } del primer <?php eso es lo que te genera que se vea el código php escrito en pantalla.

Segundo, de todo lo que te escribí qué has hecho aparte de ejecutar primero el código de subir el archivo? Porque no veo que hayas hecho nada más.

ommmm · #11 (**permalink**) 30/10/2015, 06:48

He provado las dos opciones que me comentaste,

Con la primera opcion si saco el } me dice:
Parse error: syntax error, unexpected end of file in C:\Archivos de programa\EasyPHP-DevServer-14.1VC9\data\localweb\scripts\premo\EmpCrear.php on line 84

Y lo que he hecho ha sido poner el campo archivo a la BBDD, lo de subir el código y luego he puesto el $fichero_subido al InsertInto:

Código PHP:

Ver originalfunction execute( $Data, $Suppliers, $Price, $Cry2, $MOQ, $Volume, $Investment, $Timing, $Cry, $Incoterms, $fichero_subido) 
{ 
$q1= "INSERT INTO rfq (idrfq, data, suppliers, price, cry2, moq, volume, investment, timing, cry, incoterms, archivo) 
values ('$IdRFQ', $Data', '$Suppliers', '$Price', '$Cry2', '$MOQ', '$Volume', '$Investment', '$Timing', '$Cry', '$Incoterms', $fichero_subido)"; 
$resultquery = mysql_query($q1); 
$errores = mysql_errno();

Con la segunda opción no tengo muy claro que poner en el update, asi que creo que no lo he hecho muy bien cuando lo he probado.

Muchas gracias!!

ommmm · #12 (**permalink**) 30/10/2015, 06:58

Perdona,
ya me funciona la opcion 1 en principio, ahora no soy capaz de linkear el archivo/descargar-lo, con esto voy muy perdido...

Tengo el siguiente código para hacer la descarga (lo encontré por el foro...):

Código PHP:

Ver original<?php 
$enlace = $Simages."\data\localweb\subidas".$IdRFQ; 
header ("Content-Disposition: attachment; filename=".$IdRFQ."png"); 
header ("Content-Type: application/octet-stream"); 
header ("Content-Length: ".filesize($enlace)); 
readfile($enlace); 
?>

chronos682 · #13 (**permalink**) 30/10/2015, 09:00

Si lo que quieres es mostrar algo como esto en una tabla:

Id - Nombre - Apellido - Archivo
1 - Jose - Perez - miarchivo.pdf

Es solo un ejemplo, si digamos tú quieres mostrar todos los datos ingresados en el registro y al final un link que vaya al archivo, el código que pusiste arriba no te va a servir. Lo que debes hacer es al mismo tiempo que haces el select para traer toda la información de la BD ahí mismo lees el campo archivo, lo cargas en una variable y se la pones a la etiqueta <a> como href. P ej:

Código PHP:

Ver original$q1 = "SELECT * FROM rfq"
...
...
...
$archivo = $row['archivo'];
echo "<a href='{$archivo}'>Descargar</a>";

Esa es la idea. Si puedes coloca el código con el que muestras la información de la base de datos para orientarte.

ommmm · #14 (**permalink**) 02/11/2015, 02:05

Hola chronos682,

Gracias, ahora estoy probando, aunque no tengo muy claro dónde poner-lo.

El código de la consulta es el siguiente:

Código PHP:

Ver original<?php require_once ('../conexion.inc.php');?>
    <?php include '../header.php';?>
 
                <center>
 
                    <table border="1" bordercolor="FFFFF" align="center"  bgcolor="#A9E2F3" >
                        
        <td align="left"><strong>Id</strong></td>                 
        <td align="left"><strong>Data</strong></td>
        <td align="left"><strong>Suppliers</strong></td>
        <td align="left"><strong>Project Part Number</strong></td>
        <td align="left"><strong>RM Part Number</strong></td>
        <td align="left"><strong>Price</strong></td>
        <td align="left"><strong>Cry2</strong></td>
        <td align="left"><strong>MOQ</strong></td>
        <td align="left"><strong>Volume</strong></td>
        <td align="left"><strong>Delete</strong></td>
        <td align="left"><strong>Modify</strong></td>
 
 
 
 
 
</script>
 
         <?php  
 
            
            require_once ('../conexion.inc.php');
            $db1 = @mysql_connect($servidor,$usuario,$password) or die("Error al conectar al DBMS.");
            @mysql_select_db($basedatos) or die("Error al conectar a la BD");
 
            
             
 
            $query = "Select * from rfq ";
            $result =mysql_query($query,$db1);
      
                
  while ($fila = mysql_fetch_assoc($result)) {
                            echo("\n                                <tr> 
 
                                    <td>".$fila['IdRFQ']."</td> 
 
                                    <td>".$fila['Data']."</td> 
                                      
                                    <td>".$fila['Suppliers']."</td>
 
                                    <td>".$fila['Project Part Nombre']."</td>
 
                                    <td>".$fila['RM Part Number']."</td>
 
                                    <td>".$fila['Price']."</td>
 
                                    <td>".$fila['Cry2']."</td>
 
                                    <td>".$fila['MOQ']."</td>
 
                                    <td>".$fila['Volume']."</td>
 
 
 
 
 
                                    
                                    <td><a href='EmpEliminar.php?IdRFQ=".$fila['IdRFQ']."'>Delete</a></td>
 
 
                                    <td><a href='EmpFrmModificar.php?IdRFQ=".$fila['IdRFQ']."'>Modify</a></td>
 
 
                                    <td><a href='Desc.php?IdRFQ=".$fila['IdRFQ']."'>File</a></td>
 
                                    
                                </tr>") ;
                        }
                        
                   echo "</table></center>";
mysql_close($link);
?>

chronos682 · #15 (**permalink**) 02/11/2015, 09:27

Reemplaza esto:

Código PHP:

Ver original<td><a href='Desc.php?IdRFQ=".$fila['IdRFQ']."'>File</a></td>

Por esto:

Código PHP:

Ver original<td><a href='".$fila['archivo']."'>Ver archivo</a></td>

ommmm · #16 (**permalink**) 03/11/2015, 02:18

Saludos y gracias

Esto lo habia probado, pero no me funciona. Y no entiendo el por qué, ya que si pongo el ratón encima del link: Ver archivo. Me dice que se abrirá el archivo que toca en la carpeta que toca. Pero cuando hago click no se abre nada... Puede ser que sea porqué la IdRFQ no tiene ningún valor?

Muchisimas gracias!!!

chronos682 · #17 (**permalink**) 03/11/2015, 09:08

Cual IdRFQ si ahí no se utiliza. Puede ser más bien que no tengas almacenada correctamente la ruta del archivo en el campo archivo de la base se datos. Prueba haciendo var_dump del variable $fila['archivo'] a ver que te muestra.

ommmm · #18 (**permalink**) 04/11/2015, 02:48

Hola de nuevo,

Haciendo var_dump del variable $fila['archivo'] me sale NULL, he mirado la ruta que sigue y me parece que está bien.

Y cuando mantengo el cursor encima del link me sale:
file:///C:/Archivos de programaEasyPHP=DevServer14.1VC9datalocalwebsubida sSimagen.jpg

Muchas gracias!!

rbczgz · #19 (**permalink**) 04/11/2015, 03:09

Hola ommmm,

Cita:

Iniciado por ommmm

Y cuando mantengo el cursor encima del link me sale:
file:///C:/Archivos de programaEasyPHP=DevServer14.1VC9datalocalwebsubida sSimagen.jpg

Si te fijas, te faltan las /, no creo que sea todo una carpeta... y el hecho de que var_dump te salga NULL, es porque no te está trayendo nada de la base de datos.

ommmm · #20 (**permalink**) 04/11/2015, 04:07

Gracias rbczgz

Com hago para que me salgan las /, porqué en el script estan puestas.
Y el tema de que no trae nada de la BBDD, no entiendo por qué no lo hace, si me dice que se ha enviado correctamente, y en la carpeta correspondiente hay el documento que he subido.

Adjunto los códigos, a ver si ayudan. Muchas gacias de nuevo!!

CODIGO que hay en el formulario, que te lleva al EmpCrear.php

Código PHP:

Ver original<!-- MAX_FILE_SIZE debe preceder al campo de entrada del fichero -->
    <input type="hidden" name="MAX_FILE_SIZE" value="300000" />
    <!-- El nombre del elemento de entrada determina el nombre en el array $_FILES -->
    Enviar este fichero: <input name="fichero_usuario" type="file" />

CODIGO EmpCrear.php

Código PHP:

Ver original<?php
// En versiones de PHP anteriores a la 4.1.0, debería utilizarse $HTTP_POST_FILES en lugar
// de $_FILES.
 
$dir_subida = 'C:\Archivos de programa\EasyPHP-DevServer-14.1VC9\data\localweb\subidas\S';
$fichero_subido = $dir_subida . basename($_FILES['archivo']['name']);
 
echo '<pre>';
if (move_uploaded_file($_FILES['archivo']['tmp_name'], $fichero_subido)) {
    echo "El fichero es válido y se subió con éxito.\n";
} else {
    echo "¡Posible ataque de subida de ficheros!\n";
}
 
echo 'Más información de depuración:';
print_r($_FILES);
 
print "</pre>";
 
 
?>
     <?php 
           
           
            require_once ('../conexion.inc.php');
            $db1 = @mysql_connect($servidor,$usuario,$password) or die("Error al conectar al DBMS.");
            @mysql_select_db($basedatos) or die("Error al conectar a la BD");
            $IdRFQ = $_POST['txtidrfq'];
            $Data = $_POST['txtdata'];
           
            $Suppliers = $_POST['txtsupplier'];            
            $Price = $_POST['txtprice'];   
            $Cry2 = $_POST['txtcry2']; 
            $MOQ = $_POST['txtmoq']; 
            $Volume = $_POST['txtvolume'];         
            $Investment = $_POST['txtinvestment']; 
            $Cry = $_POST['txtcry'];        
            $Timing = $_POST['txttiming']; 
             
            $Incoterms = $_POST['txtincoterms']; 
            $Comments = $_POST['txtcomments']; 
          
            
            
           
            
            
 
 
 
$q1= "INSERT INTO rfq (idrfq, data, suppliers, price, cry2, moq, volume, investment, timing, cry, incoterms, archivo) 
values ('$IdRFQ','$Data', '$Suppliers', '$Price', '$Cry2', '$MOQ', '$Volume', '$Investment', '$Timing', '$Cry', '$Incoterms', '$fichero_subido')"; 
$resultquery = mysql_query($q1); 
$errores = mysql_errno(); 
 
 
 
 
            
            
            mysql_close(); 
        ?>

pateketrueke · #21 (**permalink**) 04/11/2015, 07:26

Jamás deberías usar \ para las rutas si usas Windows, porque ese precisamente es el problema, la barra de escape \ es especial y arruina tu ruta inicial.

Código PHP:

Ver original// MAL
$dir_subida = 'C:\Archivos de programa\EasyPHP-DevServer-14.1VC9\data\localweb\subidas\S';
 
// BIEN
$dir_subida = 'C:\\Archivos de programa\\EasyPHP-DevServer-14.1VC9\\data\\localweb\\subidas\\S';
 
// MEJOR
$dir_subida = 'C:/Archivos de programa/EasyPHP-DevServer-14.1VC9/data/localweb/subidas/S';

Si vas a usar \ como separador de directorios debes escapar la barra \\ para evitar que rompa la cadena, o mejor aún, usar la barra / pues PHP entiende perfectamente la ruta y la corregirá para ser usada en Windows.

Otro error grave que cometes es que guardas la ruta física completa del archivo: tremendo error.

¿Por qué es un error?

Porque para poder descargarse un archivo debe ser accedido a través del servidor web, usando una ruta a partir de la raíz del sitio, no una ruta física porque es imposible descargarla.

Lo único que debes guardar es la ruta relativa desde la raíz del sitio hacia el archivo, de otra forma siempre obtendrás enlaces rotos e inútiles.

O mejor aún, sólo guardas el nombre del archivo y porque si ya sabes que la ruta es la misma ¿para qué guardar información de más?

ommmm · #22 (**permalink**) 05/11/2015, 02:25

Saludos pateketrueke y muchas gracias por la detallada explicación,

Pero debo confesar que lo de cambiar la ruta física no lo acabo de entender.
Es decir la ruta relativa desde la raiz no se a que te refieres y tampoc entiendo la ultima frase, porqué yo ya guardo el nombre del archivo, pero debo especificar-le dónde quiero guardar-lo no? Por eso pongo la ruta física que debo cambiar por una relativa que no se como debe ser.

Muchas gracias de nuevo por vuestro tiempo!!

rbczgz · #23 (**permalink**) 05/11/2015, 08:16

Hola ommmm,

La ruta física en tu caso se refiere a:

C:/Archivos de programa/EasyPHP-DevServer-14.1VC9/data/localweb/subidas/S

Y lo que te dice pateketrueke es que es mejor guardar en la BD solo el nombre del archivo puesto que la ruta siempre será la misma para todos, con lo que es mejor añadirla en la programación y así, además de no guardar información innecesaria en la BD, si la tienes que cambiar, por ejemplo, al subir tu proyecto al servidor, no necesitarás cambiarla en cada registro, solo en la programación.

La ruta a la raíz del sitio se refiere, por ejemplo en el caso de que tu sitio esté en la carpeta "localweb", la ruta debe ser desde ahí, es decir "/localweb/subidas/S"

Espero que se entienda.

ommmm · #24 (**permalink**) 06/11/2015, 06:28

Gracias por la aclaración rbczgz,

La entendí a la perfección, pero sigo sin poder descargar el archivo...

A ver si alguien tiene alguna otra idea, si no estoy probando otra forma de hacer-lo, pero tampoco cosnigo que funcione.

Saludos!!