PHP Upload Bilderscript erweitern

Zitronella

Vielleicht ist hier jemand, der sich mit PHP auskennt und mir helfen kann.

Ich habe ein Upload Script von hier

Beispiel: Sicherer Bildupload welches super funktioniert.

Dies würde ich gerne in der Form erweitern, dass man vor dem Upload der Datei einen bestimmten Namen vergeben kann, so wie hier gezeigt

PHP Rename image while image uploading

evtl. auch noch, wenn man das Eingabefeld leer lässt, dass das Bild dann nicht nur als ".jpg" abgespeichert wird, sondern der ursprüngliche Dateinamen automatisch übernommen wird.

Ist das machbar?

Vielen Dank schon mal im Voraus

.DeJaVu

Zu letzterem als Vorbedingung:

PHP: trim - Manual

PHP empty() Function

W3Schools offers free online tutorials, references and exercises in all the major languages of the web. Covering popular subjects like HTML, CSS, JavaScript,…

www.w3schools.com

Sollte beides vor die Zeile mit "move"

Im "sicheren Upload" sollte das vor "//Pfad zum Upload" stehen

Variablen sind anzupassen, da oder da.

$filename.'.'.$extension

Oder du füllst den Dateinamen im Formular mit dem aus der ausgewählten Datei vorab. Da bin ich aber zu lang raus.

Zitronella

puh, und ich bin da nicht lange raus, ich war da nie drin

ich hab keine Ahnung, außer dass ich sehe in dem einen heißt es "datei" im anderen "file", das würde ich gerade noch so hin bekommen, dass es einheitlich ist. Das wars dann aber leider auch...

.DeJaVu

lustich

mal schauen

PHP

<?php
$upload_folder = 'uploads/files/'; //Das Upload-Verzeichnis
$filename = pathinfo($_FILES['datei']['name'], PATHINFO_FILENAME);
$extension = strtolower(pathinfo($_FILES['datei']['name'], PATHINFO_EXTENSION));
 
 
//Überprüfung der Dateiendung
$allowed_extensions = array('png', 'jpg', 'jpeg', 'gif');
if(!in_array($extension, $allowed_extensions)) {
 die("Ungültige Dateiendung. Nur png, jpg, jpeg und gif-Dateien sind erlaubt");
}
 
//Überprüfung der Dateigröße
$max_size = 500*1024; //500 KB
if($_FILES['datei']['size'] > $max_size) {
 die("Bitte keine Dateien größer 500kb hochladen");
}
 
//Überprüfung dass das Bild keine Fehler enthält
if(function_exists('exif_imagetype')) { //exif_imagetype erfordert die exif-Erweiterung
 $allowed_types = array(IMAGETYPE_PNG, IMAGETYPE_JPEG, IMAGETYPE_GIF);
 $detected_type = exif_imagetype($_FILES['datei']['tmp_name']);
 if(!in_array($detected_type, $allowed_types)) {
 die("Nur der Upload von Bilddateien ist gestattet");
 }
}

//new, check blank name
if(isset($_POST['Submit1']))
{ 
$filename = $_POST["filename"];

$filename = trim($filename);
if (empty($filename)) {
  $filename = $_FILES["file"]["name"]; //new name = old name
}

}
else
{
$filename = $_FILES["file"]["name"];
}
// end check

//Pfad zum Upload
$new_path = $upload_folder.$filename.'.'.$extension;
 
//Neuer Dateiname falls die Datei bereits existiert
if(file_exists($new_path)) { //Falls Datei existiert, hänge eine Zahl an den Dateinamen
 $id = 1;
 do {
 $new_path = $upload_folder.$filename.'_'.$id.'.'.$extension;
 $id++;
 } while(file_exists($new_path));
}
 
//Alles okay, verschiebe Datei an neuen Pfad
move_uploaded_file($_FILES['datei']['tmp_name'], $new_path);
echo 'Bild erfolgreich hochgeladen: <a href="'.$new_path.'">'.$new_path.'</a>';
?>

Alles anzeigen

basiert auf

Code

