Page MenuHomePhabricator

Provide a default "loadPage()" implementation on "CursorPagedPolicyAwareQuery"
ClosedPublic

Authored by epriestley on May 24 2022, 5:08 PM.
Tags
None
Referenced Files
F13346146: D21838.diff
Fri, Jun 21, 1:34 PM
F13344974: D21838.diff
Fri, Jun 21, 9:12 AM
F13334042: D21838.largetrue.diff
Tue, Jun 18, 3:44 PM
F13332577: D21838.diff
Mon, Jun 17, 2:59 PM
F13332364: D21838.diff
Mon, Jun 17, 10:19 AM
F13331202: D21838.diff
Mon, Jun 17, 5:55 AM
F13330299: D21838.diff
Mon, Jun 17, 3:23 AM
F13329119: D21838.diff
Sun, Jun 16, 9:41 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13682. Many subclasses of "CursorPagedPolicyAwareQuery" have the same implementation of "loadPage()", and this is a sensible default behavior.

Test Plan

Examined changes to verify that all removed methods have the same behavior.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.May 24 2022, 5:08 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 24 2022, 5:19 PM
This revision was automatically updated to reflect the committed changes.