EC-CUBE · ji-eunsoo · Jan 24, 2024 · May 4, 2020 · May 4, 2020 · May 7, 2020
diff --git a/data/Smarty/templates/admin/basis/point.tpl b/data/Smarty/templates/admin/basis/point.tpl
@@ -24,7 +24,7 @@
 
 <form name="point_form" id="point_form" method="post" action="">
     <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
-    <input type="hidden" name="mode" value="<!--{$tpl_mode|h}-->" />
+    <input type="hidden" name="mode" value="" />
     <div id="basis" class="contents-main">
         <table>
             <tr>
@@ -51,7 +51,7 @@
 
         <div class="btn-area">
             <ul>
-                <li><a class="btn-action" href="javascript:;" onclick="eccube.fnFormModeSubmit('point_form', '<!--{$tpl_mode|h}-->', '', ''); return false;"><span class="btn-next">この内容で登録する</span></a></li>
+                <li><a class="btn-action" href="javascript:;" onclick="eccube.fnFormModeSubmit('point_form', 'update', '', ''); return false;"><span class="btn-next">この内容で登録する</span></a></li>
             </ul>
         </div>
     </div>

diff --git a/data/Smarty/templates/admin/basis/tradelaw.tpl b/data/Smarty/templates/admin/basis/tradelaw.tpl
@@ -24,7 +24,7 @@
 
 <form name="form1" id="form1" method="post" action="">
     <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
-    <input type="hidden" name="mode" value="<!--{$tpl_mode|h}-->" />
+    <input type="hidden" name="mode" value="" />
     <div id="basis" class="contents-main">
         <table class="form">
             <tr>
@@ -197,7 +197,7 @@
 
         <div class="btn-area">
             <ul>
-                <li><a class="btn-action" href="javascript:;" onclick="eccube.fnFormModeSubmit('form1', '<!--{$tpl_mode|h}-->', '', ''); return false;"><span class="btn-next">この内容で登録する</span></a></li>
+                <li><a class="btn-action" href="javascript:;" onclick="eccube.fnFormModeSubmit('form1', 'update', '', ''); return false;"><span class="btn-next">この内容で登録する</span></a></li>
             </ul>
         </div>
     </div>

