Changeset 20302

Timestamp:

2011/02/21 18:50:54 (13 years ago)

Author:

coelacanth

Message:

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

File:

• branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Control.php (modified) (5 diffs)

branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Control.php

@@ -82 +82 @@
 
                 // エラーチェック
-                $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('更新が完了しました。');";
@@ -94 +94 @@
         }
 
+        $this->arrControlList = $arrSiteControlList;
+    }
+
+    function lfGetControlList() {
         // サイト管理情報の取得
         $arrSiteControlList = $this->lfGetControlList();
@@ -110 +114 @@
             }
         }
-
-        $this->arrControlList = $arrSiteControlList;
+        return $arrSiteControlList;
     }
 
@@ -138 +141 @@
     }
 
-    /* 入力内容のチェック */
-    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;
@@ -163 +156 @@
         } 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']));
         }
     }