バグ報告 > 管理機能 > 配送業者登録ページで入力チェックエラー関連のバグ(2つ) |
管理機能
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
y_shiki |
投稿日時: 2010/3/9 19:17
対応状況: −−−
|
半人前 登録日: 2010/3/2 居住地: 山形 投稿: 25 |
配送業者登録ページで入力チェックエラー関連のバグ(2つ) 送料計算方法のカスタマイズをしていて見つけました。
考慮不足等あれば、ご指摘いただければと思います。 ◆その1 配送業者登録ページで入力チェックエラーがあった場合、$this->tpl_deliv_id への値セットがないです。 そのため、入力チェックエラー後の登録が新規扱いになります。 LC_Page_Admin_Basis_Delivery_Input::process メソッド内にて $this->tpl_deliv_id への値セットを追加すれば対処できます(83行目近辺)。 switch($_POST['mode']) { case 'edit': // 入力値の変換 $this->objFormParam->convParam(); $this->arrErr = $this->lfCheckError(); /* 追加--> */ $this->tpl_deliv_id = $_POST['deliv_id']; /* <--追加 */ if(count($this->arrErr) == 0) { $this->tpl_deliv_id = $this->lfRegistData(); $this->tpl_onload = "window.alert('配送業者設定が完了しました。');"; } break; ◆その2 名称のチェックですが新規登録の時しかチェック処理を通らないため、登録済みデータの名称を変更した場合に、同じ名称のデータが作れてしまいます。 LC_Page_Admin_Basis_Delivery_Input::lfCheckError メソッド これは…どうなんでしょう? チェックするなら更新時も、と思います。 自分のとこのカスタマイズでは、同一名称も登録させたいので名称のチェックを削除しました。 --- [EC-CUBE] 2.4.3 [レンタルサーバ] at-link [OS] RHEL5 [PHP] 5.1.6 [データベース] PostgreSQL 8.1.18 [WEBサーバ] apache2.2.3 --- |
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |