PHPでディレクトリをZIPファイル化したい

Question

###実現したいこと
PHPでリンクをクリックすると自動でダウンロードが始まる機能を実装しました。
ローカル環境では動作しますが、グローバルサーバにアップロードするとダウンロードリンクがうまく機能せず、ページが遷移してしまいます。
プログラムの改善点やアドバイスをいただければ幸いです。

###追記
Ziparchiveを導入したところページは動作しましたが、できていたZIPファイルの出力ができなくなってしまいました。
・ディレクトリ
download.php
csv（フォルダ）
image（フォルダ）
の位置関係です。

###環境
さくらVPS ubuntu

<?php
namespace My;

/**
 * ZipArchiveを拡張して、ディレクトリまるまる扱う機能を追加
 */
class MyZipArchive extends \ZipArchive
{
    /**
     * ディレクトリをまるごとzipファイルにします。
     *
     * @param string $dir              ディレクトリパス
     * @param string $inner_path       zipファイル中のディレクトリパス
     * @param bool   $create_empty_dir 空ディレクトリもディレクトリを作成するか
     * @return bool  処理の成否
     */
    public function addDir($dir, $inner_path, $create_empty_dir=false){
        $items = \array_diff(\scandir($dir), ['.','..']);
        $item_count = \count($items);

        if($create_empty_dir || $item_count > 0){
            $this->addEmptyDir($inner_path);
        }

        // 追加するものがないならここで終了する
        if($item_count === 0) return true;


        foreach($items as $_item){ // forで行うなら$itemsは一旦array_values()を通したほうがいい
            $_path = $dir . DIRECTORY_SEPARATOR . $_item;
            $_item_inner_path = $inner_path . DIRECTORY_SEPARATOR . $_item;

            // ディレクトリの場合は再帰的に処理する
            if(\is_dir($_path)){
                $_r = \call_user_func( // "$this->addDir"より保守的に好ましい
                    [$this, __FUNCTION__], $_path, $_item_inner_path);
                if(!$_r) return false;
            }
            // ファイルの場合でかつ処理に失敗したときはこちら
            else if(!$this->addFile($_path, $_item_inner_path)
                  && !$this->on_recursive_error($dir, $inner_path, $create_empty_dir)){
                return false;
            }
            // ファイルの追加成功時は何も他には行わない
        }

        return true;
    }

    /**
     * 再帰的処理のときにエラーが生じたらどうするかを定義しています。
     *
     * @param string $parent_dir              ディレクトリパス
     * @param string $parent_inner_path       zipファイル中のディレクトリパス
     * @param bool   $create_empty_dir        空ディレクトリもディレクトリを作成するか
     * @return bool Falseならその場で中断
     */
    private function on_recursive_error($parent_dir, $parent_inner_path, $create_empty_dir){
        // 自由に定義してください
        return false;
    }
}


/*
 * 下記は、質問本文のコードをちょっと改造してのテスト
 */

//Zipクラスロード
$zip = new MyZipArchive();

//Zipファイル名指定
$zipFileName = '***.zip';

//Zipファイル一時保存ディレクトリ取得
$zipTmpDir = '../';

// Zipファイルのパス
$zip_file = $zipTmpDir . DIRECTORY_SEPARATOR . $zipFileName;

//Zipファイルオープン
$result = $zip->open($zip_file, \ZIPARCHIVE::CREATE | \ZIPARCHIVE::OVERWRITE);

if ($result !== true) {
    return false;
}

//処理制限時間を外す
set_time_limit(0);

// ※今回の肝
$zip->addDir('csv', 'csv', true);
$zip->addDir('images', 'images', true);

$zip->close();


// ストリームに出力
header('Content-Type: application/zip; name="' . $zipFileName . '"');
header('Content-Disposition: attachment; filename="' . $zipFileName . '"');
header('Content-Length: '.filesize($zip_file));
echo file_get_contents($zip_file);

// 一時ファイルを削除しておく
unlink($zip_file);
exit();
?>

参考

https://ja.stackoverflow.com/questions/14416/複数ファイルや-フォルダも含むディレクトリをまるごとzipで圧縮して-ダウンロードさせたい

Comments

[yoorwm]

「※今回の肝」の4行下に>がありますが、コピペミスでしょうか？

[Zu_Chan]

コピペミスです。修正します。

Answer 1

自己解決しました。
サーバサイドの権限が原因でした。