Nach einer Weiterleitung eine Meldung ausgeben

MarvinTLA · 3. November 2021

Moin,

ich habe bei mir die Funktion das man Dateien hochladen kann. Dies klappt soweit auch sehr gut.

Mein Problem ist allerdings das ich das ganze ja mit POST mache.

Heißt also wenn man das ganze absendet und die Seite neu laden möchte, die Abfrage kommt ob man die "Erneute Formular-Übermittlung bestätigen" möchte.

Das ganze habe ich mit der headerFunktion umgangen.

Ich habe auch eigentlich eine Bestätigung gemacht, die man bekommt, wenn die Datei hochgeladen wurde. Diese wird allerdings nicht mehr angezeigt, aufgrund der Weiterleitung.

Weiß jemand wie ich das ändern kann?

Mein Code:

Code

if(move_uploaded_file($dateiTempName, $absolutePath)) {
    $erfolgreichHochgeladen = 'Die Datei wurde erfolgreich hochgeladen.';            
    if($erfolgreichHochgeladen) {
        header("LOCATION: dashboard.php");
    }
}

Hier wird das ganze ausgegeben im Code später:

Code

if(isset($erfolgreichHochgeladen)) {
    echo $erfolgreichHochgeladen;
}

Gruß

cottton · 3. November 2021

Session.

Wenn erfolgreich, dann setzt Du z.B. $_SESSION['flash_messages'][] = 'Die Datei wurde ... .';.

Und immer, wenn die Webseite ausgegeben wird,

dann guckst Du einfach in $_SESSION['flash_messages'], ob da was drin ist, das angezeigt werden soll.

Wenn Du die Nachrichten angezeigt hast (oder diese evtl. zu alt sind), dann löscht Du sie aus der Session.

Bsp.

PHP

if (isset($_SESSION['flash_messages']) and is_array($_SESSION['flash_messages'])) {
    foreach ($_SESSION['flash_messages'] as $message) {
        // Hier dein code zum ausgeben der message.
    }
    // All messages were shown - reset|remove them.
    $_SESSION['flash_messages'] = [];
}

---

Wird oft "Flash messages" genannt.

Bsp: https://stackoverflow.com/a/33224494/3411766

---

EDIT: Du kannst, und solltest wohl auch, alle Nachrichten by type sammeln und ausgeben,

um z.B. Fehler "in Rot" darzustellen ect.

Bsp:

PHP

 // Add a message.
$_SESSION['flash_messages']['info'][] = 'Die Datei wurde ... .';
// or
$_SESSION['flash_messages']['error'][] = 'Fehler: ... .';

if (isset($_SESSION['messages']) and is_array($_SESSION['messages'])) {
    foreach ($_SESSION['messages'] as $type => $messages) {
        foreach ($messages as $message) {
            // Hier dein code zum ausgeben der message.
            // $type ist z.B. "error", "info", "notice", "warning", ... .
        }
    }
    // All messages were shown - reset|remove them.
    $_SESSION['messages'] = [];
}

Alles anzeigen

MarvinTLA · 4. November 2021

Also bitte korrigiere mich wenn es Falsch ist.
Habe versucht das ganze etwas zu verstehen.

Habe das ganze jetzt so gemacht:

Code

if(move_uploaded_file($dateiTempName, $absolutePath)) {
                    
    $_SESSION['messages']['info'][] = "Wurde hochgeladen!";
    if(isset($_SESSION['messages']) and is_array($_SESSION['messages'])) {
        foreach($_SESSION['messages'] as $type => $messages) {
            foreach($messages as $message) {
                $erfolgreichHochgeladen = $message['info'];
            }
        }
    }
}

Alles anzeigen

Dort wird mir als Fehler ausgegeben:

Fatal error: Uncaught Error: [] operator not supported for strings in /var/www/html/verzeichnis.php:106 Stack trace: #0 {main} thrown in /var/www/html/verzeichnis.php on line 106

Das ist diese Zeile:

$_SESSION['messages']['info'][] = "Wurde hochgeladen!";

//edit

Hab das auch mal so versucht:

Code

$hochgeladen = $_SESSION['erfolgreich'] = "Erfolgreich hochgeladen";
                    if(isset($hochgeladen)) {
                        header("LOCATION: verzeichnis.php?folder=");
                        echo $hochgeladen;
                    }

Verstehe das noch nicht ganz mit dem header, weil der das ganze ja in dem sinne "überschreibt" bzw "verhindert" das das echo ausgegeben wird.