<form action="fileupload.php" enctype="multipart/form-data" method="post">
Select image :
<input type="file" name="file"><br/>
Enter image name :<input type="text" name="filename"><br/>
<input type="submit" value="Upload" name="Submit1">

</form>

Zitronella

danke. Leider kommt die Fehlermeldung "

"Ungültige Dateiendung. Nur png, jpg, jpeg und gif-Dateien sind erlaubt" es sind aber richtige jpg bzw. png Dateien.

.DeJaVu

Ich habe einen Fehler im Script gesehen, aber vermutlich ist das keine Antwort auf deine Frage. Script bitte neu übernehmen.

Zitronella

gleiche Fehlermeldung.

also der html Code und der PHP Code liegen beide im Verzeichnis uploads, das ist doch richtig oder? die Dateien sollen in "files" landen, welches ein Unterordner von "uploads" ist. Somit habe ich Zeile 2 der PHP Datei abgeändert in

Code

$upload_folder = 'files/'; //Das Upload-Verzeichnis

ist doch richtig so oder?

oh ich glaube ich habs endlich, Ich habe die PHP geändert, überall wo "datei" stand habe ich in "file" geändert

HTML

<?php
$upload_folder = 'files/'; //Das Upload-Verzeichnis
$filename = pathinfo($_FILES['file']['name'], PATHINFO_FILENAME);
$extension = strtolower(pathinfo($_FILES['file']['name'], PATHINFO_EXTENSION));
 
 
//Überprüfung der Dateiendung
$allowed_extensions = array('png', 'jpg', 'jpeg', 'gif');
if(!in_array($extension, $allowed_extensions)) {
 die("Ungültige Dateiendung. Nur png, jpg, jpeg und gif-Dateien sind erlaubt");
}
 
//Überprüfung der Dateigröße
$max_size = 500*1024; //500 KB
if($_FILES['file']['size'] > $max_size) {
 die("Bitte keine Dateien größer 500kb hochladen");
}
 
//Überprüfung dass das Bild keine Fehler enthält
if(function_exists('exif_imagetype')) { //exif_imagetype erfordert die exif-Erweiterung
 $allowed_types = array(IMAGETYPE_PNG, IMAGETYPE_JPEG, IMAGETYPE_GIF);
 $detected_type = exif_imagetype($_FILES['file']['tmp_name']);
 if(!in_array($detected_type, $allowed_types)) {
 die("Nur der Upload von Bilddateien ist gestattet");
 }
}

//new, check blank name
if(isset($_POST['Submit1']))
{ 

$extension = pathinfo($_FILES["file"]["name"], PATHINFO_EXTENSION);
$filename = $_POST["filename"];

$filename = trim($filename);
if (empty($filename)) {
  $filename = $_FILES["file"]["name"]; //new name = old name
}

}
else
{
$filename = $_FILES["file"]["name"];
}
// end check

//Pfad zum Upload
$new_path = $upload_folder.$filename.'.'.$extension;
 
//Neuer Dateiname falls die Datei bereits existiert
if(file_exists($new_path)) { //Falls Datei existiert, hänge eine Zahl an den Dateinamen
 $id = 1;
 do {
 $new_path = $upload_folder.$filename.'_'.$id.'.'.$extension;
 $id++;
 } while(file_exists($new_path));
}
 
//Alles okay, verschiebe Datei an neuen Pfad
move_uploaded_file($_FILES['file']['tmp_name'], $new_path);
echo 'Bild erfolgreich hochgeladen: <a href="'.$new_path.'">'.$new_path.'</a>';
?>

Alles anzeigen

jetzt mal noch testen ob alles gut funktioniert.

Zitronella

Ich danke dir vielmals

nur ein kleiner Fehler: aus "test.png" wird wenn Eingabefeld leer gelassen wird "test.png.png"

kann man das noch irgendwie ändern?

Sören Hentzschel

// Das ist zwar nur ein Fehler in einem Kommentar und hat damit keine Auswirkungen, aber das ist so offensichtlich, dass ich es loswerden muss:

Code

$max_size = 500*1024; //500 KB

