プロジェクト

全般

プロフィール

Backport(バックポート) #3331

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

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

Chiharu Nakajima約11年前に追加. ほぼ5年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2012-12-06
期日:
進捗率:

100%


説明

現象

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 の問題を回避する修正を施す

関係しているリビジョン

リビジョン 2116ba29 (差分)
kaoru n約5年前に追加

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

リビジョン f893a4de (差分)
kaoru nほぼ5年前に追加

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

リビジョン 39803ce9
kaoru nほぼ5年前に追加

Merge pull request #519 from nishizoe/t-3331

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

履歴

#1 kaoru nほぼ11年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者kaoru n にセット

#2 kaoru nほぼ11年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#3 Yuya Watanabeほぼ11年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

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

#4 Yuya Watanabeほぼ11年前に更新

  • 対象バージョン を削除 (OpenPNE 3.8.6)

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

#5 Yuya Watanabeほぼ11年前に更新

  • 対象バージョンOpenPNE 3.8.x にセット

#6 kaoru n約5年前に更新

  • 親チケット#3273 にセット

#7 kaoru n約5年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

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

#8 kaoru n約5年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#10 Rimpei Ogawaほぼ5年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

レビューOKです。

#11 kaoru nほぼ5年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.32 に変更

#12 isao sanoほぼ5年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

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

#13 kaoru nほぼ5年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF