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 に変更