バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

バグ報告 > その他 > ソースコード閲覧時に見つけた変数名不正

その他

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
akitake
投稿日時: 2013/3/1 10:23
対応状況: −−−
新米
登録日: 2013/3/1
居住地:
投稿: 1
ソースコード閲覧時に見つけた変数名不正
実際に動作確認をして発見したわけではないのですが、
手動でバージョンアップをするためにソースを確認している際に
少し気になる箇所を発見したので報告させて頂きます。

data/class/pages/entry/LC_Page_Entry.php 315行目〜
data/class/pages/mypage/LC_Page_Mypage_Change.php 206行目〜

"lfCheckError"メソッド内の"親ウィンドウの戻り値を格納するinputタグのnameのエラーチェック"部分で、
lfInputNameCheckに与える引数の変数名が
$addData['zip01'] (arrではなくadd)
$arrdata['zip02'] (dataのdが小文字)
となっています。

ソースを見る限り、入力形式不正チェックが上手く動かないのではないかと思うのですが…
shutta
投稿日時: 2013/3/1 11:37
対応状況: −−−
仙人
登録日: 2010/2/4
居住地: 関西
投稿: 835
Re: ソースコード閲覧時に見つけた変数名不正
akitakeさん、ご指摘ありがとうございます。

確かに該当部分のチェックは動作していないですね。
ただ、そもそも


        $objFormParam->addParam('郵便番号1', 'zip01', ZIP01_LEN, 'n', array('EXIST_CHECK', 'NUM_COUNT_CHECK', 'NUM_CHECK'));
        $objFormParam->addParam('郵便番号2', 'zip02', ZIP02_LEN, 'n', array('EXIST_CHECK', 'NUM_COUNT_CHECK', 'NUM_CHECK'));

と、zip01,zip02には、NUM_CHECKが掛かっているので、該当のチェック部分自体が不要ですね。
なので、動作していなくても全く不都合がなかった感じですね。

http://svn.ec-cube.net/open_trac/changeset/22591
にて、不要なチェック処理をばっさり削除しておきました。

有難うございました。


----------------

リゾート会員権(エクシブ等)の売買・仲介

スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


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

統計情報

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

投稿数ランキング

1
seasoft
7365
2
468
3217
3
AMUAMU
2712
4
nanasess
2303
5
umebius
2085
6
yuh
1818
7
h_tanaka
1610
8
red
1568
9
mcontact
1240
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
796
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.