FuelPHPのUploadクラスの不具合に遭遇したので記録しておきます。

Uploadクラスの仕様変更

あまり、はまる人は少ないかも知れませんが、FuelPHPのUploadクラスのアップロードファイルの情報が格納される配列の仕様が、UploadクラスがFuelPHP 2.0のパッケージを使うように変更された1.6から変わっていたようです。

ただし、Upload::get_files()、Upload::get_errors()では、以下のように変換されて配列が返るため、互換性は維持されているようです。

    foreach ($files as $file)
    {
        $data = array();
        foreach ($file as $item => $value)
        {
            $item == 'element' and $item = 'field';
            $item == 'tmp_name' and $item = 'file';
            $item == 'filename' and $item = 'saved_as';
            $item == 'path' and $item = 'saved_to';
            $data[$item] = $value;
        }
        $data['field'] = str_replace('.', ':', $data['field']);
        $data['error'] = ! $file->isValid();
        $data['errors'] = array();
        $result[] = $data;
    }

つまり、Upload::register()などで、この配列を直接操作する場合は、仕様変更に引っかかります。以下のようにキーが変更されています。

field    → element
file     → tmp_name
filename → basename
saved_as → filename
saved_to → path

また、errorの値はboolからPHPのUpload error codeに変わっています。

Uploadクラスのバグ情報

FuelPHP 1.6.1のZipファイルでは、ComposerでインストールするUploadパッケージの名前空間が、1.6.1リリース後にFuelPHPからFuelに変更されたことが反映されておらず、以下のエラーが発生します。

Fatal error: Class 'FuelPHP\Upload\Upload' not found in .../fuel/core/classes/upload.php on line 90

これは、oil installコマンドでインストールされるGitHubの1.6/masterでは修正済みです。

もう1つ、Upload::register()で'validate'が使えないバグがありました。

InvalidArgumentException [ Error ]: after_validate is not a valid event
.../fuel/vendor/fuelphp/upload/src/FuelPHP/Upload/Upload.php @ line 320

これは、1.7/developで修正されています。

ただし、その他のバグ修正もされているので、Uploadクラスは1.7/developの最新のものを使うのがよいように思います。

• https://github.com/fuel/core/commits/1.7/develop/classes/upload.php