HomePhorge

Make buildPagingClauseFromMultipleColumns() safer

Description

Make buildPagingClauseFromMultipleColumns() safer

Summary: Ref T7803. Reduce the amount of code we're trusting to build SQL queries.

Test Plan:

  • Paged through results in Maniphest, Differential and Diffusion.
  • Some of the NULLable groups in Maniphest are a bit funky but this was preexisting.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T7803

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

Details

Provenance
epriestleyAuthored on Apr 11 2015, 5:39 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPa43473c4b65c: Begin formalizing query orders
Branches
Unknown
Tags
Unknown

Event Timeline