no subir segun que extension

[visona]

Hola a todos, he colocado este codigo y me funciona bien, pero necesito que no se puedan subir segun que archivos, solo quiero que se puedan subir JPG, BMP, y GIF

Código PHP:

  srand (time()); 
//generamos un número aleatorio 
$mismo = rand(1,100000000000); 
$pdfnumero = rand(1,100000000000); 
$mismopdf = $pdfnumero . '.pdf'; 
 
 
$editFormAction = $_SERVER['PHP_SELF']; 
if (isset($_SERVER['QUERY_STRING'])) { 
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']); 
} 
 
if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "insertar")) { 
  $insertSQL = sprintf("INSERT INTO productos (nombre, categoria, descripcion,  imagen, pdf, usuario) VALUES (%s, %s, %s, %s, %s, %s)", 
                       GetSQLValueString($_POST['Nombre'], "text"), 
                        
                       GetSQLValueString($_POST['categoria'], "text"), 
                       GetSQLValueString($_POST['descripcion'], "text"), 
                        
                       GetSQLValueString("../archivos/".$mismo.$_FILES['imagen'] ['name'], "text"), 
                       GetSQLValueString("../pdf/" . $_FILES['pdf'] .$mismopdf, "text"), 
                       GetSQLValueString($_POST['usuario'], "text")); 
                        
$copy = copy($_FILES['imagen']['tmp_name'], "../archivos/" . $_FILES['imagen'] .$mismo ); 
$copy = copy($_FILES['pdf']['tmp_name'], "pdf/" . $_FILES['pdf'] .$mismopdf);  
   

Que tendría que añadir a este codigo para que solo se pudieran subir esos archivos?
Un saludo

[PATUX]

tendria que ser un if no mas que pregunte por las extension

algo asi como

if (substr($archivos,strlen($archivo)-4,4) == ".gif" or substr($archivos,strlen($archivo)-4,4) == ".jpg")
{
// aqui Ejecutas el codigo siempre y cuando sea de las extensiones que tu kieres.
}


ajusta el If a tu codigo y dinos como te fue.

saludos.

PAtux

[visona]

Iniciado por PATUX

tendria que ser un if no mas que pregunte por las extension

algo asi como

if (substr($archivos,strlen($archivo)-4,4) == ".gif" or substr($archivos,strlen($archivo)-4,4) == ".jpg")
{
// aqui Ejecutas el codigo siempre y cuando sea de las extensiones que tu kieres.
}


ajusta el If a tu codigo y dinos como te fue.

saludos.

PAtux

Pues no me funcina.
Una consulta, por que pones $archivo, y me explico un poco mas a ver si me podeis ayudar.
cuando subo el archivo genero un numero aleatorio que se le añade antes del nombre original de la imagen. Osea que si la imagen original que el usuario pone se llama "miimagen.jpg" el nombre que me inserta en la BD es 123456789miimagen.jpg

Código PHP:

  GetSQLValueString("../archivos/".$mismo.$_FILES['imagen'] ['name'], "text"),  
   

La variable $mismo + el nombre original es lo que va a insertar en la BD.
Tambien quiero que me imprima una alerta y ya puestos que me quite los espacios en blanco.
Un saludo y gracias

[abimaelrc]

Puedes hacerlo de esta forma antes de todo el codigo

Código PHP:

if($_FILES["imagen"]["type"] != "image/pjpeg" 
  && $_FILES["imagen"]["type"] != "image/jpeg" 
  && $_FILES["imagen"]["type"] != "image/x-png"
  && $_FILES["imagen"]["type"] != "image/png" 
  && $_FILES["imagen"]["type"] != "image/gif" ){
  //aqui va el codigo sino es un jpg por ejemplo
  header("Location: index.php");
}
 
Bueno ahi esta declarado el de png pero haz prueba añadiendo en diferentes navegadores como $_FILE["imagen"]["type"] trae la informacion de un bmp
 

[visona]

Iniciado por abimaelrc

Puedes hacerlo de esta forma antes de todo el codigo

Código PHP:

if($_FILES["imagen"]["type"] != "image/pjpeg" 
  && $_FILES["imagen"]["type"] != "image/jpeg" 
  && $_FILES["imagen"]["type"] != "image/x-png"
  && $_FILES["imagen"]["type"] != "image/png" 
  && $_FILES["imagen"]["type"] != "image/gif" ){
  //aqui va el codigo sino es un jpg por ejemplo
  header("Location: index.php");
}
 
Bueno ahi esta declarado el de png pero haz prueba añadiendo en diferentes navegadores como $_FILE["imagen"]["type"] trae la informacion de un bmp
 

Me tira al index directamente te pego el codigo tal como lo he colocado

Código PHP:

  if($_FILES["imagen"]["type"] != "image/pjpeg"  
  && $_FILES["imagen"]["type"] != "image/jpeg"  
  && $_FILES["imagen"]["type"] != "image/x-png" 
  && $_FILES["imagen"]["type"] != "image/png"  
  && $_FILES["imagen"]["type"] != "image/gif" ){ 
   
srand (time()); 
//generamos un número aleatorio 
$mismo = rand(1,100000000000);  
$pdfnumero = rand(1,100000000000); 
$mismopdf = $pdfnumero . '.pdf'; 
 
 
$editFormAction = $_SERVER['PHP_SELF']; 
if (isset($_SERVER['QUERY_STRING'])) { 
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']); 
} 
 
 
 
if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "insertar")) { 
  $insertSQL = sprintf("INSERT INTO productos (nombre, categoria, descripcion,  imagen, pdf, usuario, val_vendedor) VALUES (%s, %s, %s, %s, %s, %s, %s)", 
                       GetSQLValueString($_POST['Nombre'], "text"), 
                        
                       GetSQLValueString($_POST['categoria'], "text"), 
                       GetSQLValueString($_POST['descripcion'], "text"), 
                        
                       GetSQLValueString("../archivos/".$mismo.$_FILES['imagen'] ['name'], "text"), 
                       GetSQLValueString("../pdf/" . $_FILES['pdf'] .$mismopdf, "text"), 
                       GetSQLValueString($_POST['val_vendedor'], "text"), 
                       GetSQLValueString($_POST['usuario'], "text")); 
                        
$copy = copy($_FILES['imagen']['tmp_name'], "../archivos/" . $_FILES['imagen'] .$mismo ); 
$copy = copy($_FILES['pdf']['tmp_name'], "pdf/" . $_FILES['pdf'] .$mismopdf); 
 
  mysql_select_db($database_compramos, $compramos); 
  $Result1 = mysql_query($insertSQL, $compramos) or die(mysql_error()); 
 
  $insertGoTo = "usuario.php"; 
  if (isset($_SERVER['QUERY_STRING'])) { 
    $insertGoTo .= (strpos($insertGoTo, '?')) ? "&" : "?"; 
    $insertGoTo .= $_SERVER['QUERY_STRING']; 
  } 
  header(sprintf("Location: %s", $insertGoTo)); 
} 
 
