Changeset 20037

Timestamp:

2011/01/25 04:00:29 (12 years ago)

Author:

AMUAMU

Message:

#926 (高度なデータベース管理でチェックを押して登録を押してもindexが張られない) の解決。
#494 (プラグイン機能)の変更に伴うテンプレート呼び出し形式に対応していなかった問題があったことに対応。

Location:

branches/version-2_5-dev/data

Files:

• Smarty/templates/admin/system/editdb.tpl (modified) (2 diffs)
• class/SC_Query.php (modified) (2 diffs)
• class/db/SC_DB_DBFactory.php (modified) (1 diff)
• class/db/dbfactory/SC_DB_DBFactory_MYSQL.php (modified) (1 diff)
• class/pages/admin/system/LC_Page_Admin_System_Editdb.php (modified) (7 diffs)

branches/version-2_5-dev/data/Smarty/templates/admin/system/editdb.tpl

@@ -29 +29 @@
     </div>
       <table class="list">
-        <colgroup width="8%">
+        <colgroup width="5%">
+        <colgroup width="5%">
         <colgroup width="28%">
         <colgroup width="25%">
-        <colgroup width="45%">
+        <colgroup width="43%">
       <tr>
-        <th>インデックス</th>
-        <th>テーブル名</th>
-        <th>カラム名</th>
-        <th>説明</th>
+        <th colspan="2">インデックス</th>
+        <th rowspan="2">テーブル名</th>
+        <th rowspan="2">カラム名</th>
+        <th rowspan="2">説明</th>
+      </tr>
+      <tr>
+        <th>ON</th>
+        <th>OFF</th>
       </tr>
 
@@ -44 +49 @@
   
     <tr>
-    <td class="center"><input type="checkbox" name="indexflag_new[]" value="<!--{$smarty.section.cnt.index}-->" <!--{if $arrForm[cnt].indexflag == "1"}-->checked<!--{/if}--> /></td>
+    <td class="center"><input type="radio" name="indexflag_new[<!--{$smarty.section.cnt.iteration}-->]" value="1" <!--{if $arrForm[cnt].indexflag == "1"}-->checked<!--{/if}--> /></td>
+    <td class="center"><input type="radio" name="indexflag_new[<!--{$smarty.section.cnt.iteration}-->]" value="" <!--{if $arrForm[cnt].indexflag != "1"}-->checked<!--{/if}--> /></td>
     <th class="column"><!--{$arrForm[cnt].table_name}--></th>
     <th class="column"><!--{$arrForm[cnt].column_name}--></th>
     <td><!--{$arrForm[cnt].recommend_comment}--></td>
     </tr>
-<input type="hidden" name="table_name[]" value="<!--{$arrForm[cnt].table_name}-->" />
-<input type="hidden" name="column_name[]" value="<!--{$arrForm[cnt].column_name}-->" />
-<input type="hidden" name="indexflag[]" value="<!--{$arrForm[cnt].indexflag}-->" />
+<input type="hidden" name="table_name[<!--{$smarty.section.cnt.iteration}-->]" value="<!--{$arrForm[cnt].table_name}-->" />
+<input type="hidden" name="column_name[<!--{$smarty.section.cnt.iteration}-->]" value="<!--{$arrForm[cnt].column_name}-->" />
+<input type="hidden" name="indexflag[<!--{$smarty.section.cnt.iteration}-->]" value="<!--{$arrForm[cnt].indexflag}-->" />
 
 <!--{/section}-->

branches/version-2_5-dev/data/class/SC_Query.php

@@ -791 +791 @@
      * @param string $name インデックス名
      * @param array $definition フィールド名など　通常のフィールド指定時は、$definition=array('fields' => array('フィールド名' => array()));
+     *               MySQLのtext型フィールドを指定する場合は $definition['length'] = 'text_field(NNN)' が必要
      */
     function createIndex($table, $name, $definition) {
+        $definition = $this->dbFactory->sfGetCreateIndexDefinition($table, $name, $definition);
         $objManager =& $this->conn->loadModule('Manager');
         return $objManager->createIndex($table, $name, $definition);
@@ -807 +809 @@
         return $objManager->dropIndex($table, $name);
     }
-    
+
+    /**
+     * テーブルの詳細情報を取得する。
+     *
+     * @param string $table テーブル名
+     * @return array テーブル情報の配列
+     */
+    function getTableInfo($table) {
+        $objManager =& $this->conn->loadModule('Reverse');
+        return $objManager->tableInfo($table, NULL);
+    }
+
     /**
      * 値を適切にクォートする.

branches/version-2_5-dev/data/class/db/SC_DB_DBFactory.php

@@ -156 +156 @@
      */
     function findTableNames($expression = "") { return array(); }
+
+    /**
+     * インデックス作成の追加定義を取得する
+     *
+     * 引数に部分一致するテーブル名を配列で返す.
+     *
+     * @param string $table 対象テーブル名
+     * @param string $name 対象カラム名
+     * @return array インデックス設定情報配列
+     */
+    function sfGetCreateIndexDefinition($table, $name, $definition) { return $definition; }
+
 }
 ?>