Allerdings brauche ich den header ja auch, damit man beim Refresh keine erneute Formular Übermittlung hat.

cottton · 4. November 2021

Ne, Du setzt $_SESSION['messages']['info'][] = "Wurde hochgeladen!";

in dem upload (upload.php ?) script.

Und in dashborad.php nutzt Du die Ausgabe.

Bsp:

Das Upload - Script(upload . php ?)

PHP

// wenn upload erfolgreich -
if (move_uploaded_file($dateiTempName, $absolutePath)) {
    // dann session info message setzen -
    $_SESSION['flash_messages']['info'][] = "Wurde hochgeladen!";
    // umleiten (script dashboard.php wird gestarted)
    header("Location: dashboard.php");
    // und hier stoppen.
    exit(0); // exit(0) ^= kein fehler.
}

dashboard . php

PHP

if (isset($_SESSION['flash_messages']) and is_array($_SESSION['flash_messages'])) {
    // Wir erwarten ein array wie z.B:
    // [
    //     'flash_messages' => [
    //         'info'    => ['message 1', 'message 2', /* ... */],
    //         'warning' => ['message 1', 'message 2', /* ... */],
    //         // ...
    //     ],
    // ]
    foreach ($_SESSION['flash_messages'] as $type => $messages) {
        // $type ist z.B: "error", "info", "notice", "warning", ... .
        // $messages z.B: ['message 1', 'message 2', /* ... */]
        foreach ($messages as $message) {
            // Hier dein code zum ausgeben der message.
            // Z.B.:
            // flash_msg_info, flash_msg_warning, ...
            $cssClass = 'flash_msg_' . htmlspecialchars($type);
            // "[info] Die Datei wurde erfolgreich hochgeladen."
            $message = '[' . htmlspecialchars($type) . '] ' . htmlspecialchars($message);
            $htmlMessage = '<div class="' . $cssClass . '">' . $message . '</div>';
            // Ausgeben, oder sammeln, ... .
            echo $htmlMessage;
        }
    }
    // Alle Nachrichten (wenn es welche gab) wurden ausgegeben -
    // reset session "flash_messages".
    $_SESSION['flash_messages'] = [];
}

Alles anzeigen

MarvinTLA · 4. November 2021

Sorry wenn es zu Unklarheiten kam

die Upload Funktion ist in der Datei Verzeichnis.php

Auf diese Seite möchte ich auch wieder zurück

Also einfach Neuladen damit diese erneute Übermittlung nicht kommt

hab ausversehen Dashboard als Weiterleitung genommen

soll aber auch Verzeichnis.php sein

cottton · 4. November 2021

Ok, versteh ich.

Passt trotzdem noch.

Egal wo - wenn Du eine Message ausgeben willst, dann pack das in die session.

Und die nächste Seite, die dann geladen wird, kann die Messages ausgeben (mit dem Session code).

MarvinTLA · 5. November 2021

Alles klar.
Danke dir.

Hätte allerdings noch ein kleines Problem dazu.
Sobald ich diesen Code verwende im upload Teil. Funktioniert der "multiple" Upload nicht mehr.

Code

<input type="file" id="file" style="border: 0.3px solid gray;" name="file[]" multiple hidden="hidden" />

Code

for($i=0;$i<$countfiles;$i++) {
        
                $dateiName = $_FILES["file"]["name"][$i];
                $dateiTempName = $_FILES["file"]["tmp_name"][$i];
            
                $safe_path = "../../../../media/Files/" . htmlspecialchars(ucfirst($userData["verzeichnis"])) . "$datei/";
                $absolutePath = $safe_path . $dateiName;

                if(file_exists($absolutePath)) {
                    // Später noch hinzufügen
                }
                
                if (move_uploaded_file($dateiTempName, $absolutePath)) {
                    
                    $_SESSION['flash_messages']['Erfolgreich'][] = "Die Dateien wurden erfolgreich hochgeladen!";
                    header("Location: verzeichnis.php?folder=");
                    exit(0);
                } 
            }

Alles anzeigen

Wenn ich aber den move_uploaded_file Teil wieder mit meinem alten austausche:

Code

if(move_uploaded_file($dateiTempName, $absolutePath)){
    $erfolgreichHochgeladen = 'Datei wurde erfolgreich hochgeladen!';
}

funktioniert es wieder ...

m.scatello · 5. November 2021

"Funktioniert nicht" ist keine vernünftige Problembeschreibung.

Richtig debuggen

