File upload

ditismenno · 20 Januari 2014

Hallo,

Ik ben bezig met een soort marktplaats script en daarvoor zorg ik dat je 3 foto's kan uploaden:

PHP

if (isset($_POST['file1'])){
						$allowedExts = array("gif", "jpeg", "jpg", "png");
						$temp = explode(".", $_FILES["file1"]["name"]);
						$extension = end($temp);
						if ((($_FILES["file1"]["type"] == "image/gif")
						|| ($_FILES["file1"]["type"] == "image/jpeg")
						|| ($_FILES["file1"]["type"] == "image/jpg")
						|| ($_FILES["file1"]["type"] == "image/pjpeg")
						|| ($_FILES["file1"]["type"] == "image/x-png")
						|| ($_FILES["file1"]["type"] == "image/png"))
						&& in_array($extension, $allowedExts)){
							if ($_FILES["file1"]["error"] > 0){
								echo 112;
							} else {
								echo 1;
								$url1 = 'upload/'.$ran1.''.$_FILES["file1"]["name"].'';
								move_uploaded_file($_FILES["file1"]["tmp_name"],
								"upload/" . $ran1 . $_FILES["file1"]["name"]);
							}
						} else {
							echo 2;
						}
					}
					
					if (isset($_POST['file2'])){
						$allowedExts = array("gif", "jpeg", "jpg", "png");
						$temp = explode(".", $_FILES["file2"]["name"]);
						$extension = end($temp);
						if ((($_FILES["file2"]["type"] == "image/gif")
						|| ($_FILES["file2"]["type"] == "image/jpeg")
						|| ($_FILES["file2"]["type"] == "image/jpg")
						|| ($_FILES["file2"]["type"] == "image/pjpeg")
						|| ($_FILES["file2"]["type"] == "image/x-png")
						|| ($_FILES["file2"]["type"] == "image/png"))
						&& ($_FILES["file2"]["size"] < 20000)
						&& in_array($extension, $allowedExts)){
							if ($_FILES["file2"]["error"] > 0){
								
							} else {
								$url2 = 'upload/'.$ran2.''.$_FILES["file2"]["name"].'';
								move_uploaded_file($_FILES["file2"]["tmp_name"],
								"upload/" . $ran2 . $_FILES["file2"]["name"]);
							}
						}
					}
					if (isset($_POST['file3'])){
						$allowedExts = array("gif", "jpeg", "jpg", "png");
						$temp = explode(".", $_FILES["file3"]["name"]);
						$extension = end($temp);
						if ((($_FILES["file3"]["type"] == "image/gif")
						|| ($_FILES["file3"]["type"] == "image/jpeg")
						|| ($_FILES["file3"]["type"] == "image/jpg")
						|| ($_FILES["file3"]["type"] == "image/pjpeg")
						|| ($_FILES["file3"]["type"] == "image/x-png")
						|| ($_FILES["file3"]["type"] == "image/png"))
						&& ($_FILES["file3"]["size"] < 20000)
						&& in_array($extension, $allowedExts)){
							if ($_FILES["file3"]["error"] > 0){
								
							} else {
								$url3 = 'upload/'.$ran3.''.$_FILES["file3"]["name"].'';
								move_uploaded_file($_FILES["file3"]["tmp_name"],
								"upload/" . $ran3 . $_FILES["file3"]["name"]);
							}
						}
					}

Toon Meer

Maar omdat dit niet werkt ben ik gaan debuggen met het bovenste stukje:

PHP

if (isset($_POST['file1'])){
						$allowedExts = array("gif", "jpeg", "jpg", "png");
						$temp = explode(".", $_FILES["file1"]["name"]);
						$extension = end($temp);
						if ((($_FILES["file1"]["type"] == "image/gif")
						|| ($_FILES["file1"]["type"] == "image/jpeg")
						|| ($_FILES["file1"]["type"] == "image/jpg")
						|| ($_FILES["file1"]["type"] == "image/pjpeg")
						|| ($_FILES["file1"]["type"] == "image/x-png")
						|| ($_FILES["file1"]["type"] == "image/png"))
						&& in_array($extension, $allowedExts)){
							if ($_FILES["file1"]["error"] > 0){
								echo 112;
							} else {
								echo 1;
								$url1 = 'upload/'.$ran1.''.$_FILES["file1"]["name"].'';
								move_uploaded_file($_FILES["file1"]["tmp_name"],
								"upload/" . $ran1 . $_FILES["file1"]["name"]);
							}
						} else {
							echo 2;
						}
					}

