プロジェクト

全般

プロフィール

Bug(バグ) #2903

完了

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

Mutsumi Imamura さんが12年以上前に追加. 約9年前に更新.

ステータス:
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 (修正内容)

修正内容を記入


関連するチケット 1 (0件未完了1件完了)

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

操作

Mutsumi Imamura さんが12年以上前に更新

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

Mutsumi Imamura さんが12年以上前に更新

  • 担当者Hidenori Goto にセット

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);

Hidenori Goto さんが12年以上前に更新

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

Hidenori Goto さんが12年以上前に更新

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

Yuya Watanabe さんが12年以上前に更新

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

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

Yuya Watanabe さんが12年以上前に更新

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

Yuya Watanabe さんが12年以上前に更新

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

Yuya Watanabe さんが12年以上前に更新

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

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

21c280693e7a31bb984af25dc2eed221d3ca4ea9

Yuma Sakata さんが12年以上前に更新

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

テストOKです。

kaoru n さんが約9年前に更新

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

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