Project

General

Profile

Bug(バグ) #2903

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

Added by Mutsumi Imamura almost 8 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-03-26
Due date:
% Done:

100%

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

Description

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 (修正内容)

修正内容を記入


Related issues

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

Associated revisions

Revision 21c28069 (diff)
Added by Hidenori Goto over 7 years ago

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

History

#1 Updated by Mutsumi Imamura over 7 years ago

  • Target version set to v1.4.1

#2 Updated by Mutsumi Imamura over 7 years ago

  • Assignee set to Hidenori Goto

#3 Updated by Hidenori Goto over 7 years ago

問題箇所

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 Updated by Hidenori Goto over 7 years ago

  • Status changed from New(新規) to Accepted(着手)

#5 Updated by Hidenori Goto over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#6 Updated by Yuya Watanabe over 7 years ago

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

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

#7 Updated by Yuya Watanabe over 7 years ago

  • Project changed from opDiaryPlugin to OpenPNE 3
  • Target version deleted (v1.4.1)

#8 Updated by Yuya Watanabe over 7 years ago

  • Target version set to OpenPNE 3.6.4

#9 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

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

21c280693e7a31bb984af25dc2eed221d3ca4ea9

#10 Updated by Yuma Sakata over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#11 Updated by kaoru n over 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF