From 85b767bbdc6542782b0be5d2ed00cd89c39b43c8 Mon Sep 17 00:00:00 2001 From: Bob Trahan Date: Wed, 27 Aug 2014 12:23:55 -0700 Subject: [PATCH] Notifications - fix pager Summary: D10281 upgraded us to modern infrastructure but I think forget to set this little helper to return true. Fixes T5975. Test Plan: paged through notifications with glee Reviewers: epriestley Reviewed By: epriestley Subscribers: epriestley, Korvin Maniphest Tasks: T5975 Differential Revision: https://secure.phabricator.com/D10369 --- .../query/PhabricatorNotificationSearchEngine.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/applications/notification/query/PhabricatorNotificationSearchEngine.php b/src/applications/notification/query/PhabricatorNotificationSearchEngine.php index 4abf211594..bb9dc4e166 100644 --- a/src/applications/notification/query/PhabricatorNotificationSearchEngine.php +++ b/src/applications/notification/query/PhabricatorNotificationSearchEngine.php @@ -136,4 +136,8 @@ final class PhabricatorNotificationSearchEngine ->appendChild($view); } + public function shouldUseOffsetPaging() { + return true; + } + }