プロジェクト

全般

プロフィール

Bug(バグ) #2433

1.0.2 で追加されたマイグレートスクリプト(リビジョン番号 6)の考慮不足により、マイグレートに失敗することがある

Kousuke Ebihara12年以上前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-09-26
期日:
進捗率:

100%

3.6 で発生するか:
Yes
[QA]バグ通知済:
いいえ
3.8 で発生するか:
Unknown (未調査)
opCommunityTopicPlugin 絞込条件:

説明

#1483 で、新規インストール時には追加されるが過去バージョンからのマイグレート時には追加されなかった post_id および file_id の外部キー制約(とインデックス)の追加をおこない、過去バージョンからのマイグレートをおこなった後の DB 構造と新規インストール時の DB 構造を一致させるためのマイグレートスクリプト 006_update_op_community_topic_plugin_1_0_2.php が追加されたが、このスクリプトの実装は考慮が不足しており、マイグレートに失敗することがある。

post_id や file_id が存在しなかった過去バージョンからマイグレートをおこなった環境ではこのマイグレートスクリプトは成功するが、このマイグレートスクリプトによって追加しようとしている外部キーやインデックスが既に存在する状態を考慮していないため、 post_id や file_id が追加されたあとのバージョンを新規にインストールして運用している SNS でこのマイグレートスクリプトを実行すると失敗してしまう。

post_id や file_id の外部キー制約などが既に存在する場合、このマイグレートスクリプトの処理を実行する必要はないため、マイグレートスクリプト内で post_id や file_id の外部キーなどの存在確認をおこなうように修正する。


関連するチケット

関連している opCommunityTopicPlugin - Bug(バグ) #2449: 1.0.2 で追加されたマイグレートスクリプト(リビジョン番号 6)で外部キー制約違反により、マイグレートに失敗することがある Fixed(完了) 2011-09-30

履歴

#1 Yuya Watanabe12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Yuya Watanabe にセット

#2 Yuya Watanabe12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

下記コミットで修正しました.外部キーのインデックスを確認することで存在確認を行なっています.

https://github.com/tejimaya/opCommunityTopicPlugin/commit/7cc6441752d01fb71b706b52849883eacc0f48b0

#3 Kousuke Ebihara12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#4 Yuya Watanabe12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率70 から 50 に変更

definitionの位置を逆にしていたので入れ替えを行いました.

https://github.com/tejimaya/opCommunityTopicPlugin/commit/a57e54ef93122160e4f70c11895c645a9bdbf5da

#5 Kousuke Ebihara12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#6 Kousuke Ebihara12年以上前に更新

  • 360対象3.6.0 から RC2 に変更

#7 Mutsumi Imamura12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

動作確認しました。OKです。

他の形式にエクスポート: Atom PDF