Man bemerkt, dass ein Skript nicht das tut, was es soll.
Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(-1);
Man verwendet ini_set('display_errors', true); damit die Fehler auch angezeigt werden.
Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde. Wichtig bei MySQL Fehlern (...not a valid MySQL result resource...): mysqli_error() verwenden oder Abfrage ausgeben und zb mit phpmyadmin testen.
Schritt 5 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

MarvinTLA · 5. November 2021

1. Stimmt
2. Habe ich bereits.

3. Auch das habe ich, wird allerdings nichts angezeigt.

4. Habe ich. Liegt an dem Code mit dem exit(0);

5. Habe ich auch gemacht.

6. Ja

7. Okay, habe allerdings nichts direkt gefunden.

8. Sobald ich aus dem Code das exit(0); entferne, funktioniert es wieder. Allerdings wird dann die Nachricht nicht mehr ausgegeben. Habe auch keinen direkten Lösungsanschlag gefunden, auch dort werden weiterhin keine Fehler ausgegeben.

m.scatello · 5. November 2021

Zitat von MarvinTLA

5. Habe ich auch gemacht.

Davon sieht man aber nichts.

cottton · 5. November 2021

Kann ja nicht funktionieren, wenn Du uns Code vorenthältst (ein blödes Wort).

Du bist in einem for-loop.

Da solltest Du nicht `header("Location: ...")` nutzen.

Durch das `exit` war dann natürlich Ende des Loops.

Eine Lösung wäre:

PHP

$failure = 0;
$success = 0;
for($i=0;$i<$countfiles;$i++) {

    $dateiName = $_FILES["file"]["name"][$i];
    $dateiTempName = $_FILES["file"]["tmp_name"][$i];

    $safe_path = "../../../../media/Files/" . htmlspecialchars(ucfirst($userData["verzeichnis"])) . "$datei/";
    $absolutePath = $safe_path . $dateiName;
    if(file_exists($absolutePath)) {
        // Später noch hinzufügen
    }

    if (move_uploaded_file($dateiTempName, $absolutePath)) {
        $success++;
    } else {
        $failure++;
    }
}

if ($success) {
    $_SESSION['flash_messages']['Erfolgreich'][] = "{$success} Dateien wurden erfolgreich hochgeladen!";
}
if ($failure) {
    $_SESSION['flash_messages']['Erfolgreich'][] = "{$failure} Dateien konnten nicht hochgeladen werden!";
}
header("Location: verzeichnis.php?folder=");
exit(0);

Alles anzeigen

MarvinTLA · 5. November 2021

Ja, hat so geklappt.

Danke.

MarvinTLA · 7. November 2021

Zitat von cottton

PHP

$failure = 0;
$success = 0;
for($i=0;$i<$countfiles;$i++) {

    $dateiName = $_FILES["file"]["name"][$i];
    $dateiTempName = $_FILES["file"]["tmp_name"][$i];

    $safe_path = "../../../../media/Files/" . htmlspecialchars(ucfirst($userData["verzeichnis"])) . "$datei/";
    $absolutePath = $safe_path . $dateiName;
    if(file_exists($absolutePath)) {
        // Später noch hinzufügen
    }

    if (move_uploaded_file($dateiTempName, $absolutePath)) {
        $success++;
    } else {
        $failure++;
    }
}

if ($success) {
    $_SESSION['flash_messages']['Erfolgreich'][] = "{$success} Dateien wurden erfolgreich hochgeladen!";
}
if ($failure) {
    $_SESSION['flash_messages']['Erfolgreich'][] = "{$failure} Dateien konnten nicht hochgeladen werden!";
}
header("Location: verzeichnis.php?folder=");
exit(0);

Alles anzeigen

Gibt es auch eine Möglichkeit für $success und $failure jeweils eine eigene Schriftfarbe zu nutzen?
Für Success Grün und für $failure halt rot?

m.scatello · 7. November 2021

Zitat von MarvinTLA

Gibt es auch eine Möglichkeit für $success und $failure jeweils eine eigene Schriftfarbe zu nutzen?

Ja, die gibt es. Allerdings solltest du mal selber nachdenken, denn das ist ja wohl extrem simple und man sollte auch als Anfänger auf die richtige Lösung kommen.

Und

$_SESSION['flash_messages']['Erfolgreich'][] = "{$failure} Dateien konnten nicht hochgeladen werden!";

passt auch nicht wirklich zusammen.

Nach einer Weiterleitung eine Meldung ausgeben

Jetzt mitmachen!

Teilen