Bug（バグ） #3513
完了
論理削除であることを考慮していないので、PC画面から削除したメッセージが表示される
Shinichi Urabe さんがほぼ11年前に追加.
ほぼ8年前に更新.
説明
スマホ版メッセージ機能で、メッセージは論理削除であることを考慮していないため、
削除したメッセージが表示される
論理削除であることを考慮せずにデータを表示している
修正内容¶
モデルないで論理削除であることを考慮したメッセージの出力を行う
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
73cbd75 にて取り込みをしました。
ざっくりとですが、PC版とスマホ版でメッセージの送受信の確認をしています。
- ステータス を 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としました。
- ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
- 進捗率 を 70 から 50 に変更
#3539 のレビューをしていて、( #3539 内で該当の実装は削除されてしまうのですが)
フィードバックすべき点に気がつきました。
a7095189 にて smtChainアクションにアクセスした時点でユーザ間のメッセージを全て既読としていますが
ページャーにて画面表示されていないメッセージの場合も(読んでないにも関わらず)既読となってしまいます。
#3573 のメッセージ既読機能が実装された場合、「実際にはそのメッセージを見ていないのに、既読として表示される」
現象が起きてしまいますので差し戻しとします。
- 対象バージョン を 2.0.0 から 2.0.0.2 alpha 2 に変更
- ステータス を Rejected（差し戻し） から Fixed（完了） に変更
- 進捗率 を 50 から 100 に変更
他の形式にエクスポート: Atom
PDF
