"erweiterte" Galerie

DarkSouL1221 · 8. September 2014

So ich komme leider doch noch nicht ganz so gut voran.
Bin gerade am Script für den Upload von Bildern.
Habe es mir So gedacht das die geuploadete Datei "image-1" genannt wird und die nächste dann "image-2" und so weiter.
Hier mein Script:

PHP

<?php
    $id = 0;
    $image = "image-".$id
    move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name']);
    move_uploaded_file($_FILES['gbild']['tmp_name'], 'gallerie/gr/'.$_FILES['gbild']['name']);
    if (move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name']);){
        $id=$id+1;
        rename ('gallerie/'.$_FILES['vbild']['name'], "gallerie/".$image".jpg");
        rename ('gallerie/'.$_FILES['bbild']['name'], "gallerie/gr/".$image".jpg");
    }
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="de" lang="de">
 <head>
  <title>Geschützter Bereich</title>
 </head>
 <body>
  <form action="upload.php" method="post" enctype="multipart/form-data">
   Vorschaubild: <input name="vbild" type="file">
   <br /><br />
   richtiges Bild: <input name="gbild" type="file">
   <br>
   <input type="submit" value="Upload" />
  </form>
 </body>
</html>

Alles anzeigen

Habe das Script so auf den Server geladen aber wenn ich die Seite öffne bekomme ich folgende Meldung: "Parse error: syntax error, unexpected 'move_uploaded_file' (T_STRING) in /users/darksoul/www/TestSeite/upload.php on line 5"
Wo liegt da der Fehler beim parsen?

cottton · 8. September 2014

"syntax error"
Guck mal hier:

PHP

$image = "image-".$id

Was fehlt denn da?
(keine Angst - passiert mir auch immer noch :p)

Ich dachte Du wolltest erst mal nen Login.

DarkSouL1221 · 8. September 2014

Ach jaaaa... Das gute alte Semikolon
Habe das Script noch einmal kurz überarbeitet:

PHP

<?php
    $id = 0;
    $endung = ".jpg";
    $image = "image-".$id.$endung;
    move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name']);
    move_uploaded_file($_FILES['gbild']['tmp_name'], 'gallerie/gr/'.$_FILES['gbild']['name']);
    if (move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name'])){
        $id=$id+1;
        rename ('gallerie/'.$_FILES['vbild']['name'], "gallerie/".$image);
        rename ('gallerie/'.$_FILES['bbild']['name'], "gallerie/gr/".$image);
    }
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="de" lang="de">
 <head>
  <title>Geschützter Bereich</title>
 </head>
 <body>
  <form action="upload.php" method="post" enctype="multipart/form-data">
   Vorschaubild: <input name="vbild" type="file">
   <br /><br />
   richtiges Bild: <input name="gbild" type="file">
   <br>
   <input type="submit" value="Upload" />
  </form>
 </body>
</html>

Alles anzeigen

Er lädt jetzt die Bilder hoch aber benennt sie jedoch nicht um. woran liegt das?
Achso und zwecks des Logins habe ich mich mal im Inet schlau gemacht und ein kleines einfaches Script gefunden was für mich reicht:

PHP

<?php
     if ($_SERVER['REQUEST_METHOD'] == 'POST') {
      session_start();




      $passwort = $_POST['passwort'];




      $hostname = $_SERVER['HTTP_HOST'];
      $path = dirname($_SERVER['PHP_SELF']);




      // Benutzername und Passwort werden überprüft
      if ($passwort == 'MeinPW') {
       $_SESSION['angemeldet'] = true;




       // Weiterleitung zur geschützten Startseite
       if ($_SERVER['SERVER_PROTOCOL'] == 'HTTP/1.1') {
        if (php_sapi_name() == 'cgi') {
         header('Status: 303 See Other');
         }
        else {
         header('HTTP/1.1 303 See Other');
         }
        }




       header('Location: http://darksoul.bplaced.net/TestSeite/upload.php');
       exit;
       }
      }