Toon Meer

Maar ik snap niet waarom hij 2 echo't?

Alvast bedankt.

M.Beers · 20 Januari 2014

Upload je toevallig een .JPG file ipv .jpg? want hij checkt niet of de extensie van het bestand uit grote/kleine letters bestaat

ditismenno · 20 Januari 2014

Citaat van M.Beers

Upload je toevallig een .JPG file ipv .jpg? want hij checkt niet of de extensie van het bestand uit grote/kleine letters bestaat

net met een .jpg ook even geprobeerd met .png maar dat werkt ook niet

M.Beers · 20 Januari 2014

PHP

<?php
$allowedExts = array("gif", "jpeg", "jpg", "png");
$temp = explode(".", $_FILES["file"]["name"]);
$extension = end($temp);
if ((($_FILES["file"]["type"] == "image/gif")
|| ($_FILES["file"]["type"] == "image/jpeg")
|| ($_FILES["file"]["type"] == "image/jpg")
|| ($_FILES["file"]["type"] == "image/pjpeg")
|| ($_FILES["file"]["type"] == "image/x-png")
|| ($_FILES["file"]["type"] == "image/png"))
&& ($_FILES["file"]["size"] < 20000)
&& in_array($extension, $allowedExts))
  {
  if ($_FILES["file"]["error"] > 0)
    {
    echo "Return Code: " . $_FILES["file"]["error"] . "<br>";
    }
  else
    {
    echo "Upload: " . $_FILES["file"]["name"] . "<br>";
    echo "Type: " . $_FILES["file"]["type"] . "<br>";
    echo "Size: " . ($_FILES["file"]["size"] / 1024) . " kB<br>";
    echo "Temp file: " . $_FILES["file"]["tmp_name"] . "<br>";


    if (file_exists("upload/" . $_FILES["file"]["name"]))
      {
      echo $_FILES["file"]["name"] . " already exists. ";
      }
    else
      {
      move_uploaded_file($_FILES["file"]["tmp_name"],
      "upload/" . $_FILES["file"]["name"]);
      echo "Stored in: " . "upload/" . $_FILES["file"]["name"];
      }
    }
  }
else
  {
  echo "Invalid file";
  }
?>

Toon Meer

Dit zou moeten werken

ditismenno · 20 Januari 2014

Citaat van M.Beers

PHP

<?php
$allowedExts = array("gif", "jpeg", "jpg", "png");
$temp = explode(".", $_FILES["file"]["name"]);
$extension = end($temp);
if ((($_FILES["file"]["type"] == "image/gif")
|| ($_FILES["file"]["type"] == "image/jpeg")
|| ($_FILES["file"]["type"] == "image/jpg")
|| ($_FILES["file"]["type"] == "image/pjpeg")
|| ($_FILES["file"]["type"] == "image/x-png")
|| ($_FILES["file"]["type"] == "image/png"))
&& ($_FILES["file"]["size"] < 20000)
&& in_array($extension, $allowedExts))
  {
  if ($_FILES["file"]["error"] > 0)
    {
    echo "Return Code: " . $_FILES["file"]["error"] . "<br>";
    }
  else
    {
    echo "Upload: " . $_FILES["file"]["name"] . "<br>";
    echo "Type: " . $_FILES["file"]["type"] . "<br>";
    echo "Size: " . ($_FILES["file"]["size"] / 1024) . " kB<br>";
    echo "Temp file: " . $_FILES["file"]["tmp_name"] . "<br>";


    if (file_exists("upload/" . $_FILES["file"]["name"]))
      {
      echo $_FILES["file"]["name"] . " already exists. ";
      }
    else
      {
      move_uploaded_file($_FILES["file"]["tmp_name"],
      "upload/" . $_FILES["file"]["name"]);
      echo "Stored in: " . "upload/" . $_FILES["file"]["name"];
      }
    }
  }
else
  {
  echo "Invalid file";
  }
?>

Toon Meer

Dit zou moeten werken

Is er ook op gebaseerd maar hoe doe je dat me 3 verschillende foto's?