branches/version-2_5-dev/data/class/db/dbfactory/SC_DB_DBFactory_MYSQL.php

@@ -434 +434 @@
 
     }
+    
+    /**
+     * インデックス作成の追加定義を取得する
+     *
+     * 引数に部分一致するテーブル名を配列で返す.
+     *
+     * @param string $table 対象テーブル名
+     * @param string $name 対象カラム名
+     * @return array インデックス設定情報配列
+     */
+    function sfGetCreateIndexDefinition($table, $name, $definition) {
+        $objQuery =& SC_Query::getSingletonInstance();
+        $arrTblInfo = $objQuery->getTableInfo($table);
+        foreach($arrTblInfo as $fieldInfo) {
+            if(array_key_exists($fieldInfo['name'], $definition['fields'])) {
+                if($fieldInfo['nativetype'] == 'text') {
+                    // TODO: text型フィールドの場合に255文字以内決めうちでインデックス列のサイズとして
+                    //       指定して良いか確認は必要。
+                    $definition['fields'][$fieldInfo['name']]['length'] = '255';
+                }
+            }
+        }
+        return $definition;
+    }
 }
 ?>

branches/version-2_5-dev/data/class/pages/admin/system/LC_Page_Admin_System_Editdb.php

@@ -23 +23 @@
 
 // {{{ requires
-require_once(CLASS_REALDIR . "pages/LC_Page.php");
+require_once CLASS_REALDIR . "pages/admin/LC_Page_Admin.php";
 
 /**
- * システム情報 のページクラス.
+ * 高度なデータベース管理 のページクラス.
  *
  * @package Page
@@ -32 +32 @@
  * @version $Id: LC_Page_Admin_System_Editdb.php 18701 2010-06-14 08:30:18Z nanasess $
  */
