[Erledigt] Downloadscript liest nur Quelltext ein

[PiXeL1337]

Hallo,

Ich möchte Datein zum downloaden anbieten. Das Downloadfenster kommt auch aber wenn ich die Datei dann downloade ist nur der Qeulltext der Webseite in der Datei und nicht der eigentliche Inhalt.

public function makeDownload($file, $dir)
    {
        $dir = str_replace("@", "/", $dir);
        $dir .= "/";
        if(file_exists($dir.$file) && strstr($dir, "downloads"))
        {
            $filesize = filesize($dir.$file);
            $type = $this->getType($file);
            header("Content-Type: $type");
            header("Content-Length: $filesize");
            //header("Content-type: application/octet-stream");  
            header("Content-Disposition: attachment; filename=\"$file\"");
            readfile($dir.$file);

        }        
    } 

Der Pfad zur Datei stimmt auch.

Ich hoffe mir kann hier einer helfen

[eagle275]

schau mal ans ende der erzeugten Datei - ich hatte zuerst meine Fehlermeldung auch dort stehen . du verbiegst ja den Header ....

[PiXeL1337]

Ich hatte keine Fehlermeldungen. Das error_reporting steht auf E_ALL

[ChrisB]

Das hätte mit einem "headers already sent" quittiert werden müssen...

Nicht, wenn output_buffering aktiv ist.

[Wolla]

Den beschriebenen Effekt hatte ich neulich bei PHPmyadmin. Statt der Daten hatte ich HTML-Code des PMA in der runtergeladenen CSV-Datei.
Bei einem zweiten Versuch hat es dann normal geklappt, und der Vorfall war auch von Kollegen nicht zu reproduzieren.

[lstegelitz]

Ich hab den Fehler gefunden. Es lag daran das ich vorher doch schon ein include gemacht habe in dem HTML ausgegeben wurde.

Das hätte mit einem "headers already sent" quittiert werden müssen...
sieht so aus, als ob du das error_reporting höher schrauben musst (oder stand die Meldung in der heruntergeladenen Datei?).

[PiXeL1337]

Erstmal vielen Dank für die Hilfe!

Ich hab den Fehler gefunden. Es lag daran das ich vorher doch schon ein include gemacht habe in dem HTML ausgegeben wurde.

Einen erfolgreichen Montag noch !

[eagle275]

nun das Problem ist ziemlich offensichtlich ....


nachdem du den Header verstellt hast, versucht dein script auch die Normalen Ausgaben in die Datei weiterzuleiten, die zum user downgeloaded wird ....

daher hab ich ja als letzte Anweisung in der Funktion das die(); , damit an dieser Stelle mein Script beendet wird - und beim nächsten Bearbeitungsschritt wieder der richtige Header ausgegeben wird (der dann wieder für HTML im Browser sorgt) - das heißt deine eigentlich gewünschte Ausgabe ist "" und daran hängt das script seinen COde an ... was ja nicht so erbaulich ist ....

mithin ist also IMMER noch der PFad in Readfile falsch ....

und nimm das die() als letzte Anweisung in deine make_download-funktion auf

[PiXeL1337]

Da hat sich noch nicht viel geändert^^

public function makeDownload($file, $dir)
    {
        $dir = str_replace("@", "/", $dir);
        if (substr($dir,strlen($dir)-1) != '/')
        {
            $dir .= "/";
        }
        
        if (file_exists($dir.$file))
         {
            if(file_exists($dir.$file) && strstr($dir, "downloads"))
            {
                echo $dir.$file;
                $filesize = filesize($dir.$file);
                $type = $this->getType($file);
                //header("Content-Type: $type");
                header("Content-Length: $filesize");        
                header("Content-Type: application/force-download");
                header("Content-Type: application/octet-stream");
                header("Content-Type: application/download");   
                header("Content-Disposition: attachment; filename=\"$file\"");
                readfile($dir.$file);
    
            }
        }        
    } 

Mir ist aufgefallen das wenn ich readfile auskommentiere der Download (mit falschem inahlt) trozdem kommt.

[eagle275]

wie sieht denn mittlerweile dein Code aus - oder klappt es inzwischen "mit'm Nachbar'n"

[PiXeL1337]

Ja geht er und da liegt auch die Datei.

Hab den 2ten slash gerade mal entfernt.

[eagle275]

und geht er in den Then-Zweig ? ... liegt dort tatsächlich auch die Datei ?

der Doppelslash dort stört ..... bearbeite mal deine string-anweisungen für $dir ...

$dir .='/' ; <-- das nur ausführen, wenn nicht schon ein slash dran hängt ...

also

if (substr($dir,strlen($dir)-1)!='/')
   $dir.='/'; 

oder so ähnlich

[PiXeL1337]

ja deine lösung habe ich probiert ( #3 ).

in $dir.$file steht sowas wie: ./downloads/ordner1//test1.txt

[eagle275]

ja dann passt der Pfad nicht .. hast du es mal mit meiner Lösung über fopen ( ) .. versucht ?

und lass dir vorher den $dir.$file nochmal ausgeben ..

UND PRÜF BITTE , ob dein CODE auch in den THEN-Zweig deiner If Prüfung reingeht (Echo "Hier"; direkt nach if .. { reicht ja )

[PiXeL1337]

Ich will ja NICHT den Quelltext anbieten sondern text, doc oder pdf.

Das problem ist das in den Datein nicht der eigentliche inhalt steht sondern das HTML der Seite.