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

Timestamp:

2010/08/04 13:52:27 (16 years ago)

Author:

nanasess

bzr:base-revision:

svn-v4:1e3b908f-19a9-db11-a64c-001125224ba8:branches/version-2_5-dev:18774

bzr:committer:

Kentaro Ohkouchi <[email protected]>

bzr:file-ids:

data/class/SC_Query.php 15078@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2FSC_Query.php
data/class/db/SC_DB_DBFactory.php 15123@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fdb%2FSC_DB_DBFactory.php
data/class/db/dbfactory/SC_DB_DBFactory_MYSQL.php 15123@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fdb%2Fdbfactory%2FSC_DB_DBFactory_MYSQL.php
data/class/db/dbfactory/SC_DB_DBFactory_PGSQL.php 15123@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fdb%2Fdbfactory%2FSC_DB_DBFactory_PGSQL.php
data/class/helper/SC_Helper_DB.php 15176@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fhelper%2FSC_Helper_DB.php
data/class/pages/shopping/LC_Page_Shopping_Complete.php 15223@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fshopping%2FLC_Page_Shopping_Complete.php
data/class/util/SC_Utils.php 15078@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Futil%2FSC_Utils.php
html/install/index.php 16809@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fhtml%2Finstall%2Findex.php
html/install/sql/create_table_mysql.sql 15078@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fhtml%2Finstall%2Fsql%2Fcreate_table_mysql.sql

bzr:mapping-version:

v4

bzr:repository-uuid:

1e3b908f-19a9-db11-a64c-001125224ba8

bzr:revision-id:

[email protected]

bzr:revno:

2258

bzr:revprop:branch-nick:

branches/version-2_5-dev

bzr:root:

branches/version-2_5-dev

bzr:text-parents:

data/class/db/SC_DB_DBFactory.php [email protected]
data/class/db/dbfactory/SC_DB_DBFactory_MYSQL.php [email protected]
data/class/db/dbfactory/SC_DB_DBFactory_PGSQL.php [email protected]
data/class/helper/SC_Helper_DB.php [email protected]
data/class/pages/shopping/LC_Page_Shopping_Complete.php [email protected]
data/class/util/SC_Utils.php [email protected]
html/install/index.php [email protected]
html/install/sql/create_table_mysql.sql svn-v4:1e3b908f-19a9-db11-a64c-001125224ba8:branches/version-2_5-dev:18764

bzr:timestamp:

2010-08-04 13:52:24.183000088 +0900

bzr:user-agent:

bzr2.1.2+bzr-svn1.0.2

svn:original-date:

2010-08-04T04:52:24.183000Z

Message:

#565(SC_DbConn クラスの削除)

• MySQL でインストーラ及び購入完了画面でエラーが発生していたのを修正
• SC_Query::quote() を追加

File:

• branches/version-2_5-dev/data/class/SC_Query.php (modified) (4 diffs)

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

@@ -161 +161 @@
 
         $sth = $this->conn->prepare($sql);
+        if (PEAR::isError($sth)) {
+            /*
+             * XXX インストーラのテーブルチェックに失敗してしまう.
+             * エラーハンドリングの方法を要検討
+             */
+            return;
+        }
         $affected = $sth->execute($arrval);
 
@@ -487 +494 @@
 
         if (PEAR::isError($result)) {
-            trigger_error($result->getMessage(), E_USER_ERROR);
+            /*
+             * XXX インストーラのテーブルチェックに失敗してしまう.
+             * エラーハンドリングの方法を要検討
+             */
+            //trigger_error($result->getMessage(), E_USER_ERROR);
         }
 
@@ -495 +506 @@
     /**
      * auto_incrementを取得する.
+     *
+     * XXX MDB2 の sequence 関数を使用する
      *
      * @param string $table_name テーブル名
@@ -501 +514 @@
     function get_auto_increment($table_name){
         // ロックする
-        $this->query("LOCK TABLES $table_name WRITE");
+        $this->conn->exec("LOCK TABLES $table_name WRITE");
 
         // 次のIncrementを取得
-        $arrRet = $this->getAll("SHOW TABLE STATUS LIKE ?", array($table_name));
+        $arrRet = $this->getAll("SHOW TABLE STATUS LIKE ". $this->quote($table_name));
         $auto_inc_no = $arrRet[0]["Auto_increment"];
 
         // 値をカウントアップしておく
-        $this->query("ALTER TABLE $table_name AUTO_INCREMENT=?" , $auto_inc_no + 1);
+        $this->conn->exec("ALTER TABLE $table_name AUTO_INCREMENT=" . $this->quote($auto_inc_no + 1));
 
         // 解除する
-        $this->query('UNLOCK TABLES');
+        $this->conn->exec('UNLOCK TABLES');
 
         return $auto_inc_no;
     }
+
+    /**
+     * 値を適切にクォートする.
+     *
+     * TODO MDB2 に対応するための暫定的な措置.
+     *      ブレースホルダが使用できない実装があるため.
+     *      本来であれば, MDB2::prepare() を適切に使用するべき
+     *
+     * @see MDB2::quote()
+     * @param string $val クォートを行う文字列
+     * @return string クォートされた文字列
+     */
+    function quote($val) {
+        return $this->conn->quote($val);
+    }
 }