-class LC_Page_Admin_System_Editdb extends LC_Page {
+class LC_Page_Admin_System_Editdb extends LC_Page_Admin {
 
     // }}}
@@ -44 +44 @@
     function init() {
         parent::init();
+        
         $this->tpl_mainpage = 'system/editdb.tpl';
         $this->tpl_subnavi  = 'system/subnavi.tpl';
@@ -58 +59 @@
     function initForm() {
         $objForm = new SC_FormParam();
-        $objForm->addParam('mode', 'mode', INT_LEN, '', array('ALPHA_CHECK', 'MAX_LENGTH_CHECK'));
-        $objForm->addParam('table_name', 'テーブル名');
-        $objForm->addParam('column_name', 'カラム名');
-        $objForm->addParam('indexflag', 'インデックス');
-    $objForm->addParam('indexflag_new', 'インデックス（変更後）');
-    
-    $objForm->setParam($_POST);
+        $objForm->addParam('モード', 'mode', INT_LEN, 'n', array('ALPHA_CHECK', 'MAX_LENGTH_CHECK'));
+        $objForm->addParam('テーブル名', 'table_name');
+        $objForm->addParam('カラム名', 'column_name');
+        $objForm->addParam('インデックス', 'indexflag');
+        $objForm->addParam('インデックス（変更後）', 'indexflag_new');
+        $objForm->setParam($_POST);
         $this->objForm = $objForm;
     }
@@ -74 +74 @@
      */
     function process() {
+        $this->action();
+        $this->sendResponse();
+    }
 
+    /**
+     * Page のアクション.
+     *
+     * @return void
+     */
+    function action() {
         SC_Utils_Ex::sfIsSuccess(new SC_Session);
-        $objView = new SC_AdminView();
-    $objSiteInfo = new SC_SiteInfo();
-        $objQuery = new SC_Query();
-        $objDb = new SC_Helper_DB_Ex();
-        $objProduct = new SC_Product();
 
-    //インデックスの値を取得
-    $this->arrForm = $this->lfGetIndexList();
+        //フォームの値を取得
+        $this->initForm();
 
-    //フォームの値を取得
-        $this->initForm();
-    
         switch($this->objForm->getValue('mode')) {
-    //確認画面へ
-    case 'confirm' :
-        //POSTの値と配列の値を比較
-        // TODO: $_POSTを使わないように修正する。
-            $arrIndexFlag  = $_POST['indexflag'];
-            $arrIndexFlagNewIndex  = $_POST['indexflag_new'];
-
-            $arrIndexFlagNew = array();
-            for($i = 0; $i < count($arrIndexFlag); $i++) {
-                if(array_search($i, $arrIndexFlagNewIndex) !== false) {
-                    $arrIndexFlagNew[] = true;
-                } else {
-                    $arrIndexFlagNew[] = false;
-        }
+        case 'confirm' :
+            $this->lfDoChange();
+            break;
+        default:
+            break;
         }
 
-            $objQuery = new SC_Query();
-        foreach($arrIndexFlag as $key => $val){
-        if($val != $arrIndexFlagNew[$key]) {
-                    //値が異なっていた場合、インデックスのアップデートをかける
-                    $index_name = $_POST["table_name"][$key] . "_" . $_POST["column_name"][$key] . "_key";
-                    if($arrIndexFlagNew[$key] == false) {
-                        $objQuery->dropIndex($_POST["table_name"][$key], $index_name);
-                    } else {
-                        $objQuery->createIndex($_POST["table_name"][$key], $index_name, array('fields' => array($_POST["column_name"][$key] => array())));
-                    }
-
-                    // フォームに引継ぐ
-                    $this->arrForm[$key]["indexflag"] = true;
-        }
-        }
-        
-        $this->tpl_onload = "window.alert('インデックスの変更が完了しました。');";
-        default:
-        break;
-        }
-
-        $objView->assignobj($this);
-        $objView->display(MAIN_FRAME);
+        //インデックスの現在値を取得
+        $this->arrForm = $this->lfGetIndexList();
     }
 
@@ -139 +110 @@
     }
     
+    function lfGetTargetData() {
+        $objQuery = new SC_Query();
+        $arrIndexFlag = $this->objForm->getValue('indexflag');
+        $arrIndexFlagNew = $this->objForm->getValue('indexflag_new');
+        $arrTableName = $this->objForm->getValue('table_name');
+        $arrColumnName =$this->objForm->getValue('column_name');
+        $arrTarget = array();
+        // 変更されている対象を走査
+        for($i = 1; $i <= count($arrIndexFlag); $i++) {
+            //入力値チェック
+            $param = array('indexflag' => $arrIndexFlag[$i],
+                            'indexflag_new' => $arrIndexFlagNew[$i],
+                            'table_name' => $arrTableName[$i],
+                            'column_name' => $arrColumnName[$i]);
+            $objErr = new SC_CheckError($param);
+            $objErr->doFunc(array("インデックス(" . $i . ")", 'indexflag', INT_LEN), array("NUM_CHECK"));
+            $objErr->doFunc(array("インデックス変更後(" . $i . ")", 'indexflag_new', INT_LEN), array("NUM_CHECK"));
+            $objErr->doFunc(array("インデックス変更後(" . $i . ")", 'indexflag_new', INT_LEN), array("NUM_CHECK"));
+            $objErr->doFunc(array("テーブル名(" . $i . ")", 'table_name', STEXT_LEN), array("GRAPH_CHECK", "EXIST_CHECK", "MAX_LENGTH_CHECK"));
+            $objErr->doFunc(array("カラム名(" . $i . ")", 'column_name', STEXT_LEN), array("GRAPH_CHECK", "EXIST_CHECK", "MAX_LENGTH_CHECK"));
+            $arrErr = $objErr->arrErr;
+            if(count($arrErr) != 0) {
+                // 通常の送信ではエラーにならないはずです。
+                $this->tpl_onload = "window.alert('不正なデータがあったため処理を中断しました。');";
+                return;
+            }
+            if($param['indexflag'] != $param['indexflag_new']) {
+                // 入力値がデータにある対象テーブルかのチェック
+                if($objQuery->count('dtb_index_list', 'table_name = ? and column_name = ?', array($param['table_name'], $param['column_name']))) {
+                    $arrTarget[] = $param;
+                }
+            }
+        }
+        return $arrTarget;
+    }
+
+    function lfDoChange() {
+        $objQuery = new SC_Query();
+        $arrTarget = $this->lfGetTargetData();
+        if(count($arrTarget) == 0) {
+            $this->tpl_onload = "window.alert('変更対象となるデータはありませんでした。');";
+            return;
+        }
+
+        // 変更対象の設定変更
+        foreach($arrTarget as $item) {
+            $index_name = $item['table_name'] . '_' . $item['column_name'] . "_key";
+            $arrField = array( 'fields' => array($item['column_name'] => array()));
+            if($item['indexflag_new'] == '1') {
+                $objQuery->createIndex($item['table_name'], $index_name, $arrField);
+            }else{
+                $objQuery->dropIndex($item['table_name'], $index_name);
+            }
+        }
+        $this->tpl_onload = "window.alert('インデックスの変更が完了しました。');";
+    }
+
     /**
      * インデックス設定を行う一覧を返す関数
@@ -158 +186 @@
                 $arrIndexes = $objQuery->listTableIndexes($table);
             }
- 
             // インデックスが設定されているかを取得
-            if(array_search($table . "_" . $arrIndex["column_name"] . "_key", $arrIndexes) === false) {
-                $arrIndexList[$key]["indexflag"] = false;
+            $idx_name = $table . "_" . $arrIndex["column_name"] . "_key";
+            if(array_search($idx_name, $arrIndexes) === false) {
+                $arrIndexList[$key]["indexflag"] = '';
             } else {
-                $arrIndexList[$key]["indexflag"] = true;
+                $arrIndexList[$key]["indexflag"] = '1';
             }
         }
-    
         return $arrIndexList;
     }