mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-23 22:10:55 +01:00
Fix project queries in Ponder
Summary: Fixes T10167. We were dropping infrastructure joins. Test Plan: Queried for questions by project. Reviewers: chad Reviewed By: chad Maniphest Tasks: T10167 Differential Revision: https://secure.phabricator.com/D15043
This commit is contained in:
parent
7ab970d1a7
commit
495e1384dd
1 changed files with 3 additions and 3 deletions
|
@ -129,13 +129,13 @@ final class PonderQuestionQuery
|
||||||
return $questions;
|
return $questions;
|
||||||
}
|
}
|
||||||
|
|
||||||
protected function buildJoinClauseParts(AphrontDatabaseConnection $conn_r) {
|
protected function buildJoinClauseParts(AphrontDatabaseConnection $conn) {
|
||||||
$joins = array();
|
$joins = parent::buildJoinClauseParts($conn);
|
||||||
|
|
||||||
if ($this->answererPHIDs) {
|
if ($this->answererPHIDs) {
|
||||||
$answer_table = new PonderAnswer();
|
$answer_table = new PonderAnswer();
|
||||||
$joins[] = qsprintf(
|
$joins[] = qsprintf(
|
||||||
$conn_r,
|
$conn,
|
||||||
'JOIN %T a ON a.questionID = q.id AND a.authorPHID IN (%Ls)',
|
'JOIN %T a ON a.questionID = q.id AND a.authorPHID IN (%Ls)',
|
||||||
$answer_table->getTableName(),
|
$answer_table->getTableName(),
|
||||||
$this->answererPHIDs);
|
$this->answererPHIDs);
|
||||||
|
|
Loading…
Reference in a new issue