File upload

ditismenno

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

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

ditismenno

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

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

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

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

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

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

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

Stel je hebt dit:

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

cakemasher

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

ICTscripters

Dé plek voor IT

Dé plek voor IT

Op zoek naar de legends

Na 15 jaar terug van weggeweest: iCriminals.nl is terug (BETA)!

Developer Gezocht

[FREE] WeFact Hosting module

Help testers nodig voor android app Urgent

Versio vervanger

Afspraken systeem met planbeperking

Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

321 Nieuwe Domeinnamen December 2025

Meerdere mafia game template te koop

Van een pixelige afbeelding naar een strakke, moderne website

File upload

Participate now!

Fijne feestdagen

Kritieke update voor Really Simple Security-plug-in

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Functioneel ontwerp

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

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

Gebruikers die dit topic bekijken