M.Beers · 20 Januari 2014

file[] in je input field zetten als name ... vervolgens is het makkelijk om een sorteer functie te gebruiken waarmee je vervolgens alles kan uploaden d.m.v. een for-loopje

Onderstaand de functie

PHP

function multiple_files(array $_files, $top = TRUE)
{
    $files = array();
    foreach($_files as $name=>$file){
        if($top) $sub_name = $file['name'];
        else    $sub_name = $name;
        
        if(is_array($sub_name)){
            foreach(array_keys($sub_name) as $key){
                $files[$name][$key] = array(
                    'name'     => $file['name'][$key],
                    'type'     => $file['type'][$key],
                    'tmp_name' => $file['tmp_name'][$key],
                    'error'    => $file['error'][$key],
                    'size'     => $file['size'][$key],
                );
                $files[$name] = multiple_files($files[$name], FALSE);
            }
        }else{
            $files[$name] = $file;
        }
    }
    return $files;
}


$files = multiple_files($_FILES);
print_r($files);

Toon Meer

ditismenno · 21 Januari 2014

Citaat van M.Beers

file[] in je input field zetten als name ... vervolgens is het makkelijk om een sorteer functie te gebruiken waarmee je vervolgens alles kan uploaden d.m.v. een for-loopje

Onderstaand de functie

PHP

function multiple_files(array $_files, $top = TRUE)
{
    $files = array();
    foreach($_files as $name=>$file){
        if($top) $sub_name = $file['name'];
        else    $sub_name = $name;
        
        if(is_array($sub_name)){
            foreach(array_keys($sub_name) as $key){
                $files[$name][$key] = array(
                    'name'     => $file['name'][$key],
                    'type'     => $file['type'][$key],
                    'tmp_name' => $file['tmp_name'][$key],
                    'error'    => $file['error'][$key],
                    'size'     => $file['size'][$key],
                );
                $files[$name] = multiple_files($files[$name], FALSE);
            }
        }else{
            $files[$name] = $file;
        }
    }
    return $files;
}


$files = multiple_files($_FILES);
print_r($files);

Toon Meer

Oke dus als ik het nu goed begrijp moeten alle 3 de file inputs nu file[] als name hebben?

M.Beers · 21 Januari 2014

wat je had is makkelijker dus file1, file2 etc. had de code uit een projectje van mezelf gepakt zonder het even goed door te lezen... hij maakt er nu automatisch een mooie array van d.m.v. die sorteer functie anders had je namelijk conflicten als je $_FILE ging vardumpen

ditismenno · 21 Januari 2014

Hmm ik probeer het zo apart mogelijk te laten verlopen want 3 foto's zijn niet verplicht.
Maar de fout zit dus in:

PHP

if ((($_FILES["file1"]["type"] == "image/gif")
                        || ($_FILES["file1"]["type"] == "image/jpeg")
                        || ($_FILES["file1"]["type"] == "image/jpg")
                        || ($_FILES["file1"]["type"] == "image/pjpeg")
                        || ($_FILES["file1"]["type"] == "image/x-png")
                        || ($_FILES["file1"]["type"] == "image/png"))
                        && in_array($extension, $allowedExts)){

Maar mag ["file1"] dan geen file1 zijn want meestal is dat alleen file

GiovanniK · 22 Januari 2014

Stel je hebt dit:

Dan krijg je dus een array met de naam hoi met de waardes 1 t/m 3.

cakemasher · 23 Januari 2014

En om te kijken wat er precies in die array staat, print 'm even. Kan je ook meteen zien hoe je je script moet aanpassen.

Code

<input name="bestand[]" type="file" />
<input name="bestand[]" type="file" />
<input name="bestand[]" type="file" />

PHP

<?php
	
	if ($_SERVER['REQUEST_METHOD'] == 'POST')
	{
		echo "<pre>";
		
		print_r ($_FILES);
		
		echo "</pre><br /><br />";
	}
	
?>

Toon Meer

een eigen cloud sever

DNS update Directadmin

Graag advies voor mijn website

Ik wil op afstand studeren, kennen jullie Tech?

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

Ik ben op zoek naar een goede hosting

Dating software

250 Nieuwe domeinnamen April 2024

Xenforo buckaroo plugin

File upload

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Delen