HomePhabricator

Give ConpherenceThreadQuery a primary table alias

Description

Give ConpherenceThreadQuery a primary table alias

Summary: Fixes T11113. On the 2nd+ page, we could end up with an ambiguous id WHERE clause because we don't define a primary table alias on this query. Define one.

Test Plan:
Changed SearchEngine to return pages of size 5, searched for my threads, toggled to second page, no exception.

Used DarkConsole to examine that second-page query, saw that it had thread.id explicitly instead of id implicitly.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11113

Differential Revision: https://secure.phabricator.com/D16080

Details

Provenance
epriestleyAuthored on Jun 8 2016, 2:33 PM
epriestleyPushed on Jun 8 2016, 2:39 PM
Reviewer
chad
Differential Revision
D16080: Give ConpherenceThreadQuery a primary table alias
Parents
rPc71f92a1eba7: Fix new Countdown link
Branches
Unknown
Tags
Unknown
Tasks
T11113: Query error in Conpherence when searching for specific user
Build Status
Buildable 12563
Build 15941: Run Core Tests