プロジェクト

全般

プロフィール

Bug(バグ) #4421

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

kaoru n約4年前に追加. ほぼ4年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2020-03-27
期日:
進捗率:

100%

3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Yes (はい)

説明

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 (修正内容)

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


子チケット

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

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

関係しているリビジョン

リビジョン d9140ff5 (差分)
kaoru n約4年前に追加

(fixed #4421) push/search.json にて退会したメンバーからの通知の場合に 500 が発生するのを修正

リビジョン 3031c17a (差分)
kaoru nほぼ4年前に追加

(fixed #4421) op_api_member() の引数に null が渡された場合に null を返却するよう変更

リビジョン 1fa1b64b
kaoru nほぼ4年前に追加

Merge pull request #587 from nishizoe/t-4421

(fixed #4421) push/search.json にて退会したメンバーからの通知の場合に 500 が発生するのを修正

履歴

#1 kaoru n約4年前に更新

  • 説明 を更新 (diff)

#2 kaoru n約4年前に更新

  • 説明 を更新 (diff)
  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者kaoru n にセット

#3 kaoru n約4年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#4 kaoru nほぼ4年前に更新

  • 説明 を更新 (diff)

#5 kaoru nほぼ4年前に更新

  • 説明 を更新 (diff)

#6 Rimpei Ogawaほぼ4年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

#7 kaoru nほぼ4年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

#8 Rimpei Ogawaほぼ4年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#9 isao sanoほぼ4年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

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

#10 kaoru nほぼ4年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF