Ticket #1983 (new 改善提案) — at Initial Version

Opened 11 years ago

Last modified 10 years ago

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.