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"]);
							}
						}
					}

Meer tonen

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;
						}
					}

Meer tonen

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";
  }
?>

Meer tonen

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";
  }
?>

Meer tonen

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);

Meer tonen

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);

Meer tonen

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 />";
	}
	
?>

Meer tonen

ICTscripters

Dé plek voor IT

Dé plek voor IT

nieuwe game

[Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

Herkent deze Notion-structuurscan echte problemen?

🚀 WE ZOEKEN TESTERS! 🚀

[Aanbod] Stadskaart voor je browsergame: van menu naar klikbare stad, vanaf €200

[Verkoop] G4G (Gangsters 4 Gangsters): volledige broncode, nieuw gebouwd in PHP 8.3 | € 30 per domein

[TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

File upload

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

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