バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

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

その他

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
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ペイメント

公式ストアEC-CUBE4系デザインテンプレート続々リリース中

統計情報

総メンバー数は66,569名です
総投稿数は98,355件です

投稿数ランキング

1
seasoft
7332
2
AMUAMU
2712
3
468
2646
4
nanasess
2103
5
umebius
1753
6
yuh
1612
7
red
1424
8
h_tanaka
1044
9
fukap
907
10
tsuji
863
11
shutta
835
12
tao_s
792
13 ramrun 789
14 karin 657
15 sumida 641
16
homan
633
17 DELIGHT 571
18
patapata
502
19
flealog
485
20 tonton 437


ネットショップの壺

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

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