Actions
Bug（バグ） #4421
closed
push/search.json にて 500 が発生する
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 が
Way to fix (修正内容)¶
通知元メンバーが存在しない場合、通知を削除するよう修正する
Updated by kaoru n over 4 years ago
- Description updated (diff)
- Status changed from New（新規） to Accepted（着手）
- Assignee set to kaoru n
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
にてプルリクエストしました
Updated by Rimpei Ogawa over 4 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
op_api_member() 側でも 500 エラーが出ないようにしておいた方が安全なように思いました。
Updated by kaoru n over 4 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
Updated by Rimpei Ogawa over 4 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
Updated by isao sano over 4 years ago
- Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
- % Done changed from 70 to 80
試験完了しました。
問題ありません。
Updated by kaoru n over 4 years ago
- Status changed from Pending Merge（マージ待ち） to Fixed（完了）
- % Done changed from 80 to 100
マージしました
Actions