diff --git a/data/class/helper/SC_Helper_DB.php b/data/class/helper/SC_Helper_DB.php
@@ -30,6 +30,8 @@
  */
 class SC_Helper_DB
 {
+    const BASIS_DATA_CACHE_REALFILE = MASTER_DATA_REALDIR . 'dtb_baseinfo.serial';
+
     /** ルートカテゴリ取得フラグ */
     public $g_root_on;
 
@@ -121,20 +123,32 @@
     /**
      * 店舗基本情報を取得する.
      *
-     * 引数 $force が false の場合は, 初回のみ DB 接続し,
-     * 2回目以降はキャッシュされた結果を使用する.
+     * 引数 $force が false の場合は, キャッシュされた結果を使用する.
      *
-     * @param  boolean $force 強制的にDB取得するか
+     * @param  boolean $force キャッシュファイルを生成し、ローカルキャッシュを削除するか
      * @return array   店舗基本情報の配列
      */
     public static function sfGetBasisData($force = false)
     {
         static $arrData = null;
 
-        if ($force || is_null($arrData)) {
-            $objQuery = SC_Query_Ex::getSingletonInstance();
+        // キャッシュファイルが存在しない場合、キャッシュファイルを生成する
+        if (!$force && !file_exists(SC_Helper_DB_Ex::BASIS_DATA_CACHE_REALFILE)) {
+            $force = true;
+        }
 
-            $arrData = $objQuery->getRow('*', 'dtb_baseinfo');
+        if ($force) {
+            // キャッシュファイルを生成
+            $success = SC_Helper_DB_Ex::sfCreateBasisDataCache();
+
+            // ローカルキャッシュを削除
+            $arrData = null;
+        }
+
+        // ローカルキャッシュが無い場合、キャッシュファイルを読み込む
+        if (is_null($arrData)) {
+            // キャッシュデータファイルを読み込む
+            $arrData = SC_Helper_DB_Ex::getBasisDataFromCacheFile();
         }
 
         return $arrData;
@@ -143,29 +157,61 @@
     /**
      * 基本情報のキャッシュデータを取得する
      *
+     * エラー画面表示で直接呼ばれる。キャッシュファイルが存在しなくとも空の配列を応答することで、(幾らかの情報欠落などはあるかもしれないが) エラー画面の表示できるよう考慮している。
      * @param  boolean $generate キャッシュファイルが無い時、DBのデータを基にキャッシュを生成するか
      * @return array   店舗基本情報の配列
+     * @deprecated 2.17.1 本体で使用されていないため非推奨
      */
     public static function sfGetBasisDataCache($generate = false)
     {
-        // テーブル名
-        $name = 'dtb_baseinfo';
-        // キャッシュファイルパス
-        $filepath = MASTER_DATA_REALDIR . $name . '.serial';
+        $cacheData = [];
+
         // ファイル存在確認
-        if (!file_exists($filepath) && $generate) {
+        if (!file_exists(SC_Helper_DB_Ex::BASIS_DATA_CACHE_REALFILE) && $generate) {
             // 存在していなければキャッシュ生成
             static::sfCreateBasisDataCache();
         }
-        // 戻り値初期化
-        $cacheData = array();
-        // キャッシュファイルが存在すれば読み込む
-        if (file_exists($filepath)) {
+
+        $cacheData = SC_Helper_DB_Ex::getBasisDataFromCacheFile(true);
+
+        return $cacheData;
+    }
+
+    /**
+     * 基本情報のキャッシュデータを取得する
+     *
+     * エラー画面表示で直接呼ばれる。キャッシュファイルが存在しなくとも空の配列を応答することで、(幾らかの情報欠落などはあるかもしれないが) エラー画面の表示できるよう考慮している。
+     * @param  boolean $ignore_error エラーを無視するか
+     * @return array   店舗基本情報の配列
+     */
+    public function getBasisDataFromCacheFile($ignore_error = false)
+    {
+        $arrReturn = [];
+
+        // ファイル存在確認
+        if (file_exists(SC_Helper_DB_Ex::BASIS_DATA_CACHE_REALFILE)) {
             // キャッシュデータファイルを読み込みアンシリアライズした配列を取得
-            $cacheData = unserialize(file_get_contents($filepath));
+            $arrReturn = unserialize(file_get_contents(SC_Helper_DB_Ex::BASIS_DATA_CACHE_REALFILE));
         }
-        // return
-        return $cacheData;
+        elseif (!$ignore_error) {
+            throw new Exception('基本情報のキャッシュデータファイルが存在しません。');
+        }
+
+        return $arrReturn;
+    }
+
+    /**
+     * 店舗基本情報をDBから取得する.
+     *
+     * @return array   店舗基本情報の配列
+     */
+    public function getBasisDataFromDB()
+    {
+        $objQuery = SC_Query_Ex::getSingletonInstance();
+
+        $arrReturn = $objQuery->getRow('*', 'dtb_baseinfo');
+
+        return $arrReturn;
     }
 
     /**
@@ -181,16 +227,12 @@
      */
     public static function sfCreateBasisDataCache()
     {
-        // テーブル名
-        $name = 'dtb_baseinfo';
-        // キャッシュファイルパス
-        $filepath = MASTER_DATA_REALDIR . $name . '.serial';
         // データ取得
-        $arrData = static::sfGetBasisData(true);
+        $arrData = static::getBasisDataFromDB(true);
         // シリアライズ
         $data = serialize($arrData);
         // ファイルを書き出しモードで開く
-        $handle = fopen($filepath, 'w');
+        $handle = fopen(SC_Helper_DB_Ex::BASIS_DATA_CACHE_REALFILE, 'w');
         if (!$handle) {
             // ファイル生成失敗
             return false;
@@ -1650,6 +1692,9 @@
             $objQuery->insert('dtb_baseinfo', $arrData);
             GC_Utils_Ex::gfPrintLog('dtb_baseinfo に INSERT を実行しました。');
         }
+
+        // キャッシュデータファイルを生成する
+        SC_Helper_DB_Ex::sfCreateBasisDataCache();
     }
 
     /**

diff --git a/data/class/pages/admin/basis/LC_Page_Admin_Basis.php b/data/class/pages/admin/basis/LC_Page_Admin_Basis.php
@@ -105,12 +105,10 @@ public function action()
                 $arrData = $objFormParam->getDbArray();
                 SC_Helper_DB_Ex::registerBasisData($arrData);
 
-                // キャッシュファイル更新
-                $objDb->sfCreateBasisDataCache();
                 $this->tpl_onload .= "window.alert('SHOPマスターの登録が完了しました。');";
             // breakはつけない
             default:
-                $arrRet = $objDb->sfGetBasisData(true);
+                $arrRet = $objDb->getBasisDataFromDB();
                 $objFormParam->setParam($arrRet);
                 $this->arrForm = $objFormParam->getHashArray();
                 $this->arrForm['regular_holiday_ids'] = explode('|', $this->arrForm['regular_holiday_ids']);

diff --git a/data/class/pages/admin/basis/LC_Page_Admin_Basis_Point.php b/data/class/pages/admin/basis/LC_Page_Admin_Basis_Point.php
@@ -72,11 +72,7 @@ public function action()
         // POST値の取得
         $objFormParam->setParam($_POST);
 
-        if ($objDb->sfGetBasisExists()) {
-            $this->tpl_mode = 'update';
-        } else {
-            $this->tpl_mode = 'insert';
-        }
+        $this->tpl_mode = 'update'; // 旧バージョンテンプレート互換
 
         if (!empty($_POST)) {
             // 入力値の変換
@@ -86,16 +82,15 @@ public function action()
             if (count($this->arrErr) == 0) {
                 switch ($this->getMode()) {
                     case 'update':
-                        $this->lfUpdateData($objFormParam->getHashArray()); // 既存編集
-                        break;
-                    case 'insert':
-                        $this->lfInsertData($objFormParam->getHashArray()); // 新規作成
+                        SC_Helper_DB_Ex::registerBasisData($objFormParam->getHashArray());
+
+                        // 再表示
+                        $this->tpl_onload = "window.alert('ポイント設定が完了しました。');";
+
                         break;
                     default:
                         break;
                 }
-                // 再表示
-                $this->tpl_onload = "window.alert('ポイント設定が完了しました。');";
             }
         } else {
             $arrRet = $objDb->sfGetBasisData();
@@ -115,24 +110,19 @@ public function lfInitParam(&$objFormParam)
         $objFormParam->addParam('会員登録時付与ポイント', 'welcome_point', INT_LEN, 'n', array('EXIST_CHECK', 'MAX_LENGTH_CHECK', 'NUM_CHECK'));
     }
 
+    /**
+     * @deprecated SC_Helper_DB_Ex::registerBasisData() を使う。
+     */
     public function lfUpdateData($post)
     {
-        // 入力データを渡す。
-        $sqlval = $post;
-        $sqlval['update_date'] = 'CURRENT_TIMESTAMP';
-        $objQuery = SC_Query_Ex::getSingletonInstance();
-        // UPDATEの実行
-        $objQuery->update('dtb_baseinfo', $sqlval);
+        SC_Helper_DB_Ex::registerBasisData($post);
     }
 
+    /**
+     * @deprecated SC_Helper_DB_Ex::registerBasisData() を使う。
+     */
     public function lfInsertData($post)
     {
-        // 入力データを渡す。
-        $sqlval = $post;
-        $sqlval['id'] = 1;
-        $sqlval['update_date'] = 'CURRENT_TIMESTAMP';
-        $objQuery = SC_Query_Ex::getSingletonInstance();
-        // INSERTの実行
-        $objQuery->insert('dtb_baseinfo', $sqlval);
+        SC_Helper_DB_Ex::registerBasisData($post);
     }
 }
diff --git a/data/class/pages/admin/basis/LC_Page_Admin_Basis_Tradelaw.php b/data/class/pages/admin/basis/LC_Page_Admin_Basis_Tradelaw.php
@@ -73,11 +73,7 @@ public function action()
         $this->lfInitParam($objFormParam);
         $objFormParam->setParam($_POST);
 
-        if ($objDb->sfGetBasisExists()) {
-            $this->tpl_mode = 'update';
-        } else {
-            $this->tpl_mode = 'insert';
-        }
+        $this->tpl_mode = 'update'; // 旧バージョンテンプレート互換
 
         if (!empty($_POST)) {
             // 入力値の変換
@@ -87,16 +83,15 @@ public function action()
             if (count($this->arrErr) == 0) {
                 switch ($this->getMode()) {
                     case 'update':
-                        $this->lfUpdateData($objFormParam->getHashArray()); // 既存編集
-                        break;
-                    case 'insert':
-                        $this->lfInsertData($objFormParam->getHashArray()); // 新規作成
+                        SC_Helper_DB_Ex::registerBasisData($objFormParam->getHashArray());
+
+                        // 再表示
+                        $this->tpl_onload = "window.alert('特定商取引法の登録が完了しました。');";
+
                         break;
                     default:
                         break;
                 }
-                // 再表示
-                $this->tpl_onload = "window.alert('特定商取引法の登録が完了しました。');";
             }
         } else {
             $arrRet = $objDb->sfGetBasisData();
@@ -135,20 +130,20 @@ public function lfInitParam(&$objFormParam)
         $objFormParam->addParam('返品・交換について', 'law_term06', MLTEXT_LEN, 'KVa', array('EXIST_CHECK', 'MAX_LENGTH_CHECK'));
     }
 
+    /**
+     * @deprecated SC_Helper_DB_Ex::registerBasisData() を使う。
+     */
     public function lfUpdateData($sqlval)
     {
-        $sqlval['update_date'] = 'CURRENT_TIMESTAMP';
-        $objQuery = SC_Query_Ex::getSingletonInstance();
-        // UPDATEの実行
-        $objQuery->update('dtb_baseinfo', $sqlval);
+        SC_Helper_DB_Ex::registerBasisData($sqlval);
     }
 
+    /**
+     * @deprecated SC_Helper_DB_Ex::registerBasisData() を使う。
+     */
     public function lfInsertData($sqlval)
     {
-        $sqlval['update_date'] = 'CURRENT_TIMESTAMP';
-        $objQuery = SC_Query_Ex::getSingletonInstance();
-        // INSERTの実行
-        $objQuery->insert('dtb_baseinfo', $sqlval);
+        SC_Helper_DB_Ex::registerBasisData($sqlval);
     }
 
     /* 入力内容のチェック */

diff --git a/data/class/pages/error/LC_Page_Error.php b/data/class/pages/error/LC_Page_Error.php
@@ -68,7 +68,7 @@
         }
 
         // キャッシュから店舗情報取得（DBへの接続は行わない）
-        $this->arrSiteInfo = SC_Helper_DB_Ex::sfGetBasisDataCache(false);
+        $this->arrSiteInfo = SC_Helper_DB_Ex::getBasisDataFromCacheFile(true);
     }
 
     /**

diff --git a/data/class/pages/error/LC_Page_Error_DispError.php b/data/class/pages/error/LC_Page_Error_DispError.php
@@ -56,7 +56,7 @@
         $objHelperPlugin->arrRegistedPluginActions = array();
 
         // キャッシュから店舗情報取得（DBへの接続は行わない）
-        $this->arrSiteInfo = SC_Helper_DB_Ex::sfGetBasisDataCache(false);
+        $this->arrSiteInfo = SC_Helper_DB_Ex::getBasisDataFromCacheFile(true);
     }
 
     /**

diff --git a/tests/class/SC_SendMailTest.php b/tests/class/SC_SendMailTest.php
@@ -148,8 +148,10 @@ public function testGetRecip()
      */
     public function testGetBackendParams()
     {
-        $this->objQuery->update('dtb_baseinfo', ['email04' => '[email protected]']);
         $objDb = new SC_Helper_DB_Ex();
+
+        SC_Helper_DB_Ex::registerBasisData(['email04' => '[email protected]']);
+
         $objSite = $objDb->sfGetBasisData();
         $this->expected = [
             '-f '.$objSite['email04']
-Original file line number
+Diff line change
@@ Expand Up / @@ -68,7 +68,7 @@ @@
             }
             // キャッシュから店舗情報取得（DBへの接続は行わない）
-            $this->arrSiteInfo = SC_Helper_DB_Ex::sfGetBasisDataCache(false);
+            $this->arrSiteInfo = SC_Helper_DB_Ex::getBasisDataFromCacheFile(true);
         }
         /**
@@ Expand Down @@