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

Opened 9 years ago

Last modified 8 years ago

純粋な定数を定義する場を用意する

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)

現状、パラメータ (mtb_constants) に純粋な定数を定義している実装が見受けられる。

パラメータは、管理機能からの変更が想定されそうな範囲に留め、純粋な定数はスタティックなソースとして記述する。

  • 本質的には、(PHP4をサポート外とした今となっては) クラス定数として定義すべきと見受けられるものもあるが、本チケットでは扱わない。
  • そもそも定数として怪しい物もあるが、本チケットでは改修しない。
  • ORDER_* は悩んだが、現状で妥当・不適の判断がつかないため保留した。

r21460

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 8 years ago by kajiwara

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

Seasoftさん

本件、ご報告ならびにご対応いただき、ありがとうございます。

今まで、据え置きしていた部分でしたが、ご指摘の通り、「管理機能からの変更が想定されそうな範囲に留め」という部分は仰る通りと思いますので、こちらの方向でOKとしたいと思います。

よろしくお願いします。

Note: See TracTickets for help on using tickets.