500 Bytes * 1000 wären 500 KB (Kilobytes), aber 500 Bytes * 1024 sind 500 KiB (Kibibytes).

.DeJaVu

Zitat von Zitronella

kann man das noch irgendwie ändern?

Bitte das neue Script oben testen, $extension wurde zwischendurch neu geschrieben, sollte nicht.

Doku zu "pathinfo"

PHP pathinfo() Function

W3Schools offers free online tutorials, references and exercises in all the major languages of the web. Covering popular subjects like HTML, CSS, JavaScript,…

www.w3schools.com

Zitronella

Code sieht nun so aus, habe Zeile 32 auskommentiert

PHP

<?php
$upload_folder = 'uploads/'; //Das Upload-Verzeichnis
$filename = pathinfo($_FILES['file']['name'], PATHINFO_FILENAME);
$extension = strtolower(pathinfo($_FILES['file']['name'], PATHINFO_EXTENSION));
 
 
//Überprüfung der Dateiendung
$allowed_extensions = array('png', 'jpg', 'jpeg', 'gif');
if(!in_array($extension, $allowed_extensions)) {
 die("Ungültige Dateiendung. Nur png, jpg, jpeg und gif-Dateien sind erlaubt");
}
 
//Überprüfung der Dateigröße
$max_size = 5000*1024; //500 KB
if($_FILES['file']['size'] > $max_size) {
 die("Bitte keine Dateien größer 5000kb hochladen");
}
 
//Überprüfung dass das Bild keine Fehler enthält
if(function_exists('exif_imagetype')) { //exif_imagetype erfordert die exif-Erweiterung
 $allowed_types = array(IMAGETYPE_PNG, IMAGETYPE_JPEG, IMAGETYPE_GIF);
 $detected_type = exif_imagetype($_FILES['file']['tmp_name']);
 if(!in_array($detected_type, $allowed_types)) {
 die("Nur der Upload von Bilddateien ist gestattet");
 }
}

//new, check blank name
if(isset($_POST['Submit1']))
{ 

//$extension = pathinfo($_FILES["file"]["name"], PATHINFO_EXTENSION);
$filename = $_POST["filename"];

$filename = trim($filename);
if (empty($filename)) {
  $filename = $_FILES["file"]["name"]; //new name = old name
}

}
else
{
$filename = $_FILES["file"]["name"];
}
// end check

//Pfad zum Upload
$new_path = $upload_folder.$filename.'.'.$extension;
 
//Neuer Dateiname falls die Datei bereits existiert
if(file_exists($new_path)) { //Falls Datei existiert, hänge eine Zahl an den Dateinamen
 $id = 1;
 do {
 $new_path = $upload_folder.$filename.'_'.$id.'.'.$extension;
 $id++;
 } while(file_exists($new_path));
}
 
//Alles okay, verschiebe Datei an neuen Pfad
move_uploaded_file($_FILES['file']['tmp_name'], $new_path);
echo 'Bild erfolgreich hochgeladen: <a href="'.$new_path.'">'.$new_path.'</a>';
?>

Alles anzeigen

Fehler besteht weiterhin. aus test.png wird test.png.png

.DeJaVu

Lass dir bitte die einzelnen Variablen mit echo anzeigen, ab wann der Fehler auftritt. Ich hab leider keine PHP-Umgebung zum Testen.

Zitronella

Zitat von Sören Hentzschel

zwar nur ein Fehler in einem Kommentar

ich raff es nicht. Die Größe ist mir eh zu wenig, es sollen bis zu 5MB große Bilder hoch geladen werden können. Wie müsste das dann heißen?

.DeJaVu

Müsste nach Beispiel sein

$max_size = 5*1024*1024;

Was Sören meint ist, dass K für Kilo = 1000 steht, es sollte nach neuer Ordnung Kibi sein = 1024.

Meine 1TB hat nur 930Mebibyte.

Zitronella

Zitat von .DeJaVu

Lass dir bitte die einzelnen Variablen mit echo anzeigen

Wie meinen? Was genau muss ich tun? Sorry, ich hab wirklich wenig Ahnung von der Materie.

.DeJaVu

Aus deinem Link, da steht was mit echo

