OpenPNE 3

Overview (現象)¶

ホーム画面やメンバー検索などのメンバー画像が表示されるページで、下記のように member_image に対する同一のクエリが2度実行される。

SELECT m.id AS m__id, m.member_id AS m__member_id, m.file_id AS m__file_id, m.is_primary AS m__is_primary, m.created_at AS m__created_at, m.updated_at AS m__updated_at FROM member_image m WHERE (m.member_id = '1') ORDER BY m.is_primary DESC LIMIT 1
SELECT m.id AS m__id, m.member_id AS m__member_id, m.file_id AS m__file_id, m.is_primary AS m__is_primary, m.created_at AS m__created_at, m.updated_at AS m__updated_at FROM member_image m WHERE (m.member_id = '1') ORDER BY m.is_primary DESC LIMIT 1

Causes (原因)¶

lib/model/doctrine/Member.class.php の 196-204 行目:

  public function getImageFileName()
  {
    if ($this->getImage())
    {
      return $this->getImage()->getFile();
    }

    return false;
  }

メンバー画像のファイル名取得時、上記のように $this->getImage() が2度呼び出されクエリも2回実行される。メンバー画像が設定されていない場合は if ブロック内が実行されないためこの現象は起きない。

Way to fix (修正内容)¶

Member::getImageFileName() 内で $this->getImage() を1度だけ呼び出すように修正する。