Merge pull request #807 from seasoftjapan/seasoft-805

SC_CartSession::setPrevURL() の呼び出しを削除する #805
EC-CUBE · Feb 14, 2024 · 850c71d · 850c71d
2 parents 7a1e2c6 + 4aa5ac3
commit 850c71d
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 32 deletions.
diff --git a/data/class/SC_AdminView.php b/data/class/SC_AdminView.php
@@ -23,11 +23,6 @@
 
 class SC_AdminView extends SC_View_Ex
 {
-    public function __construct()
-    {
-        parent::__construct();
-    }
-
     public function init()
     {
         parent::init();

diff --git a/data/class/SC_CartSession.php b/data/class/SC_CartSession.php
@@ -279,7 +279,11 @@ public function addProduct($product_class_id, $quantity)
         }
     }
 
-    // 前頁のURLを記録しておく
+    /**
+     * 前頁のURLを記録しておく
+     *
+     * @deprecated 2.18.0 本体では呼ばれない。
+     */
     public function setPrevURL($url, $excludePaths = array())
     {
         // 前頁として記録しないページを指定する。
@@ -301,7 +305,11 @@ public function setPrevURL($url, $excludePaths = array())
         }
     }
 
-    // 前頁のURLを取得する
+    /**
+     * 前頁のURLを取得する
+     *
+     * @deprecated 2.18.0 本体では利用していない。
+     */
     public function getPrevURL()
     {
         return isset($_SESSION['prev_url']) ? $_SESSION['prev_url'] : '';

diff --git a/data/class/SC_Display.php b/data/class/SC_Display.php
@@ -48,17 +48,16 @@ class SC_Display
      * const('ADMIN',99);
      */
 
-    public function __construct($hasPrevURL = true)
+    public function __construct()
     {
         $this->response = new SC_Response_Ex();
-        if ($hasPrevURL) {
-            $this->setPrevURL();
-        }
     }
 
+    /**
+     * @deprecated 2.18.0 本体では利用していない。
+     */
     public function setPrevURL()
     {
-        // TODO SC_SiteSession で実装した方が良さげ
         $objCartSess = new SC_CartSession_Ex();
         $objCartSess->setPrevURL($_SERVER['REQUEST_URI']);
     }

diff --git a/data/class/SC_MobileView.php b/data/class/SC_MobileView.php
@@ -26,11 +26,6 @@
  */
 class SC_MobileView extends SC_SiteView_Ex
 {
-    public function __construct($setPrevURL = true)
-    {
-        parent::__construct($setPrevURL);
-    }
-
     public function init()
     {
         parent::init();

diff --git a/data/class/SC_SiteView.php b/data/class/SC_SiteView.php
@@ -23,15 +23,6 @@
 
 class SC_SiteView extends SC_View_Ex
 {
-    public function __construct($setPrevURL = true)
-    {
-        parent::__construct();
-
-        if ($setPrevURL) {
-            $this->setPrevURL();
-        }
-    }
-
     public function init()
     {
         parent::init();
@@ -42,6 +33,9 @@ public function init()
         $this->assignTemplatePath(DEVICE_TYPE_PC);
     }
 
+    /**
+     * @deprecated 2.18.0 本体では利用していない。
+     */
     public function setPrevURL()
     {
             $objCartSess = new SC_CartSession_Ex();

diff --git a/data/class/SC_SmartphoneView.php b/data/class/SC_SmartphoneView.php
@@ -23,11 +23,6 @@
 
 class SC_SmartphoneView extends SC_SiteView_Ex
 {
-    public function __construct($setPrevURL = true)
-    {
-        parent::__construct($setPrevURL);
-    }
-
     public function init()
     {
         parent::init();

diff --git a/data/class/pages/cart/LC_Page_Cart.php b/data/class/pages/cart/LC_Page_Cart.php
@@ -209,7 +209,6 @@ public function action()
         }
 
         // 前頁のURLを取得
-        // TODO: SC_CartSession::setPrevURL()利用不可。
         $this->lfGetCartPrevUrl($_SESSION, $_SERVER['HTTP_REFERER']);
         $this->tpl_prev_url = (isset($_SESSION['cart_prev_url'])) ? $_SESSION['cart_prev_url'] : '';