$editFormAction = $_SERVER['PHP_SELF']; 
if (isset($_SERVER['QUERY_STRING'])) { 
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']); 
} 
 
header("Location: index.php"); 
}  
   

[abimaelrc]

Ese if es si no es un archivo jpg, png o gif. En otras palabras arriba debes colocarlo asi

Código PHP:

if($_FILES["imagen"]["type"] != "image/pjpeg"  
  && $_FILES["imagen"]["type"] != "image/jpeg"  
  && $_FILES["imagen"]["type"] != "image/x-png" 
  && $_FILES["imagen"]["type"] != "image/png"  
  && $_FILES["imagen"]["type"] != "image/gif" ){ 
  header("Location: index.php");
  die();
}
 

Si se cumple una de los que he mencionado continua el codigo, sino te redirecciona al index.php

[DvD AdN]

Que no en el codigo deberias usa Or y no And?
Porque solo terminaria el script si el mimetype de imagen fuera todos los tipos listados...

Editado
Hagan caso omiso a este post xD

[DvD AdN]

Tambien podria ser

Código:

if($_FILES["imagen"]["type"] == "image/pjpeg"  

  || $_FILES["imagen"]["type"] == "image/jpeg"  

  || $_FILES["imagen"]["type"] == "image/x-png"

  || $_FILES["imagen"]["type"] == "image/png"  

  || $_FILES["imagen"]["type"] == "image/gif" ){
//subes la imagen
}else{
echo "error blablabla tipo de imagen debe ser solo blablabla"

[visona]

ok ya funciona. Muchas gracias