プロジェクト

全般

プロフィール

Bug(バグ) #2903

OpenPNE2系からコンバートした環境で日記コメントに添付した画像が表示されない現象が発生する場合がある

Mutsumi Imamura約12年前に追加. 8年以上前に更新.

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

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

Overview (現象)

OpenPNE2系からコンバートした環境で日記コメントに添付した画像が表示されない現象が発生する場合がある。
コンバート後のDBを確認すると、diary_comment テーブルの has_images フラグが0になっているため表示されない。
(恐らく、この分岐(has_images)はコンバーターが作られたときは無く、現在のコンバータで考慮されていないためこの問題が発生したと思われる)

コンバーターはOpenPNEコア側の機能なので判断に迷いましたが、こちらにチケットを作成しました。
opDiaryPluginプロジェクトにチケットを作成するのが不適切でしたら、お手数ですがOpenPNE3コアのプロジェクトにチケットの移動をお願いいたします。

再現バージョン

  • OpenPNE3.6.2
  • opDiaryPlugin1.4.0.1

再現手順

  1. 2系のデータで画像添付ありの日記コメントを作成する
  2. 手順1で作成したデータをコンバートする
  3. コンバート後、手順1で作成した日記にアクセスする
  4. 画像が表示されない

Causes (原因)

バグが発生した原因を記入

Way to fix (修正内容)

修正内容を記入


関連するチケット

関連している opDiaryPlugin - Bug(バグ) #3071: OpenPNE2系からOpenPNE 3.6.4 より前のバージョンへコンバートした環境で日記コメントに添付した画像が表示されない現象が発生する場合がある Fixed(完了) 2012-06-07

関係しているリビジョン

リビジョン 21c28069 (差分)
Hidenori Gotoほぼ12年前に追加

(refs #2903) Fixed converter SQL for diary_comment to set has_images value.

履歴

#1 Mutsumi Imamuraほぼ12年前に更新

  • 対象バージョンv1.4.1 にセット

#2 Mutsumi Imamuraほぼ12年前に更新

  • 担当者Hidenori Goto にセット

#3 Hidenori Gotoほぼ12年前に更新

問題箇所

overviewにあるとおり、コンバータの data/upgrade/2/sql/diary.sql でV2側のc_diary_commentからV3のdiary_commentへレコードを変換する際に、diary_commentのhas_imagesフィールドを設定する処理がない。

修正方針

  • data/upgrade/2/sql/diary.sqlにて、コメントごとの画像有無判定コードを追加する。
INSERT INTO diary_comment (id, diary_id, member_id, number, body, created_at, updated_at, has_images)
  (SELECT c_diary_comment_id, c_diary_id, c_member_id, number, body, r_datetime, r_datetime,
    if(image_filename_1 <> "" or image_filename_2 <> "" or image_filename_3 <> "", 1, 0)
      FROM c_diary_comment);

#4 Hidenori Gotoほぼ12年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#5 Hidenori Gotoほぼ12年前に更新

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

#6 Yuya Watanabeほぼ12年前に更新

この問題では,すでにコンバートが行われているOpenPNEでは解決が難しいでしょうか?日記コメントは編集が不可能のため日記コメントの画像が増えることはないので,コメントの画像が増えることはなく,日記コメントに紐付く画像が note-5 で示していただいたように member_comment_image をなめることができるのならば,管理画面などで画像が削除されている場合にも問題なさそうであるため,マイグレートスクリプトで対応できそうな気がしています.

このチケットも https://redmine.openpne.jp/issues/2900#note-7 と同様に,本体側の修正であるため プロジェクトを移動します.また,マイグレートによる対応が可能ならば opDiaryPlugin 側で別チケットを作成し,そちらでマイグレートについて対応したいと思います.

#7 Yuya Watanabeほぼ12年前に更新

  • プロジェクトopDiaryPlugin から OpenPNE 3 に変更
  • 対象バージョン を削除 (v1.4.1)

#8 Yuya Watanabeほぼ12年前に更新

  • 対象バージョンOpenPNE 3.6.4 にセット

#9 Yuya Watanabeほぼ12年前に更新

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

コンバート時の修正についてはレビューOKです. 下記コミットで stable-3.6.x ブランチにマージを行いました.

21c280693e7a31bb984af25dc2eed221d3ca4ea9

#10 Yuma Sakataほぼ12年前に更新

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

テストOKです。

#11 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

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