Bug（バグ） #2310
[PATCH] アクセスランキングに退会者が含まれる場合、アクセスランキング画面で Fatal error が発生する
100%
説明
アクセスランキングに退会者が含まれる場合、アクセスランキング画面 ( /ranking/access ) で以下のようなエラーが発生する。
Fatal error: Call to a member function getName() on a non-object in /OPENPNE_DIR/plugins/opRankingPlugin/apps/pc_frontend/modules/ranking/templates/_access.php on line 8確認バージョン¶
- OpenPNE 3.6beta12
- opRankingPlugin 1.0.0.1
関連するチケット
履歴
#1
Kiwa Sakai がほぼ13年前に更新
- ファイル opRankingPlugin.patch 表示 を追加
- 題名 を アクセスランキングに退会者が含まれる場合、アクセスランキング画面で Fatal error が発生する から [PATCH] アクセスランキングに退会者が含まれる場合、アクセスランキング画面で Fatal error が発生する に変更
修正案のパッチを添付します。
#2
Shingo Yamada が12年以上前に更新
- 優先度 を Normal（通常） から High（高め） に変更
#3
Shingo Yamada が12年以上前に更新
- 360対象 を RC1 にセット
#4
Shingo Yamada が12年以上前に更新
- 担当者 を Yuya Watanabe にセット
#5
Yuya Watanabe が12年以上前に更新
- ステータス を New（新規） から Accepted（着手） に変更
#6
Yuya Watanabe が12年以上前に更新
修正案でいただいているPATCHでFatal Errorは回避できるものの，順位の表示に関して不整合が発生する可能性があるので対処を行います．
#7
Yuya Watanabe が12年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
修正内容を下記pull requestで行いました．
#8
Kousuke Ebihara が12年以上前に更新
pull request にコメントしました。確認をお願いします。
$ashiato['member_id_to'] に該当するプライマリキーを持つレコードを取得しているのだから、 !$member が true になることはあっても !$member->getId() が true になることはないと思います。これは何を意図したコードでしょうか。
#9
Kousuke Ebihara が12年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
#10
Yuya Watanabe が12年以上前に更新
PATCHの内容は下記コミットを参考に作ったと思われます．
https://github.com/nothan/opRankingPlugin/commit/43218a74d4156f632d290442bc39e63359ce4978
ashiatoテーブルにmember_id_toの値がNULLとなるレコードを追加して動作を見るという確認をした限りでは，確かに!$memberがtrueになり，!$member->getId()がtrueになるものは存在しませんでした．よって条件!$member->getId()は削除する方向でいきます．
別件となりますが，上記コミットでの修正においてもPATCHと同様に，順位のインクリメントが行われた後に存在確認が行われています．
そのため，本チケットで対処した「順位の表示に関して不整合が発生する可能性」が存在するものと思われます．
#11
Yuya Watanabe が12年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
下記コミットでnote-8およびnote-10の対応を行いました．
https://github.com/nise-nabe/opRankingPlugin/commit/bc16082f59c39404e521f6513cddda72ce9e092a
#12
Kousuke Ebihara が12年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 対象バージョン を v1.0.1 にセット
- 進捗率 を 50 から 70 に変更
master に取り込みました。
#13
Yuma Sakata が12年以上前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テストOKです。