Bug（バグ） #3513: 論理削除であることを考慮していないので、PC画面から削除したメッセージが表示される - opMessagePlugin - OpenPNE Issue Tracking System

操作

リンクをコピー

Bug（バグ） #3513

完了

論理削除であることを考慮していないので、PC画面から削除したメッセージが表示される

Shinichi Urabe さんがほぼ11年前に追加. ほぼ8年前に更新.

ステータス:

Fixed（完了）

優先度:

Normal（通常）

担当者:

Shinichi Urabe

対象バージョン:

2.0.0.2 alpha 2

開始日:

2013-12-05

期日:

進捗率:

100%

予定工数:

3.6 で発生するか:

Unknown (未調査)

［QA］バグ通知済:

いいえ

3.8 で発生するか:

Unknown (未調査)

説明

現象¶

スマホ版メッセージ機能で、メッセージは論理削除であることを考慮していないため、
削除したメッセージが表示される

原因¶

論理削除であることを考慮せずにデータを表示している

修正内容¶

モデルないで論理削除であることを考慮したメッセージの出力を行う

操作

リンクをコピー

Shinichi Urabe さんがほぼ11年前に更新

ステータス を New（新規） から Pending Review（レビュー待ち） に変更
進捗率 を 0 から 50 に変更

こちらでプルリクエストをしました

https://github.com/martini2002jp/opMessagePlugin/pull/5

操作

リンクをコピー

Maki Takahashi さんが10年以上前に更新

73cbd75 にて取り込みをしました。
ざっくりとですが、PC版とスマホ版でメッセージの送受信の確認をしています。

操作

リンクをコピー

Maki Takahashi さんが10年以上前に更新

ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
進捗率 を 50 から 70 に変更

レビュー済みです。

apps/api/modules/message/templates/searchSuccess.php 内で op_api_messageの引数が間違っている(MessageSendListを渡すべきところSendMessageDataを渡している)部分については、 405150a ( #3540 内)で修正されているようですのでここではスルーします。
- テストする場合は #3540 の修正も含まれている状態で行う必要があります。
lib/model/doctrine/PluginMessageSendListTable::createLeftJoinMessageDataQuery() はLeftJoinするテーブルの正式名称はSendMessageDataなので、修正したほうがよいかも？とも思いましたが(現状でも長いメソッド名なので)メソッド名が長くなってしまうこと、意味が通じること、修正範囲が増えてしまうことを考慮してOKとしました。