Project

General

Profile

Bug(バグ) #1757

message と message_send_list のレコードの登録順が異なっている場合にメッセージが閲覧できない

Added by Y Kimura over 8 years ago. Updated about 8 years ago.

Status:
Fixed(完了)
Priority:
Urgent(急いで)
Target version:
Start date:
2010-11-04
Due date:
% Done:

100%

3.6 で発生するか:
Yes
[QA]バグ通知済:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

不具合の理由

  • 以下テンプレートのリンク生成関数で、パラメータにmessege_idを渡すところにsend_message_idが渡されている
    • plugins/opMessagePlugin/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php

      • <?php echo sprintf('%s (%s)',
          link_to(op_truncate($message->getSubject(), 28), '@readReceiveMessage?id='. $message->getId()),
          ($message->getSendFrom()->getId()) ? $message->getSendFrom()->getName() : ''
        ); ?>
        

      • <?php echo sprintf('%s (%s)',
          link_to(op_truncate($message->getSubject(), 28), '@readReceiveMessage?id='. $message->getMessageId()),
          ($message->getSendFrom()->getId()) ? $message->getSendFrom()->getName() : ''); ?>
        
        • 生成されるリンクに誤りがあったが、全レコードで不具合にならない理由は、messege_id と send_message_id は基本的に同じ値である為、エラーとなっていない
        • メッセージ作成処理が重なった、もしくはDBが高負荷などの理由で、message と message_send_list のレコードの登録順が異なっている場合(messege_id と send_message_id が異なっている場合)にidがずれ、エラーとなる。
修正パッチ
Index: plugins/opMessagePlugin/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php
===================================================================
--- plugins/opMessagePlugin/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php
+++ plugins/opMessagePlugin/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php
@@ -7,6 +7,6 @@
 <font color="#FF0000">(<?php echo __('Unopened') ?>)</font>
 <?php endif; ?><br>
 <?php echo sprintf('%s (%s)',
-  link_to(op_truncate($message->getSubject(), 28), '@readReceiveMessage?id='. $message->getId()),
+  link_to(op_truncate($message->getSubject(), 28), '@readReceiveMessage?id='. $message->getMessageId()),
   ($message->getSendFrom()->getId()) ? $message->getSendFrom()->getName() : ''
 ); ?>

※原因は異なりますが(携帯版、PC版両方で)
messageのidとmessage_send_listのidが異なる状態で
ゴミ箱に捨てた受信メッセージを閲覧しようとするとエラーが発生する


Related issues

Related to opMessagePlugin - Backport(バックポート) #2053: message と message_send_list のレコードの登録順が異なっている場合にメッセージが閲覧できない Pending Testing(テスト待ち) 2011-05-05
Related to opMessagePlugin - Bug(バグ) #1162: 携帯版の受信メッセージリストの詳細リンクのメッセージidが間違っている Invalid(無効) 2010-06-14

Associated revisions

Revision 591abf50 (diff)
Added by Maki Takahashi about 8 years ago

fixed invalid reference id (refs #1757)

History

#1 Updated by Y Kimura over 8 years ago

  • Assignee set to Kousuke Ebihara

#2 Updated by Kousuke Ebihara over 8 years ago

  • Assignee deleted (Kousuke Ebihara)

#3 Updated by Kousuke Ebihara over 8 years ago

  • Priority changed from Urgent(急いで) to High(高め)

#4 Updated by Kousuke Ebihara over 8 years ago

  • Priority changed from High(高め) to Urgent(急いで)

#5 Updated by Kousuke Ebihara over 8 years ago

  • [QA]バグ通知済 set to No

#6 Updated by Kousuke Ebihara over 8 years ago

  • [QA]バグ通知済 changed from No to Yes

#7 Updated by Shingo Yamada about 8 years ago

  • Assignee set to Maki Takahashi

#8 Updated by Maki Takahashi about 8 years ago

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

#9 Updated by Maki Takahashi about 8 years ago

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

https://github.com/martini2002jp/opMessagePlugin/commit/591abf509a8da914aa5ba09b77915237d1f45cbe にて修正いたしました。

報告いただいたモバイル版での受信メッセージの確認の他、ゴミ箱に捨てられた受信メッセージの表示でも同様の現象が起きていました(PC/モバイル両方)。
messageのidとmessage_send_listのidが異なる状態で、
受信、送信、下書き、ゴミ箱(送信⇒ゴミ/受信⇒ゴミ)のそれぞれのメッセージ閲覧が正常にできることを
モバイル、PC版それぞれで確認いたしました。

レビューよろしくお願いいたします。

#10 Updated by Shingo Yamada about 8 years ago

  • Target version changed from 0.9.1 to 0.9.2

#11 Updated by Naoya Tozuka about 8 years ago

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

修正点確認しました。ロジック的にもコーディング規約的にも問題ありませんのでレビューOKとします。

【メモ】(SendMessageDataクラスだけでなく)MessageSendListクラスのオブジェクトも $message という名前の変数に格納する実装は変数名からオブジェクトの種類が類推しにくいため、将来的にバグの温床となりえます。機会がありましたらリファクタリングしてみて下さい。

#12 Updated by Yuma Sakata about 8 years ago

テストOKです。

#13 Updated by Yuma Sakata about 8 years ago

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

Also available in: Atom PDF