?>
<!DOCTYPE html>
<html>
 <head>
  <title>Geschützter Bereich</title>
 </head>
 <body>
  <form action="login.php" method="post">
   Passwort: <input type="password" name="passwort" /><br />
   <input type="submit" value="Login" />
  </form>
 </body>
</html>

Alles anzeigen

Jedoch öffnet er nach der Eingabe nicht die erwünschte Seite

cottton · 8. September 2014

PHP

$id = 0;
    $endung = ".jpg";
    $image = "image-".$id.$endung;




    // hier verschiebst du vbild -- es liegt also nicht mehr im tmp-verzeichnis
    move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name']);




    // hier verschiebst du gbild -- es liegt also nicht mehr im tmp-verzeichnis
    move_uploaded_file($_FILES['gbild']['tmp_name'], 'gallerie/gr/'.$_FILES['gbild']['name']);




    // hier versuchst du vbild nocheinmal zu verschieben, was nicht mehr functionieren wird
    if (move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name'])){




        // warum id +1 ? du nutzt doch die id hier garnicht mehr        
        $id=$id+1;




        // das hier brauchst du auch gar nicht:
        rename ('gallerie/'.$_FILES['vbild']['name'], "gallerie/".$image);
        rename ('gallerie/'.$_FILES['bbild']['name'], "gallerie/gr/".$image);
    }

Alles anzeigen

hier mal ein Vorschlag:

PHP

// EDIT: ganz wichtig- ist adm eingeloggt?
    session_start();
    if(!isset($_SESSION['angemeldet']) or $_SESSION['angemeldet'] !== true){
        die; // stop
    }
    // erstmal prüfen, ob etwas ge-uploaded wurde:
    if(isset($_POST[...dein submit button ...])){
        // gab es fehler?
        if(isset($_FILES['error'])){
            // ...
        }else{
            // wurde vbild hochgeladen?
            if(isset($_FILES['vbild'])){
                // der name der datei (wie sie auf dem pc vom client benannt wurde)
                $name = $_FILES['vbild']['name'];




                // der tmp-name
                // (bei einem upload wird die datei in einem tmp verzeichnis abgelegt
                // wird die datei nicht verschoben, dann wird sie nach der sitzung gelöscht)
                $tmp_name = $_FILES['vbild']['tmp_name'];




                // der typ der hochgeladenen datei
                $type = $_FILES['vbild']['type'];




                // die größe der hochgeladenen datei
                $size = $_FILES['vbild']['size'];




                // jetzt sollte man prüfen, ob alles io ist
                // also stimmt die größe, der typ, versucht jemand schaden anzurichten?
                // dazu aber später...




                // verschiebe file:
                if(move_uploaded_file($_FILES['vbild']['tmp_name'], 'gallerie/'.$_FILES['vbild']['name'])){
                    // hat geklappt
                }else{
                    // fehler ...
                }
            }







            // wurde gbild hochgeladen?
            if(isset($_FILES['gbild'])){
                // hier das ganze dann nochmal für gbild ...
            }
        }
    }

Alles anzeigen

mach dich damit erstmal vertraut. Da kommt später noch einiges zwecks Sicherheit dazu.

Login später. Bin grad zu müde und alles auf einmal wird unübersichtlich =)

DarkSouL1221 · 8. September 2014

Danke erstmal für deine Bemühungen!

zum 1. Quelltext:
Ich wollte halt Ordnung in die Sache bringen und die Bilder halt ordentlich durchnummerieren. Sprich image-1/image-2/image-3/...

PHP

//deshalb das hier damit das ordentlich nummeriert wird
$id=$id+1;




//und hier sollten die Bilder dann halt umbenannt werden in image-1...
rename ('gallerie/'.$_FILES['vbild']['name'], "gallerie/".$image);

Zum 2. Quelltext:
Wie oben schon gesagt... Hier fehlt mir halt die Ordnung der Namen.
Also ich habe dein Script jetzt so verstanden das er die Bilder auf dem Server jetzt genauso nennt wie sie auch auf dem PC heißen von dem aus sie hochgeladen werden.

