(stable) Fix an issue with "nextPage()" on worker trigger queries
Audit Required3d481fb75772
Actions

Description

(stable) Fix an issue with "nextPage()" on worker trigger queries

Ref T13266. We never page these queries, and previously never reached the
"nextPage()" method. The call order changed recently and this method is now
reachable. For now, just no-op it rather than throwing.

Details

Group Auditors

Cowboy Commits

Provenance

epriestley	Authored on Mar 23 2019, 5:41 PM
epriestley	Pushed on Mar 23 2019, 5:43 PM

Parents

rP7477da3d4f5b: (stable) Promote 2019 Week 12

Branches

Unknown

Tags

Unknown

Tasks

T13266: Upgrading: Minor Changes to Query Paging / Cursors

Build Status

Buildable 22339
Build 30565: Run Core Tests

Event Timeline

epriestley committed rP3d481fb75772: (stable) Fix an issue with "nextPage()" on worker trigger queries (authored by epriestley).Mar 23 2019, 5:43 PM

epriestley added a task: T13266: Upgrading: Minor Changes to Query Paging / Cursors.

Herald added an auditor: Cowboy Commits. · View Herald TranscriptMar 23 2019, 5:43 PM

This commit now requires audit.Mar 23 2019, 5:43 PM

Harbormaster completed building B22339: rP3d481fb75772: (stable) Fix an issue with "nextPage()" on worker trigger queries.Mar 23 2019, 5:45 PM

Changes (1)

Path

Size

src/

infrastructure/

daemon/

workers/

query/

PhabricatorWorkerTriggerQuery.php

rP3d481fb75772

View Options

src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php