Ticket #2044 (closed 改善提案: 修正済)

Opened 7 years ago

Last modified 6 years ago

無駄な処理を改善する 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 7 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • Status changed from new to assigned

comment:2 Changed 7 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が小文字)
となっています。

ソースを見る限り、入力形式不正チェックが上手く動かないのではないかと思うのですが…

comment:3 Changed 6 years ago by Seasoft

  • Summary changed from 無駄な処理を改善する for 2.12.4 to 無駄な処理を改善する for 2.13.0

comment:4 Changed 6 years ago by shutta

r22742 をコミット。

comment:5 Changed 6 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • 修正済み set
  • Resolution set to 修正済

本チケットは以上の対応でクローズとさせて頂きます。

ありがとうございました。

Note: See TracTickets for help on using tickets.