チケット #371 (closed バグ指摘: 修正済)
支払方法設定で、Warning: array_merge() のエラー
| 報告者: | homan | 担当者: | nakanishi |
|---|---|---|---|
| 優先度: | 中 | マイルストーン: | EC-CUBE2.3.4 |
| コンポーネント: | 管理画面 | バージョン: | 2.3.0 |
| キーワード: | 関係者: | ||
| 修正済み: |
説明
PHP5の環境でなる模様(僕もPHP5ですが、今のところ確認できていません)。
array_merge()の仕様がPHP4とPHP5とで変更されているようで、 必ず型データを渡さないとエラー(警告?)が出るようです。
参照: http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=flat&topic_id=2917&forum=9
上記の参照スレッドでの問題は、$this->arrErrが未定義の為に起こるみたいですが、 ファイル存在チェックの前に
$this->arrErr = $this->lfCheckError(); // ファイル存在チェック $this->arrErr = array_merge($this->arrErr, $this->objUpFile->checkEXISTS($_POST['image_key']));
をするか、それとも
// ファイル存在チェック $this->arrErr = array_merge($this->lfCheckError(), $this->objUpFile->checkEXISTS($_POST['image_key']));
とか、型キャストを利用して
// ファイル存在チェック $this->arrErr = array_merge((array)$this->arrErr, $this->objUpFile->checkEXISTS($_POST['image_key']));
とするのか・・・どれがいいのか、どれもよくないのか 判断ができないのでご査収頂けると幸いです。 エラー内容をマージしてるので、 3番目の型キャストはあんまり意味がない?でしょうかね。
チケットの履歴
Note: チケットについてのヘルプは
TracTickets を参照
して下さい。
