Ticket #3484 (closed defect: fixed)

Opened 1 year ago

Last modified 1 year ago

ホーム表示時にplugins/opBlogPlugin/lib/model/BlogPeer.phpエラー

Reported by: tsukamoto Assigned to: nagasawa
Priority: major Milestone: OpenPNE3.0.0
Component: plugins Version: 3.0.x
Keywords: opBlogPlugin Cc:

Description

OpenPNE3 rev.10451で、自分が招待して未登録の人がいる時、ホームを開こうとすると以下のエラーが発生します。 

Fatal error: Call to a member function getConfig() on a non-object in
C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\OpenPNE3
\plugins\opBlogPlugin\lib\model\BlogPeer.php on line 57

該当行を含むメソッドgetBlogListByMemberId($member_id, &$list)に$member_id=2が渡ってきており、これは招待を出した、未登録の人でした。この時、以下の部分で、まずMemberPeer::retrieveByPk($member_id)が失敗し、それをケアせずに次行でオブジェクトメソッドを呼ぼうとしてエラーになっているようです。 

 public static function getBlogListByMemberId($member_id, &$list)
 {
   $member = MemberPeer::retrieveByPk($member_id);
   $root = self::getXmlByUrl($member->getConfig('blog_url'));

$memberが空の時にreturnさせれば、期待通りの動きではないかと思います。rev.10451用のパッチを添付します。

Attachments

BlogPeer.php.rev10451.diff (0.5 kB) - added by tsukamoto on 01/27/09 01:48:26.

Change History

01/27/09 01:48:26 changed by tsukamoto

  • attachment BlogPeer.php.rev10451.diff added.

01/27/09 02:36:58 changed by tsukamoto

追記ですが、member_relationshipを見てみると、招待しただけのメンバーとの関係で、is_friend_preではなくis_friendが1になっていました。 

mysql> select member_id_to, member_id_from, is_friend, is_friend_pre
    -> from member_relationship;
+--------------+----------------+-----------+---------------+
| member_id_to | member_id_from | is_friend | is_friend_pre |
+--------------+----------------+-----------+---------------+
|            2 |              1 |         1 |             0 |
|            1 |              2 |         1 |             0 |
+--------------+----------------+-----------+---------------+

もしかするとBlogの取得でエラーになることより、ここでis_friendが1にセットされているのがおかしいのかもしれません。その場合、上記箇所及びパッチの問題ではなく、plugins/opInviteFriendPlugin/lib/model/InviteFriendPeer.phpの以下の個所の最後の行がおかしいということになるかもしれません。 

  public static function getNotFriendMembers($send_id, $target_id)
  {
    $relations = MemberRelationshipPeer::retrievesByMemberIdFrom($send_id);
    $c = new Criteria();
    $c->add(MemberRelationshipPeer::MEMBER_ID_FROM, $send_id);
    $c->add(MemberRelationshipPeer::MEMBER_ID_TO, $target_id, Criteria::NOT_EQUAL);
    $c->add(MemberRelationshipPeer::IS_FRIEND, true);

どちらが正しいのか、ここは設計の決めの問題ですので、ご判断ください。

01/27/09 04:27:24 changed by ebihara

  • version set to 3.0.x.
  • milestone set to OpenPNE3.0beta4.

ご報告ありがとうございます、確認します。

member_relationship の is_friend が 1 になっているのは正しい挙動です。

招待者と新規登録者はSNS登録時からフレンド関係にある必要があるため、このような状態になっています。 is_friend が 1 であったとしても、新規登録者の member.is_active が 0 であるため、問題にはなりません。

01/27/09 10:24:29 changed by ogawa

  • milestone deleted.

Milestone OpenPNE3.0beta4 deleted

01/27/09 10:34:13 changed by ogawa

  • keywords set to opBlogPlugin.
  • priority changed from minor to major.
  • component changed from 指定しない to plugins.
  • milestone set to OpenPNE3.0.0.

01/27/09 10:52:17 changed by nagasawa

  • owner changed from nobody to nagasawa.

ランキングについても同様に問題がありましたの修正します。 報告ありがとうございました。

01/27/09 10:58:30 changed by nagasawa

  • status changed from new to closed.
  • resolution set to fixed.

r10464 で修正しました。