PHP Rename image while image uploading

Einfach zum Debuggen zwischendurch einsetzen, so halte ich das auch bei meinen Scripten, wenn das Ergebnis falsch ist. Damit kann ich eingrenzen, wann was steht und wo nicht.

Zitronella

ja, da steht irgendwas mit echo. Und was soll ich genau wo hin kopieren und ausführen?

Bitte ganz ausführlich erklären was ich machen muss, ich kenne mich wirklich nicht damit aus. Danke

.DeJaVu

Ich bin da eben mit rapid php rein und da ist mir was aufgefallen. bitte vorab testen, den teil mit php musst du wahrscheinlich rauskopieren. weiter bin ich damit nicht gekommen, weil ich das html nicht zum laufen bekomme.

PHP

<!DOCTYPE html>

<html>

<head>
  <title>Bild hochladen</title>
</head>

<body>

<form action="fileupload.php" enctype="multipart/form-data" method="post">
Select image :
<input type="file" name="file"><br/>
Enter image name :<input type="text" name="filename"><br/>
<input type="submit" value="Upload" name="Submit1">

</form>

<?php
$upload_folder = 'uploads/files/'; //Das Upload-Verzeichnis
$filename_old = pathinfo($_FILES['file']['name'], PATHINFO_FILENAME);
$extension = strtolower(pathinfo($_FILES['file']['name'], PATHINFO_EXTENSION));

//Überprüfung der Dateiendung
$allowed_extensions = array('png', 'jpg', 'jpeg', 'gif');
if (!in_array($extension, $allowed_extensions))
{
    die("Ungültige Dateiendung. Nur png, jpg, jpeg und gif-Dateien sind erlaubt");
}

//Überprüfung der Dateigröße
$max_size = 5 * 1024 * 1024; //5 MB
if ($_FILES['file']['size'] > $max_size)
{
    die("Bitte keine Dateien größer 5MB hochladen");
}

//Überprüfung dass das Bild keine Fehler enthält
if (function_exists('exif_imagetype')) //exif_imagetype erfordert die exif-Erweiterung
{
    $allowed_types = array(IMAGETYPE_PNG, IMAGETYPE_JPEG, IMAGETYPE_GIF);
    $detected_type = exif_imagetype($_FILES['file']['tmp_name']);
    if (!in_array($detected_type, $allowed_types))
    {
        die("Nur der Upload von Bilddateien ist gestattet");
    }
}

//new, check blank name
if (isset($_POST['Submit1']))
{
    $filename_new = $_POST["filename"];
    $filename_new = pathinfo($filename_new, PATHINFO_FILENAME);

    $filename_new = trim($filename_new);
    if (empty($filename_new))
    {
        $filename_new = $filename_old; //new name = old name
    }

}
else
{
    $filename_new = $filename_old;
}

// end check

//Pfad zum Upload
$new_path = $upload_folder.$filename_new.'.'.$extension;

//Neuer Dateiname falls die Datei bereits existiert
if (file_exists($new_path)) //Falls Datei existiert, hänge eine Zahl an den Dateinamen
{
    $id = 1;
    do
    {
        $new_path = $upload_folder . $filename_new . '_' . $id . '.' . $extension;
        $id++;
    }
    while (file_exists($new_path));
}

//Alles okay, verschiebe Datei an neuen Pfad
move_uploaded_file($_FILES['datei']['tmp_name'], $new_path);
echo 'Bild erfolgreich hochgeladen: <a href="' . $new_path . '">' . $new_path . '</a>';
?>

</body>
</html>

Alles anzeigen

Zitronella

Ging erst nicht, es gab zwar die Meldung es sei erfolgreich hoch geladen worden, aber Datei war nicht auf dem Server.

Bis ich merkte, dass in Zeile 84 wieder ['datei'] stand anstatt ['file']

als ich das änderte klappt auch alles prima jetzt. Keine doppelte Dateiendung mehr!

Vielen Dank

.DeJaVu

Das ist mir auch aufgefallen, diese Änderung stand schon drin, es gibt aber noch eine weitere Änderung seit eben.