Project

General

Profile

Bug(バグ) #3546

DeletedMessage::getSendFromOrTo()がnullを返した場合、op_message_link_to_memberでエラーが発生する

Added by Maki Takahashi almost 6 years ago. Updated almost 3 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2014-01-24
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
[QA]バグ通知済:
No
3.8 で発生するか:
Yes (はい)

Description

Overview (現象)

DeletedMessage::getSendFromOrTo()がnullを返した場合、op_message_link_to_memberヘルパー使用部分で以下のエラーが発生する。

Catchable fatal error: Argument 1 passed to op_message_link_to_member() must be an instance of sfOutputEscaper, null given, called in /home/xxxx/openpne/plugins/opMessagePlugin/apps/pc_frontend/modules/message/templates/listSuccess.php on line 135 and defined in /home/xxxx/openpne/plugins/opMessagePlugin/lib/helper/opMessageHelper.php on line 19

また、「DeletedMessage::getSendFromOrTo()がnullを返した場合」とは
DeletedMessageのmessage_id、message_send_list_idともに0で登録されている場合に発生しますが
通常の操作ではどちらかに関連するidが入るはずなので、
コミュニティトピックでの報告にあるように2系からのアップグレード時に上記の様なデータが作成されてしまったのではないかと思われます。
(が、こちらについては本チケットの対象から外します)

再現方法

  1. 任意のメッセージを削除する
  2. 削除したメッセージ(DeletedMessage)のmessage_id、message_send_list_idを共に0とする(どちらかは既に0となっているはず)
  3. ゴミ箱にアクセスする

Causes (原因)

op_message_link_to_memberにタイプヒンティングを利用しているが、上記のようにsfOutputEscaperではなくnullが来ると
タイプヒントの指定を満たしていないエラーが発生してしまう。

Way to fix (修正内容)

https://github.com/martini2002jp/opMessagePlugin/pull/2 の修正にあるように
デフォルトパラメータとしてnullを使用する


Related issues

Copied to opMessagePlugin - Backport(バックポート) #3555: DeletedMessage::getSendFromOrTo()がnullを返した場合、op_message_link_to_memberでエラーが発生する Pending Testing(テスト待ち) 2014-02-06

History

#1 Updated by Maki Takahashi almost 6 years ago

  • Description updated (diff)

取り込みおよびレビュー担当します

#2 Updated by Maki Takahashi almost 6 years ago

  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)
  • 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)

#3 Updated by Maki Takahashi almost 6 years ago

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

#4 Updated by Maki Takahashi almost 6 years ago

  • Status changed from Accepted(着手) to Pending Testing(テスト待ち)
  • % Done changed from 0 to 70

a304cbe6 にて取り込みました(コミットメッセージにチケット関連付け忘れました…)。
レビュー済みです。

#5 Updated by Maki Takahashi almost 6 years ago

#6 Updated by kaoru n over 5 years ago

  • Target version changed from 2.0.0 to 2.0.0.2 alpha 2

#7 Updated by Shinichi Urabe almost 3 years ago

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

Also available in: Atom PDF