Wozu die ganze ordnung mit den Namen? Ich wollte dann halt noch ein Script einfügen das er die Namen automatisch in das Array imagedata.php einfügt.
Also dieses hier meine ich:

PHP

$array = [
        [
            'src'   => 'gallerie/image-1.jpg',
            'href'  => 'gallerie/gr/image-1.jpg',
            'title' => 'Eine Beschreibung die angezeigt wird wenn das bild nicht läd oder man mit der Maus drauf stehen bleibt',
        ],
        [
            'src'   => 'gallerie/image-2.jpg',
            'href'  => 'gallerie/gr/image-2.jpg',
            'title' => 'Eine Beschreibung die angezeigt wird wenn das bild nicht läd oder man mit der Maus drauf stehen bleibt',
        ],
        [
            'src'   => 'gallerie/image-3.jpg',
            'href'  => 'gallerie/gr/image-3.jpg',
            'title' => 'Eine Beschreibung die angezeigt wird wenn das bild nicht läd oder man mit der Maus drauf stehen bleibt',
        ],
    ];#$array
?>

Alles anzeigen

cottton · 8. September 2014

das Verzeichnis kannst Du dann einlesen.
hier mal ein Bsp:

PHP

$path = ''; // der zu durchsuchende pfad
$extension = '.png'; // die gesuchte endung




foreach(glob($path."*".$extension) as $file){
    echo '<br>'.$file;
// hier könntest du das array befüllen
}

glob() siehe http://php.net/manual/de/function.glob.php

es wird nach allen Files gesucht, die im Pfad $path liegen, die Endung $extension haben, und * heißen (also alle).

es gibt auch noch scandir(), aber ich denke glob() passt hier besser.

DarkSouL1221 · 9. September 2014

Okay danke aber das kann ich ja später noch machen.
Ich habe halt immer noch das Problem mit dem hochladen und umbenennen.
Ich habe dein Script auch mal kopiert eingefügt und da hat er das Bild gar nicht erst hochgeladen.

wolf · 9. September 2014

Ne das machst du anders....
Das array kannst du dir auch sparen, wenn du eh immer alles aus dem gleichen Ordner einliest... Dann Bekommst du halt Daten wie die beschreibung nicht...

Du kannst einfach statt das array einzulesen alle Dateien aus dem ordner einlesen ( scandir() ) und diese dann verarbeiten. Die Pfade kannst du ja STatisch eingeben.

so etwa

PHP

<?php
    //alle Fehler anzeigen mit »E_ALL« -> mit »0« (NULL) werden keine fehler angezeigt
    error_reporting(E_ALL);
    //eine Datei Importieren
    #include 'imagedata.php';




    #$bilderliste = $array; //Das array aus »imagedata.php« in bilderliste schreiben




    //die Pfase angeben
    $kleinerPfad = 'pfad/';
    $groserPfad = 'pfad/gr/';







    $bilderliste = scandir($kleinerPfad);




    echo '<div class="img-gallerie" >';
    // für jedes bild in $bilderliste:
    foreach($bilderliste as $id => $name){
        // htmlspecialchars siehe: http://php.net/manual/de/function.htmlspecialchars.php
        $src = htmlspecialchars($name);




        //Prüfen ob der name zu kurz ist (z.B. »./« oder »../« oder »gr/« )
        if ( strlen($name) <= 5) {
        	//dann zum nächsten foreach springen
        	exit(0);
        }
        //ich habe hier die Daten so hässlich aufgedröselt, das man es besser lesen kann was für Daten übergeben werden
        //beim a-tag wird das zielscript übergeben (falls JS ausgeschaltet ist) mit folgendem link: ScriptName.php?img=$ID (es wird einfach die ID noch übergeben)
        echo '
            <div class="img-container">
                <a
                    class = "img-link"
                    href  = "'.$groserPfad.$name'"
                    rel   = "lightbox"
                    title = "'.$name.'"
                >
                    <img
                        class ="img-img"
                        src   = "'.$kleinerPfad.$name'"
                        alt   = "'.$name'"
                        data-img-id="'.$id.'"
                    />
                </a>
            </div>
        ';
    }#foreach
    echo '</div>';
