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

Description

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

rPaae7bdbbdd56: Reference raw "title" row in Phriction paging, not "c.title"

Branches

Unknown

Tags

Unknown

Tasks

T13266: Upgrading: Minor Changes to Query Paging / Cursors

Build Status

Buildable 22338
Build 30564: Run Core Tests

Event Timeline

epriestley added a task: T13266: Upgrading: Minor Changes to Query Paging / Cursors.Mar 23 2019, 5:43 PM

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

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 B22338: rPf0ae75c9916a: 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

rPf0ae75c9916a

View Options

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