LC_Page_Admin_* の extends にバラつきがある

[seasoftjapan]

基本は extends LC_Page_Admin_Ex だが、以下の例外がある。

data\class\pages\admin\order\LC_Page_Admin_Order_Disp.php(33): class LC_Page_Admin_Order_Disp extends LC_Page_Admin_Order_Ex
data\class\pages\admin\order\LC_Page_Admin_Order_Edit.php(33): class LC_Page_Admin_Order_Edit extends LC_Page_Admin_Order_Ex
data\class\pages\admin\order\LC_Page_Admin_Order_Mail.php(33): class LC_Page_Admin_Order_Mail extends LC_Page_Admin_Order_Ex
data\class\pages\admin\products\LC_Page_Admin_Products_ReviewEdit.php(33): class LC_Page_Admin_Products_ReviewEdit extends LC_Page_Admin_Products_Review

ケアレスミスだと思うが、意味があるならコメントを記述すべき。

[seasoftjapan]

LC_Page_Admin_Products_Product は、

        parent::lfInitParam($objFormParam);

として、参照がある様子。

動作としては良いのだろうが、設計的に違和感がある。

[so-amuamu]

検索パラメーターの引き継ぎをして、「戻る」を有効にする関係で、使っている記憶があります 本来筋であれば、検索用パラメーターはSC_Helper系にまとめて、そちらを各ページが読み込むべきですが 構造の変更になってしまうため今回は避けたほうが良いかも