Project

General

Profile

Backport(バックポート) #3331

Bug(バグ) #3273: activity/search.json?member_id=? で特定メンバーのアクティビティを取得すると500エラーとなる

activity/search.json?member_id=? で特定メンバーのアクティビティを取得すると500エラーとなる

Added by Chiharu Nakajima over 11 years ago. Updated over 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-12-06
Due date:
% Done:

100%


Description

現象

MySQL を使用している SNS で、API を使用して http://sns.example.com/api.php/activity/search.json?member_id=1 のように特定メンバーのアクティビティを取得しようとすると Internal Server Error が返ってしまう場合がある。

原因

opActivityQueryBuilder::buildMemberQuery() 内で andWhereIn メソッドを使用している。ここで生成されるクエリはサブクエリとして使用されるため #3135 で報告されているように不正なクエリが生成されてしまう。

  protected function buildMemberQuery($query, $memberId = null, $publicFlag = ActivityDataTable::PUBLIC_FLAG_SNS)
  {
    if (is_array($memberId))
    {   
      $query->andWhereIn('a.member_id', $memberId);
    }   
    elseif ($memberId instanceof Doctrine_Query)
    {   
      $query->andWhere('a.member_id IN ('.$memberId->getDql().')');
    }   
    elseif (is_scalar($memberId))
    {   
      $query->andWhere('a.member_id = ?', $memberId);
    }   

    $query->andWhereIn('a.public_flag', $this->table->getViewablePublicFlags($publicFlag));

    return $query;
  }

修正内容

opActivityQueryBuilder::buildMemberQuery() メソッドに #3135 の問題を回避する修正を施す

Associated revisions

Revision 2116ba29 (diff)
Added by kaoru n over 5 years ago

(refs #3331, BP from #3273) public_flag の数によって addWhere または addWhereIn の使用を分岐するよう修正

Revision f893a4de (diff)
Added by kaoru n over 5 years ago

(refs #3331, BP from #3273) 定義した変数を使用するよう修正

Revision 39803ce9
Added by kaoru n over 5 years ago

Merge pull request #519 from nishizoe/t-3331

(refs #3331, BP from #3273) public_flag の数によって addWhere または addWhereIn の使用を分岐するよう修正

History

#1 Updated by kaoru n over 11 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to kaoru n

#2 Updated by kaoru n over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/78
プルリクエストしました。

#3 Updated by Yuya Watanabe over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

親チケットを差し戻したので同様の理由により本チケットも差し戻しを行います.

https://redmine.openpne.jp/issues/3273#note-6

#4 Updated by Yuya Watanabe over 11 years ago

  • Target version deleted (OpenPNE 3.8.6)

修正の目処が立っていないため一旦バージョンを外します.

#5 Updated by Yuya Watanabe over 11 years ago

  • Target version set to OpenPNE 3.8.x

#6 Updated by kaoru n over 5 years ago

  • Parent task set to #3273

#7 Updated by kaoru n over 5 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

http://redmine.openpne.jp/issues/3273#note-12 に記載の回避のための修正を行うため、
https://github.com/openpne/OpenPNE3/pull/78
は close します。

#8 Updated by kaoru n over 5 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#10 Updated by Rimpei Ogawa over 5 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです。

#11 Updated by kaoru n over 5 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.32

#12 Updated by isao sano over 5 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

確認しました。問題ありません。

#13 Updated by kaoru n over 5 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF