[Svn-src-all:1411] [version-2_5-dev 20302] #964 リファクタリング 宣言を除き, 引数や返り値が無く, すべて内部のメンバ変数で処理するような関数は極力作成しない

[coelacanth]

Subversion committed to /home/svn/open 20302
http://svn.ec-cube.net/open_trac/changeset/20302
┌────────────────────────────┐
│更新者　:  coelacanth                                   │
│更新日時:  2011-02-21 18:50:54 +0900 (月, 21  2月 2011)│
└────────────────────────────┘

Log:
--------------------------------------------------------
#964 リファクタリング 宣言を除き, 引数や返り値が無く, すべて内部のメンバ変数で処理するような関数は極力作成しない

Changed:                      [U:修正，A:追加，D:削除]
--------------------------------------------------------
U   branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Control.php

変更: branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Control.php
===================================================================
--- branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Control.php	2011-02-21 09:37:50 UTC (rev 20301)
+++ branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Control.php	2011-02-21 09:50:54 UTC (rev 20302)
@@ -81,9 +81,9 @@
                 $objFormParam->convParam();
 
                 // エラーチェック
-                $this->arrErr = $this->lfCheckError();
+                $this->arrErr = $objFormParam->checkError();
                 if(count($this->arrErr) == 0) {
-                    $this->lfSiteControlData($_POST['control_id'], $objFormParam->getHashArray());
+                    $this->lfSiteControlData($objFormParam->getHashArray());
                     // javascript実行
                     $this->tpl_onload = "alert('更新が完了しました。');";
                 }
@@ -93,6 +93,10 @@
                 break;
         }
 
+        $this->arrControlList = $arrSiteControlList;
+    }
+
+    function lfGetControlList() {
         // サイト管理情報の取得
         $arrSiteControlList = $this->lfGetControlList();
         $masterData = new SC_DB_MasterData_Ex();
@@ -109,8 +113,7 @@
                     break;
             }
         }
-
-        $this->arrControlList = $arrSiteControlList;
+        return $arrSiteControlList;
     }
 
     /**
@@ -137,18 +140,8 @@
         $objFormParam->addParam("設定状況", "control_flg", INT_LEN, "n", array("EXIST_CHECK", "NUM_CHECK", "MAX_LENGTH_CHECK"));
     }
 
-    /* 入力内容のチェック */
-    function lfCheckError(&$objFormParam) {
-        // 入力データを渡す。
-        $arrRet =  $objFormParam->getHashArray();
-        $objErr = new SC_CheckError($arrRet);
-        $objErr->arrErr = $objFormParam->checkError();
-
-        return $objErr->arrErr;
-    }
-
     /* DBへデータを登録する */
-    function lfSiteControlData($control_id = "", $post) {
+    function lfSiteControlData($post) {
         $objQuery =& SC_Query::getSingletonInstance();
         $sqlval = $post;
         $sqlval['update_date'] = 'Now()';
@@ -162,7 +155,7 @@
         // 既存編集
         } else {
             $where = "control_id = ?";
-            $objQuery->update("dtb_site_control", $sqlval, $where, array($control_id));
+            $objQuery->update("dtb_site_control", $sqlval, $where, array($post['control_id']));
         }
     }
 }