1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-19 13:22:42 +01:00

Fix a fatal on the application list

Summary:
Fixes T7816. This just punts adding proper cursor-based paging to Applications for now, since they don't have a handy order.

If we get to 101, we can either fix this properly or change the SearchEngine to return 200 results.

(Previously, we generated a cursor only if we absolutely needed to, so this code wasn't called. We generate cursors in some cases where we may not need them now, but the code is simpler this way.)

Test Plan: `/applications/` no longer fatals.

Reviewers: chad, btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T7816

Differential Revision: https://secure.phabricator.com/D12403
This commit is contained in:
epriestley 2015-04-13 14:51:00 -07:00
parent 281bdf87cf
commit ff7215f3d6

View file

@ -170,4 +170,10 @@ final class PhabricatorApplicationQuery
return null;
}
protected function getResultCursor($object) {
// TODO: This won't work, but doesn't matter until we write more than 100
// applications. Since we only have about 70, just avoid fataling for now.
return null;
}
}