Moodle開発者フォーラム

見落としがありました．

　同じコースファイルでも，モジュール（フォルダのリンク）で指示されているフォルダの内容は，コースファイルのバックアップの前にバックアップされるのですね．こちらも対策しないといけませんでした．

　ちょっと気になったのは，backup/lib.phpの中のfunction backup_copy_file()がコースファイルその他をガバッとコピーするリカーシブな関数（対象がフォルダの場合は自分自身を呼ぶことで再帰的にフォルダ構造ごとコピーする）です．でも，いま気付いたように，先にmod/resource/backuplib.phpでリソースの指し示す先にあるフォルダなりファイルなりをコピーした後に，改めて（コースファイルのバックアップを指示されているならば）コピーします．これ，かなり無駄では？

　backup/lib.phpのfunction backup_copy_file()のコピーを行う箇所，まずファイルの有無をチェックして，存在しないならばコピーすれば良いのじゃ無いでしょうか．

    function backup_copy_file ($from_file,$to_file,$log_clam=false) {

        global $CFG;

        if (is_file($from_file)) {
            //echo "<br />Copying ".$from_file." to ".$to_file;              //Debug
            //$perms=fileperms($from_file);
            //return copy($from_file,$to_file) && chmod($to_file,$perms);
            umask(0000);
            if (file_exists($to_file) or copy($from_file,$to_file)) {
                chmod($to_file,$CFG->directorypermissions);
                if (!empty($log_clam)) {
                    clam_log_upload($to_file,null,true);
                }
                return true;
            }
            return false;
        }

赤い箇所を追加しても，大して危険は無いと思うのですけれども．どうでしょうか．