Fix a followup notification paging error with partial objects
ClosedPublic
Actions

Authored by epriestley on Mar 12 2021, 5:08 PM.

Tags

None

Referenced Files

	F17948033: D21609.id51441.diff
	Thu, Jul 31, 7:54 PM

	F17943506: D21609.diff
	Thu, Jul 31, 8:25 AM

	F17939780: D21609.id.diff
	Thu, Jul 31, 3:25 AM

	F17708950: D21609.id51441.diff
	Thu, Jul 17, 3:22 AM

	Unknown Object (File)
	Jun 8 2025, 9:03 PM

	Unknown Object (File)
	Jun 2 2025, 1:26 PM

	Unknown Object (File)
	Apr 27 2025, 1:36 PM

	Unknown Object (File)
	Apr 26 2025, 8:46 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13623: Notifications may fail to paginate with GROUP BY error
Commits: rP4b529e600928: Fix a followup notification paging error with partial objects

Summary

Ref T13623. In D21603, I made the "partial object" this query returns a raw row, which paging keys can no longer be extracted from properly.

Test Plan

Paged notifications to page 2, no longer saw an error.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 12 2021, 5:08 PM

Harbormaster completed remote builds in B25248: Diff 51441.Mar 12 2021, 5:09 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 12 2021, 5:09 PM

epriestley requested review of this revision.

Closed by commit rP4b529e600928: Fix a followup notification paging error with partial objects (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP4b529e600928: Fix a followup notification paging error with partial objects.

Revision Contents
Changeset List

Path

Size

src/

applications/

notification/

query/

PhabricatorNotificationQuery.php

2 lines

Diff 51442

View Options

src/applications/notification/query/PhabricatorNotificationQuery.php

Fix a followup notification paging error with partial objectsClosedPublicActions