OpenPNE 3

差し戻します。以下の説明を読んで、修正方法について再度検討しなおしてください。

AUTO_INCREMENT の取得方法について¶

c_member テーブルのレコードは、 OpenPNE 3 のデータベースには (まだ) 入れることのできない情報を持っているために、アップグレードが終わっても c_member テーブルの削除処理をおこなっていません。

c_member テーブルは今後も使用する可能性があるため、ダミーのデータを挿入したのち、そのデータを削除していないことは問題です。

また、できるだけ無用なデータ操作処理を伴わない形で実現できないかについても再考してください。

アップグレードに関しては MySQL に依存していても問題ありません。ということで、以下の手段によって AUTO_INCREMENT を取得することができます。

• SHOW TABLE STATUS LIKE "c_member" によって得られたレコードの、 Auto_increment カラムを取得する
• SELECT AUTO_INCREMENT FROM information_schema.TABLES WHERE TABLE_NAME='c_member' AND TABLE_SCHEMA = DATABASE(); によって取得する

ただし後者は information_schema を利用する関係で、 MySQL 5.1 以降でしか使えない手ということになり、採用することができません。前者は OpenPNE がサポートするバージョンよりも古い MySQL でも利用可能だと思いますが、もし採用する場合は念のため確認をお願いします。

AUTO_INCREMENT の設定方法について¶

PREPARE 文や EXECUTE 文は MySQL 5.1 の新機能であり、 5.1 未満のバージョンでは利用できません。

http://bugs.mysql.com/bug.php?id=20665#c92132

[10 Nov 2006 18:05] Paul DuBois

Clarification: The additional statements are supported both
for SQL syntax for prepared statements (PREPARE/EXECUTE)
and by the binary protocol. The additional supported statements
are noted in both these places:

http://dev.mysql.com/doc/refman/5.1/en/sqlps.html
http://dev.mysql.com/doc/refman/5.1/en/c-api-prepared-statements.html

http://dev.mysql.com/doc/refman/5.1/en/news-5-1-12.html

The following statements now can be executed as prepared statements (using PREPARE plus EXECUTE):

OpenPNE 3 がサポートする MySQL は 4.1 以降であると doc/ja/OpenPNE3_Setup_Guide.txt 等に明示されていますから、少なくとも安定版においては MySQL 4.1 でも動作する形での対応をおこなわなければなりません。

おそらく PREPARE 文と EXECUTE 文を使わないことには MySQL のみで対応をおこなうことは難しいと思います。今回の場合は無理して SQL のみで完結させる必要はないと思うので、 PHP 等のレベルでクエリを構築して実行するわけにはいかないでしょうか。

Mysql4.1以降で動作する機能のみを使用するように修正しました。
（SHOW TABLE STATUS はバージョン3.23で導入された機能なので問題ありませんでした）

また、MysqlのみではAuto_incrementの値を取得できないためPHPを使用するよう変更しました。

diff --git data/upgrade/2/definition.yml data/upgrade/2/definition.yml
index 7b5e651..25e83f7 100644
--- data/upgrade/2/definition.yml
+++ data/upgrade/2/definition.yml
@@ -8,7 +8,8 @@ create_tables:
   strategy: CreateTable

 member_basic:
-  strategy: SQLImport
+  strategy: opUpgradeFrom2MemberStrategy
+  file: %SF_DATA_DIR%/upgrade/2/opUpgradeFrom2MemberStrategy.class.php

 files:
   strategy: SQLImport
diff --git data/upgrade/2/opUpgradeFrom2MemberStrategy.class.php data/upgrade/2/opUpgradeFrom2MemberStrategy.class.php
new file mode 100644
index 0000000..721a0ec
--- /dev/null
+++ data/upgrade/2/opUpgradeFrom2MemberStrategy.class.php
@@ -0,0 +1,49 @@
+<?php
+
+/**
+ * This file is part of the OpenPNE package.
+ * (c) OpenPNE Project (http://www.openpne.jp/)
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file and the NOTICE file that were distributed with this source code.
+ */
+
+/**
+ * The upgrating strategy by importing member.
+ *
+ * @package    OpenPNE
+ * @subpackage task
+ * @author     tatsuya ichikawa <ichikawa@tejimaya.com>
+ */
+class opUpgradeFrom2MemberStrategy extends opUpgradeAbstractStrategy
+{
+  public function run()
+  {
+    $this->getDatabaseManager();
+    $this->conn = Doctrine_Manager::connection();
+
+    $this->conn->beginTransaction();
+    try
+    {
+      $this->doRun();
+
+      $this->conn->commit();
+    }
+    catch (Exception $e)
+    {
+      $this->conn->rollback();
+
+      throw $e;
+    }
+  }
+
+  public function doRun()
+  {
+    $this->conn->execute('INSERT INTO member (id, name, is_active, is_login_rejected, created_at, updated_at) (SELECT c_member_id, nickname, 1, is_login_rejected, r_date, u_datetime FROM c_member)');
+
+    $tableStatus = $this->conn->fetchArray('SHOW TABLE STATUS LIKE "c_member"');
+    $autoIncrement = $tableStatus[10];
+    $this->conn->execute("ALTER TABLE member AUTO_INCREMENT = $autoIncrement");
+    $this->conn->execute('UPDATE member, c_member SET invite_member_id = c_member_id_invite WHERE id = c_member_id AND c_member_id_invite <> 0;');
+  }
+}

レビュー対象となるコミットを示すか、もしくは共有リポジトリのリリースブランチにコミットを追加することによって、このチケットにコミットを紐づけてください。

動作確認OKです。