Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
seasoftjapan opened this issue Jan 26, 2018 · 3 comments
Closed

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

seasoftjapan opened this issue Jan 26, 2018 · 3 comments
Labels
Milestone

Comments

@seasoftjapan
Copy link
Contributor

基本は 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
Copy link
Contributor Author

LC_Page_Admin_Products_Product は、

        parent::lfInitParam($objFormParam);

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

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

@okazy okazy added this to the eccube-2.13.6 milestone Feb 1, 2018
@okazy okazy added the bug:Low label Feb 1, 2018
@so-amuamu
Copy link

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

@nobuhiko nobuhiko added question and removed bug:Low labels Aug 2, 2018
@so-amuamu so-amuamu modified the milestones: eccube-2.13.6, 2.x Aug 16, 2018
@nanasess
Copy link
Contributor

#526 で該当の extends 不要となったため、こちらはクローズします

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants