OpenPNE3 Plugins » opMessagePlugin

不具合の理由¶

• 以下テンプレートのリンク生成関数で、パラメータに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が異なる状態で
ゴミ箱に捨てた受信メッセージを閲覧しようとするとエラーが発生する