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

Opened 9 years ago

Last modified 9 years ago

SC_FormParam#initParam 一部クラス変数の初期化漏れ

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.12.0
Component: その他 Version: 2.12.0 α
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

少なくとも $param$html_disp_name は漏れている。

外部から呼び出しが無ければ実質的には問題無いとは思う。しかし、LC_Page_Admin_Basis_DeliveryInput から呼ばれている。

そもそも、初期化が必要となるケースは非常にレア(現状だと上述の1箇所)なので、クラス変数の定義時に初期化を行い、ページクラスで初期化が必要となるケースでは、インスタンスを生成すれば済む。また、今回のような実装誤りも防げる。

r21645

Change History

comment:1 Changed 9 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • 修正済み set
  • Status changed from new to assigned
  • Description modified (diff)

comment:2 Changed 9 years ago by kajiwara

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

修正ありがとうございます。

こちらにてクローズさせていただきます。

Note: See TracTickets for help on using tickets.