LEFT JOIN instead of JOIN for ApplicationSearch ordering
ClosedPublic
Actions

Authored by epriestley on Aug 2 2014, 8:35 AM.

Tags

None

Referenced Files

	F14034963: D10119.diff
	Sun, Nov 10, 3:11 AM

	F14025363: D10119.id24340.diff
	Thu, Nov 7, 5:04 PM

	F14021601: D10119.diff
	Wed, Nov 6, 10:05 AM

	F14014147: D10119.diff
	Sat, Nov 2, 11:28 PM

	F14013634: D10119.id24340.diff
	Sat, Nov 2, 8:09 AM

	F14004446: D10119.id24340.diff
	Sat, Oct 26, 8:41 PM

	F13990122: D10119.id24349.diff
	Tue, Oct 22, 1:23 AM

	F13982428: D10119.diff
	Sat, Oct 19, 11:13 PM

View All Files

Subscribers

epriestley

Details

Reviewers

hach-que
btrahan

Maniphest Tasks

T4659: Cannot use custom "date" field for sorting

Commits

Restricted Diffusion Commit
rPeafd7070badb: LEFT JOIN instead of JOIN for ApplicationSearch ordering

Summary

Ref T4659. Because we JOIN, tasks with no value are filtered out. Instead, LEFT JOIN.

Test Plan

Issued an "Order by" and got all applicable tasks. Adjusted values and saw order change.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24340.Aug 2 2014, 8:35 AM

epriestley retitled this revision from to LEFT JOIN instead of JOIN for ApplicationSearch ordering.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: hach-que, btrahan.

epriestley added a task: T4659: Cannot use custom "date" field for sorting.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 2 2014, 8:35 AM

Harbormaster completed remote builds in B1999: Diff 24340.Aug 2 2014, 8:36 AM

hach-que accepted this revision.Aug 2 2014, 12:04 PM

hach-que edited edge metadata.

This revision is now accepted and ready to land.Aug 2 2014, 12:04 PM

Closed by commit rPeafd7070badb (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

query/

policy/

PhabricatorCursorPagedPolicyAwareQuery.php

2 lines

Diff 24349

View Options

src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php

LEFT JOIN instead of JOIN for ApplicationSearch orderingClosedPublicActions