Actions
Bug(バグ) #4421
closedpush/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