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

	Unknown Object (File)
	Sat, Nov 2, 7:10 AM

	Unknown Object (File)
	Oct 24 2024, 3:17 AM

	Unknown Object (File)
	Oct 9 2024, 1:59 PM

	Unknown Object (File)
	Sep 25 2024, 11:51 AM

	Unknown Object (File)
	Sep 25 2024, 9:02 AM

	Unknown Object (File)
	Sep 25 2024, 8:55 AM

	Unknown Object (File)
	Sep 21 2024, 11:16 PM

	Unknown Object (File)
	Sep 6 2024, 8:48 AM

View All 89 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