Bug（バグ） #3513
論理削除であることを考慮していないので、PC画面から削除したメッセージが表示される
開始日:
2013-12-05
期日:
進捗率:
100%
3.6 で発生するか:
Unknown (未調査)
［QA］バグ通知済:
いいえ
3.8 で発生するか:
Unknown (未調査)
関係しているリビジョン
(refs #3513) Not check about deleted or draft unread message.
(refs #3513) Not show about deleted or draft message in list page.
(refs #3513) Not show about deleted or draft message in api results.
(refs #3513) Commonized processing in MessageSendListTable.
(refs #3513) set sender api user in message search API.
履歴
#1
Shinichi Urabe が10年以上前に更新
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
こちらでプルリクエストをしました
#2
Maki Takahashi が約10年前に更新
73cbd75 にて取り込みをしました。
ざっくりとですが、PC版とスマホ版でメッセージの送受信の確認をしています。
#3
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としました。
#4
Maki Takahashi がほぼ10年前に更新
- ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
- 進捗率 を 70 から 50 に変更
#6
Shinichi Urabe が7年以上前に更新
- ステータス を Rejected（差し戻し） から Fixed（完了） に変更
- 進捗率 を 50 から 100 に変更