Ticket #2591 (closed バグ指摘: 修正済)

Opened 6 years ago

Last modified 6 years ago

SC_FormParam::recursionCheck()の2次元以上の配列への対応

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.13.3
Component: その他 Version: 2.13.2
Keywords: Cc:
修正済み: yes

Description

SC_FormParam::recursionCheck()が、関数内のコメントのTODOやFIXMEへの記載にもあるように、2次元以上の配列へ対応できていない。

不具合のある例としては、
受注編集にて、複数のお届け先を指定し、商品の数量に数字以外を入力して入力エラーにすると、エラーでない数量欄がエラーと判別されてしまう。

Change History

comment:1 Changed 6 years ago by shutta

  • Owner changed from somebody to shutta
  • 修正済み set
  • Status changed from new to assigned

r23567 にて修正。
引数を変えていますが、この関数は内部的に利用されているもので、影響が少ないと思いますので、変更を了承して頂きたいです。
また、危険なeval()が使用されていたので、使用しないようにも改修しました。

comment:2 Changed 6 years ago by kim

shutta 様

ありがとうございます。
仮パッケージ含め検証しましたが、問題なさそうですので
最終パッケージに採用予定です。

最終パッケージでモジュール含め検証を行った上でクローズいたします。

comment:3 Changed 6 years ago by kim

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

こちら特に問題が確認できませんでしたので、クローズいたします。

Note: See TracTickets for help on using tickets.