Project

General

Profile

Enhancement(機能追加・改善) #2535

[optimization] メンバー画像の表示時にmember_imageに対する同じクエリが2度実行される

Added by Youichi Kimura almost 8 years ago. Updated over 7 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2011-10-20
Due date:
% Done:

100%


Description

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度だけ呼び出すように修正する。

Associated revisions

Revision da872384 (diff)
Added by Youichi Kimura almost 8 years ago

fix double call Member::getImage() (fixes #2535)

Revision 55bb2ae9 (diff)
Added by Youichi Kimura over 7 years ago

fix double call Member::getImage() (fixes #2535)

History

#1 Updated by Youichi Kimura almost 8 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 da872384f33fd7d4da60125411f6203edf1a11f2 で適用されました。

#2 Updated by Youichi Kimura over 7 years ago

  • Target version set to OpenPNE 3.7.0

#3 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#4 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • Target version changed from 252 to OpenPNE 3.8beta1
  • % Done changed from 50 to 70

コードの修正自体は単純であるため,テストは不要であると思います.

しかし, #2698 「OpenPNE3 コアに JSON API を追加する」 で指摘を行ったため, #2698 のチケットのテストが完了したことを本チケットのテスト完了とみなして大丈夫だと思います.

#5 Updated by Yuma Sakata over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テスト不要です。

Also available in: Atom PDF