バージョン選択

フォーラム

メニュー

オンライン状況

56 人のユーザが現在オンラインです。 (46 人のユーザが フォーラム を参照しています。)
登録ユーザ: 0
ゲスト: 56
もっと...

サイト内検索

バグ報告 > 管理機能 > 配送業者登録ページで入力チェックエラー関連のバグ(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
---
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBE公式 Amazon Payプラグイン

統計情報

総メンバー数は88,710名です
総投稿数は109,948件です

投稿数ランキング

1
seasoft
7367
2
468
3217
3
AMUAMU
2712
4
nanasess
2313
5
umebius
2085
6
yuh
1819
7
h_tanaka
1638
8
red
1570
9
mcontact
1285
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
799
14 ramrun 789
15 karin 689
16 sumida 641
17
homan
633
18 DELIGHT 572
19
patapata
502
20
flealog
485


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.