Page MenuHomePhabricator
Differential D8121

Allow ApplicationSearch to be offset-paged
ClosedPublic
Actions

Authored by epriestley on Jan 31 2014, 9:09 PM.
Tags
None
Referenced Files
F15477170: D8121.id18369.diff
Mon, Apr 7, 12:50 PM
F15470524: D8121.diff
Fri, Apr 4, 11:15 PM
F15461919: D8121.id18369.diff
Tue, Apr 1, 12:20 PM
F15433748: D8121.diff
Tue, Mar 25, 12:28 AM
F15418878: D8121.id18368.diff
Fri, Mar 21, 2:03 AM
F15393062: D8121.id.diff
Sat, Mar 15, 7:20 PM
F15382880: D8121.diff
Fri, Mar 14, 2:49 PM
F15367543: D8121.diff
Tue, Mar 11, 7:14 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4365: Use ApplicationSearch in primary search
Commits
Restricted Diffusion Commit
rPa82f57310230: Allow ApplicationSearch to be offset-paged
Summary

Ref T4365. It's not practical to cursor-page all engines; allow main search engines to be offset-paged. Basically, this comes down to setting a flag and then doing a couple of tiny things differently.

Test Plan

Used this two diffs from now.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 31 2014, 9:09 PM
  • Correct diff.
btrahan accepted this revision.Feb 1 2014, 9:56 PM
epriestley closed this revision.Feb 3 2014, 8:52 PM

Closed by commit rPa82f57310230 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
search/
controller/
27 lines
engine/
2 lines
index/
11 lines
view/
control/
11 lines

Diff 18405

View Options

src/applications/search/controller/PhabricatorApplicationSearchController.php

Loading...
View Options

src/applications/search/engine/PhabricatorApplicationSearchEngine.php

Loading...
View Options

src/applications/search/index/PhabricatorSearchIndexer.php

Loading...
View Options

src/view/control/AphrontPagerView.php

Loading...
Phabricator · Built by Phacility