Download via header()

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Download via header()

    HiHo,

    habe da "mal wieder" ein Problem mit einem Code

    PHP-Quellcode

    1. $filename = $row->filename;
    2. // header("Content-Disposition: attachment; filename=\"$filename\""); // Dateiname
    3. // header("Content-Length: ".filesize($filename)); // Dateigröße
    4. // header("Content-Type: application/x-gzip"); // MIME-Typ
    6. header('Content-Description: File Transfer');
    7. header('Content-Type: application/x-gzip');
    8. header('Content-Disposition: attachment; filename="'.basename($filename).'"'); //<<< Note the " " surrounding the file name
    9. header('Content-Transfer-Encoding: binary');
    10. header('Connection: Keep-Alive');
    11. header('Expires: 0');
    12. header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
    13. header('Pragma: public');
    14. header('Content-Length: ' . filesize('files/'.$filename));
    15. readfile('files/'.$filename); // Ausgabe
    Alles anzeigen


    die Ausgangsdatei liegt ist files. Der Name der Datei steckt in $filename (tar.gz File)
    Er läd die Datei herunter, hat den Richtigen Namen und die richtige Größe aber das herunter geladenen Archiv ist Defekt.
    Wo steckt der Fehler in meinem Code?

    Edit: Easy

  • Das Archiv wird ja nicht erstellt. Das ist bereits vorhanden und liegt auf dem Server.
    Es soll beim Klick lediglich nur herunter geladen werden.

    Der Link download.php?page=download&dl_id=26 greift dann bei Zeile 93

    PHP-Quellcode

    1. class Download {
    3. /**
    4. * Download Link
    5. *
    6. * @access public
    7. * @return int
    8. * @param $id ID
    9. *
    10. */
    11. public function dl_link($id) {
    12. global $Core, $Smarty, $lang;
    14. /*
    15. * Filename auslesen
    16. */
    17. $pdoparams = array(
    18. 'id' => (int) $id
    19. );
    20. $sql = 'SELECT * FROM ' . DOWNLOAD . ' WHERE id = :id ';
    21. $stmt = MyDB::getInstance()->prepare($sql);
    22. $stmt->execute($pdoparams);
    23. $row = $stmt->fetch(PDO::FETCH_OBJ);
    25. $filename = $row->filename;
    26. // header("Content-Disposition: attachment; filename=\"$filename\""); // Dateiname
    27. // header("Content-Length: ".filesize($filename)); // Dateigröße
    28. // header("Content-Type: application/x-gzip"); // MIME-Typ
    30. header('Content-Description: File Transfer');
    31. header('Content-Type: application/x-gzip');
    32. header('Content-Disposition: attachment; filename="'.basename($filename).'"'); //<<< Note the " " surrounding the file name
    33. header('Content-Transfer-Encoding: binary');
    34. header('Connection: Keep-Alive');
    35. header('Expires: 0');
    36. header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
    37. header('Pragma: public');
    38. header('Content-Length: ' . filesize('files/'.$filename));
    39. readfile('files/'.$filename); // Ausgabe
    41. }
    43. /**
    44. * download
    45. *
    46. * @access public
    47. * @return String
    48. *
    49. */
    50. public function download() {
    52. global $Core, $Smarty, $lang;
    54. $download = array();
    55. /*
    56. * Kategorie auslesen
    57. */
    58. $pdoparams = array(
    59. ':cat_id' => (int) 9
    60. );
    61. $sql = 'SELECT * FROM ' . DOWNLOAD_CAT . ' WHERE cat_id = :cat_id ';
    62. $stmt = MyDB::getInstance()->prepare($sql);
    63. $stmt->execute($pdoparams);
    64. $cat_row = $stmt->fetch(PDO::FETCH_OBJ);
    66. /*
    67. * Downloads auslesen
    68. */
    69. $sql = 'SELECT * FROM ' . DOWNLOAD . ' WHERE cat_id = :cat_id ';
    70. $stmt = MyDB::getInstance()->prepare($sql);
    71. $stmt->execute($pdoparams);
    72. while ($row = $stmt->fetch()) {
    73. $row['created_time'] = $Core->userdate($row['created_time']);
    74. $row['description'] = nl2br($row['description']);
    75. /*
    76. * Anhänge?
    77. */
    78. if ( $row['thumbnails'] ) {
    79. $attChunks = explode(";", $row['thumbnails']);
    80. for($i = 0; $i < count($attChunks); $i++){
    81. $row['att'] .= ' <img src="'.PATH.'images/'.$attChunks[$i].'" weight="100" height="100" />';
    82. }
    83. }
    85. $download[] = $row;
    86. }
    88. // Ausgabe Header
    89. $Core->page_header($cat_row->cat_name);
    91. // Link angeklickt?
    92. if ($Core->get(GET, 'dl_id', true)) {
    93. $this->dl_link($Core->get(GET, 'dl_id'));
    94. }
    96. $Smarty->assign(array(
    97. 'DOWNLOAD' => $download,
    98. 'CAT_NAME' => $cat_row->cat_name,
    99. 'CAT_DESC' => $cat_row->cat_desc,
    100. ));
    102. $Smarty->display('download.htm');
    104. }
    106. }
    Alles anzeigen

  • Moin,

    das Problem scheint bei Zeile 89 zu sein

    PHP-Quellcode

    1. // Ausgabe Header
    2. $Core->page_header($cat_row->cat_name);


    Setzte ich das unter die if Abfrage in Zeile 95, so ist das herunter geladene Archiv nicht mehr Defekt und lässt sich auch Entpacken.Ist ja auch klar. page_header() hat folgenden Inhalt

    PHP-Quellcode

    1. /**
    2. * Ausgabe des Header
    3. *
    4. * @param $page_title String Titel der Aufgerufenen Seite
    5. * @return mixed
    6. * @access public
    7. */
    8. public function page_header($page_title) {
    9. global $Smarty, $lang;
    11. $Smarty->assign(array(
    12. 'PAGE_TITLE' => $page_title . ' &bull; ' . self::get_config('page_title'),
    13. 'PAGE_DESC' => self::get_config('page_desc'),
    14. 'LANG' => $lang,
    15. ));
    17. $Smarty->display('page_header.htm');
    18. }
    Alles anzeigen


    Ganz klar Anfängerfehler :whistling: vor header() darf keine Ausgabe erfolgen

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von Grommel ()