Project

General

Profile

Actions

Bug(バグ) #4421

closed

push/search.json にて 500 が発生する

Added by kaoru n over 4 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2020-03-27
Due date:
% Done:

100%

Estimated time:
(Total: 0:00 h)
3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Yes (はい)

Description

Overview (現象)

通知センターにアクセスした際、push/search.json にて 500 が発生する。

PHP Fatal error:  Call to a member function getImageFileName() on boolean in (snip)/apps/api/lib/helper/opJsonApiHelper.php on line 22

■OpenPNE バージョン
3.8.35

■アプリケーションプラグイン
opLikePlugin-1.2.8

■スキンプラグイン
opSkinThemePlugin-1.0.15
にバージョンアップする前からいるメンバーでのみ発生する模様。
バージョンアップ後に登録したメンバーでは発生しない。

Causes (原因)

https://github.com/openpne/OpenPNE3/blob/stable-3.8.x/apps/api/lib/helper/opJsonApiHelper.php#L18

function op_api_member($member)

https://github.com/openpne/OpenPNE3/blob/master/apps/api/lib/helper/opJsonApiHelper.php#L205

    'member_from' => op_api_member($fromMember)

$fromMember が すでに退会済みのメンバーである 取得できない場合に 500 が発生する。

Way to fix (修正内容)

通知元メンバーが存在しない場合、通知を削除するよう修正する


Subtasks 2 (0 open2 closed)

Backport(バックポート) #4422: push/search.json にて 500 が発生するFixed(完了)kaoru n2020-03-27

Actions
Backport(バックポート) #4423: push/search.json にて 500 が発生するFixed(完了)kaoru n2020-03-27

Actions
Actions #1

Updated by kaoru n over 4 years ago

  • Description updated (diff)
Actions #2

Updated by kaoru n over 4 years ago

  • Description updated (diff)
  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to kaoru n
Actions #3

Updated by kaoru n over 4 years ago

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

https://github.com/openpne/OpenPNE3/pull/587
にてプルリクエストしました

Actions #4

Updated by kaoru n over 4 years ago

  • Description updated (diff)
Actions #5

Updated by kaoru n over 4 years ago

  • Description updated (diff)
Actions #6

Updated by Rimpei Ogawa over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

op_api_member() 側でも 500 エラーが出ないようにしておいた方が安全なように思いました。

Actions #7

Updated by kaoru n over 4 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
Actions #8

Updated by Rimpei Ogawa over 4 years ago

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

Updated by isao sano over 4 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

試験完了しました。
問題ありません。

Actions #10

Updated by kaoru n over 4 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Actions

Also available in: Atom PDF