Problema subiendo imagen con php

[Narxon]

Buenas otra vez! para variar encontré otro error en mi código xD a ver si me pueden ayudar a arreglar este "uploader" casero que hice, el problema radica en que no filtra como yo le mando, si no que sube el archivo que le indiques aunque no sea .jpeg, ojalá encuentren el fallo :P

Código PHP:

      if(isset($_POST['Modelo']))//Validamos que el formulario fue enviado
    {
        //variables BD
        $modelo=$_POST["Modelo"];
        $precio=$_POST["Precio"];
        $tipo=$_POST["Tipo"];
        //variables subir archivo
        $nombre_archivo=$_FILES['fichero']['name'];
        $tipo_archivo=$_FILES['fichero']['type'];
        $tamano_archivo=$_FILES['fichero']['size'];
        $ruta_archivo=$_FILES['fichero']['tmp_name'];
        //comprobar si las caracteristicas son las deseadas
        if($tipo_archivo=="jpeg" && $tamano_archivo <25000)
        {
            echo"El archivo debe ser .jpeg y pesar menos de 250Kb.";
        }
        else
        {
            if (move_uploaded_file($_FILES['fichero']['tmp_name'], "fotos/".$nombre_archivo))
            {
                mysql_query("INSERT INTO coches(foto,modelo,precio,tipo) VALUES ('$nombre_archivo','$modelo','$precio','$tipo')");
                echo "El fichero ya ha sido cargado correctamente en imágenes como $nombre_archivo .";
            }
            else
            {
                echo "Ocurrió algún error al subir el fichero. No pudo guardarse.";
            }
        }
    } 
   

Gracias por su ayuda

[pepeluis1]

Bueno primero imprime las variables haber si te estan trayando resultados:

echo "tipo_archivo = ".$tipo_archivo."<br>";

echo "tamano_archivo = ".$tamano_archivo."<br>";

[jcxnet]

Código PHP:

  ...
 if($tipo_archivo=="jpeg" && $tamano_archivo <25000) {
            echo"El archivo debe ser .jpeg y pesar menos de 250Kb.";
 }
else
... 
   

significa que subirá cualquiera que no sea .jpeg y tenga más de 250kb
debes cambiar la condición

[andystefano2]

Te recomiendo usar la clase class.upload para hacer el trabajo que deseas hacer es muy buena, recomendado 100% , te maneja errores , conbierte formatos tamaños y todo muy facilmente.
saludos!

[DavidBL]

Iniciado por jcxnet

Código PHP:

  ...
 if($tipo_archivo=="jpeg" && $tamano_archivo <25000) {
            echo"El archivo debe ser .jpeg y pesar menos de 250Kb.";
 }
else
... 
   

significa que subirá cualquiera que no sea .jpeg y tenga más de 250kb
debes cambiar la condición

Creo que como dice jcxnet en vez de :
if($tipo_archivo==
Seria:
if($tipo_archivo!=

[Narxon]

Ah si disculpad, me faltó negar esa condición !, el caso es que permite subir archivos .jpeg igualmente, cosa que no debería permitir. En cuanto a lo de class.upload...no tengo ni idea de como rula eso, me lo miraré a ver si sale algo bueno!

[Mxrck]

a ver a ver el problema esta en que segun yo recuerde
esto:

$tipo_archivo=$_FILES['fichero']['type'];

devuelve algo como esto:

"image/jpeg"

por lo que deberias cambiar el

if($tipo_archivo=="jpeg" && $tamano_archivo >25000)

por

if($tipo_archivo!="image/jpeg" && $tamano_archivo <25000)

[kirst]

Jajaja no es asi.... Ninguna de las formas que dicen, tenes que pasarlo primero por un strpos:

Código PHP:

      if ( !empty($HTTP_POST_FILES['userfile']['name']) )         $file_name         = $HTTP_POST_FILES['userfile']['name'];
    if ( !empty($HTTP_POST_FILES['userfile']['tmp_name']) )    $file_name_tmp    = $HTTP_POST_FILES['userfile']['tmpname'];
    if ( !empty($HTTP_POST_FILES['userfile']['type']) )         $file_type         = $HTTP_POST_FILES['userfile']['type'];
    if ( !empty($HTTP_POST_FILES['userfile']['size']) )     $file_size         = $HTTP_POST_FILES['userfile']['size'];



    if ( empty($file_name) )            $error['file']                    = 'Es obligatorio completar éste campo.';
                                        
    if (!( (strpos($file_type, "gif") or strpos($file_type, "png") or strpos($file_type, "jpg") or strpos($file_type, "jpeg") ) ))
                                        $error['file']                    = 'La imagen no concuerda con los tipos permitidos. '.$file_type;

    if (! ($file_size < 300000) )        $error['file']                    = 'La imagen pesa más de 300KBytes.'; 
   

Espero que te sirva, saludos!

[Narxon]

Vale, muchísimas gracias a todos, sobre todo a Mxrck, y por cierto el valor que devuelve $_FILES['fichero']['type'] es concretamente "image/pjpeg"
lo que hice fue poner un echo $tipo_fichero para comprobarlo y ya vi el error! Gracias por darme la pista de que podía estar mal el tipo de fichero! :D