Ticket #1983 (new 改善提案) — at Initial Version
LC_Page_Admin_Total のセッションの扱いが無駄に危険
Reported by: | Seasoft | Owned by: | somebody |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.13.x |
Component: | 管理画面 | Version: | 2.12.2 |
Keywords: | Cc: | ||
修正済み: | no |
Description
$_SESSION = $this->lfSaveDateSession($_SESSION, $this->arrHidden);
- セッション変数を丸ごと書き換えている
- セッション変数はスーパーグローバル変数なので渡す必要は無い。($_SESSION['total']を渡すならば分からなくもないが・・・)
- $_SESSION['total'] はあまりにも安直な命名と感じる。せめて、ページクラス名に相当する程度の冗長さがあって良いように感じる。
Note: See
TracTickets for help on using
tickets.