Account for preempting events on the Phrequent list view
ClosedPublic
Actions

Authored by epriestley on Aug 11 2014, 6:58 PM.

Tags

None

Referenced Files

	F14045463: D10223.diff
	Wed, Nov 13, 4:46 AM

	F14016029: D10223.diff
	Mon, Nov 4, 4:28 AM

	F14007082: D10223.id24590.diff
	Mon, Oct 28, 11:15 PM

	F14007081: D10223.id24600.diff
	Mon, Oct 28, 11:15 PM

	F13987606: D10223.id.diff
	Mon, Oct 21, 10:19 AM

	F13974386: D10223.id24590.diff
	Fri, Oct 18, 5:20 AM

	F13960028: D10223.id24600.diff
	Mon, Oct 14, 9:42 PM

	F13955204: D10223.id24590.diff
	Oct 14 2024, 12:29 AM

View All Files

Subscribers

epriestley

Details

Reviewers

hach-que
btrahan

Maniphest Tasks

T5850: Phrequent list view doesn't know about preempting events

Commits

Restricted Diffusion Commit
rP0292793d4d05: Account for preempting events on the Phrequent list view

Summary

Fixes T5850. Also fixes some logic where the wrong preempting events could be attached during a bulk query.

Test Plan

Phrequent list now shows preemption-aware times.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24590.Aug 11 2014, 6:58 PM

epriestley retitled this revision from to Account for preempting events on the Phrequent list view.

epriestley updated this object.

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

epriestley added reviewers: hach-que, btrahan.

epriestley added a task: T5850: Phrequent list view doesn't know about preempting events.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 11 2014, 6:58 PM

Harbormaster completed remote builds in B2155: Diff 24590.Aug 11 2014, 6:59 PM

btrahan accepted this revision.Aug 11 2014, 7:18 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 11 2014, 7:18 PM

Closed by commit rP0292793d4d05 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

phrequent/

query/

PhrequentSearchEngine.php

16 lines

PhrequentUserTimeQuery.php

27 lines

Diff 24600

View Options

src/applications/phrequent/query/PhrequentSearchEngine.php