?>

Alles anzeigen

DarkSouL1221 · 9. September 2014

Okay danke dann wäre das schonmal geklärt:)

Jedoch habe ich immer noch das Problem mit dem Hochladen

wolf · 9. September 2014

Welches denn genau?

DarkSouL1221 · 9. September 2014

Na ich habe das Script von Cotton mal benutzt und versucht aber es wurden dann halt keine Bilder hochgeladen.

cottton · 9. September 2014

hast Du es denn angepasst?

DarkSouL1221 · 9. September 2014

Na ich musste ja eigentlich nur noch einmal das move_uploaded_file eintragen oder?

cottton · 9. September 2014

Du hast in Deinem Upload script keinen name="" für den Submitbutton.
Bei mir funktionierts.
Muss natürlich auch dei SESSION existieren.

DarkSouL1221 · 9. September 2014

aaach stimmt! das habe ich ganz übersehen.
ich versuche das gleich mal

wolf · 9. September 2014

Wenn du es wirklich genau haben willst wie man es komlett macht (mit bilder kleinrendern ....)
https://github.com/wolf-w/past…ob/master/IMAGEUPLOAD.php

DarkSouL1221 · 9. September 2014

Danke für die vielen Lösungsvorschläge aber ich will das alles so einfach wie möglich halten.
Das hochladen funktioniert jetzt nur das automatische umbenennen in image-1, image-2... noch nicht

PHP

$id=0;
    $endung=".jpg";
    $uploaddirv="gallerie/";




    if(isset($_POST['Upload'])){
        if(isset($_FILES['error'])){




        }else{




            if(isset($_FILES['vbild'])){
                $id=$id++;
                $image="image-".$id.$endung;
                /*$name = $_FILES['vbild']['name'];
                $tmp_name = $_FILES['vbild']['tmp_name'];
                $type = $_FILES['vbild']['type'];
                $size = $_FILES['vbild']['size'];*/




                if(move_uploaded_file($_FILES['vbild']['tmp_name'], "gallerie/".$image)){
                    // hat geklappt
                }else{
                    // fehler ...
                }
            }

Alles anzeigen

Das erste Bild nennt er zwar image-1 aber wenn ich danach gleich ein zweites hochlade nennt er es auch image-1 und überschreibt dann das erste

lauras · 9. September 2014

PHP

$id=0;

Du fängst ja auch immer bei 0 an..
Zähle die Elemente im Ordner und fang bei der Anzahl an.

DarkSouL1221 · 9. September 2014

Achso ich hatte mir es so gedacht das es halt bei 0 anfängt und sobald ein Bild hochgeladen wurde

PHP

$id=$id++;

die Zahl dadurch um 1 erhöt wird und $id jetzt 1 ist und dann halt immer so weiter.

Aber habe es jetzt durch Internetrecherche anders gelöst

PHP

$dirpic = "gallerie/gr/";
  $handle = opendir($dirpic);




  while ($filepic = readdir($handle))
  {
  if($filepic != "." AND $filepic != "..")
        {
            $id++;
        }
  }
  closedir($handle);

Alles anzeigen

cottton · 9. September 2014

Erweitere da mal um das hier:

PHP

if($filepic != "." AND $filepic != ".." AND !is_link($filepic) AND !is_dir($filepic))

btw:

PHP

$id=$id++;

- ist ein bissl witzig
Warum:

PHP

$id=$id++;
// weise $id (dem linken) den Wert von $id (dem rechten) zu, und zähle dann $id (das rechte) um eins hoch =)
// es hätte also $id++ gereicht

EDIT:
ha, hab selbst was gelernt:

PHP

$id=0;
$id=$id++;
echo $id; // ergibt 0

"erweiterte" Galerie

Jetzt mitmachen!

Teilen