Skip to content

Commit 8c6fa3e

Browse files
committed
Conduit user.query
Summary: specify user table to make things not ambiguous Test Plan: conduit console still works, including ID query...! Reviewers: epriestley Reviewed By: epriestley CC: aran, Koolvin Maniphest Tasks: T1075 Differential Revision: https://secure.phabricator.com/D2419
1 parent 758db32 commit 8c6fa3e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/applications/people/query/PhabricatorPeopleQuery.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ private function buildWhereClause($conn_r) {
8585

8686
if ($this->usernames) {
8787
$where[] = qsprintf($conn_r,
88-
'userName IN (%Ls)',
88+
'user.userName IN (%Ls)',
8989
$this->usernames);
9090
}
9191
if ($this->emails) {
@@ -95,17 +95,17 @@ private function buildWhereClause($conn_r) {
9595
}
9696
if ($this->realnames) {
9797
$where[] = qsprintf($conn_r,
98-
'realName IN (%Ls)',
98+
'user.realName IN (%Ls)',
9999
$this->realnames);
100100
}
101101
if ($this->phids) {
102102
$where[] = qsprintf($conn_r,
103-
'phid IN (%Ls)',
103+
'user.phid IN (%Ls)',
104104
$this->phids);
105105
}
106106
if ($this->ids) {
107107
$where[] = qsprintf($conn_r,
108-
'id IN (%Ld)',
108+
'user.id IN (%Ld)',
109109
$this->ids);
110110
}
111111

0 commit comments

Comments
 (0)