Ticket #2044 (closed 改善提案: 修正済)
無駄な処理を改善する for 2.13.0
Reported by: | Seasoft | Owned by: | Seasoft |
---|---|---|---|
Priority: | 低 | Milestone: | EC-CUBE2.13.0 |
Component: | その他 | Version: | |
Keywords: | Cc: | ||
修正済み: | yes |
Description
Change History
comment:1 Changed 11 years ago by Seasoft
- Owner changed from somebody to Seasoft
- Status changed from new to assigned
comment:2 Changed 11 years ago by shutta
r22591 にて、モバイルでの会員情報入力・変更画面の「自動住所入力」ボタン押下時の無駄なエラーチェック処理を削除。
zip01,zip02には、NUM_CHECKが掛かっているので、ここのチェックは不要。
そもそも、下記スレッドの指摘通り、ここのチェック部分は動作していなかった。
参照スレッド
ソースコード閲覧時に見つけた変数名不正
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=12273&forum=1[[BR]]
実際に動作確認をして発見したわけではないのですが、 手動でバージョンアップをするためにソースを確認している際に 少し気になる箇所を発見したので報告させて頂きます。 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が小文字) となっています。 ソースを見る限り、入力形式不正チェックが上手く動かないのではないかと思うのですが…
Note: See
TracTickets for help on using
tickets.