Differential D21824

Fix an exception in cursor pagination of Conpherence threads
ClosedPublic
Actions

Authored by epriestley on May 17 2022, 11:08 PM.

Tags

None

Referenced Files

	F15543340: D21824.id.diff
	Fri, Apr 25, 9:57 PM

	F15515288: D21824.id52004.diff
	Fri, Apr 18, 9:16 AM

	F15512836: D21824.id52011.diff
	Thu, Apr 17, 2:22 PM

	F15511012: D21824.id52004.diff
	Wed, Apr 16, 11:24 PM

	F15502992: D21824.diff
	Mon, Apr 14, 5:57 AM

	F15485204: D21824.id52011.diff
	Wed, Apr 9, 10:32 PM

	F15483788: D21824.id52004.diff
	Apr 9 2025, 2:26 PM

	F15480123: D21824.id.diff
	Apr 8 2025, 11:58 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13680: Pagination in Conpherence may pass an int to `phutil_nonempty_string()`, throwing an exception
Commits: rP147b48b934b3: Fix an exception in cursor pagination of Conpherence threads

Summary

Ref T13680. Conpherence may pass values with an integer type to this layer of the stack. These are "supposed" to be strings, but just be accepting.

Test Plan

Wrote 100+ messages to a Conpherence room.
Clicked "Show Older Messages".
Before: exception, int passed to "phutil_nonempty_string()".
After: older messages loaded.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley requested review of this revision.May 17 2022, 11:08 PM

epriestley created this revision.

Harbormaster completed remote builds in B25723: Diff 52004.May 17 2022, 11:08 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 17 2022, 11:31 PM

Closed by commit rP147b48b934b3: Fix an exception in cursor pagination of Conpherence threads (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP147b48b934b3: Fix an exception in cursor pagination of Conpherence threads.

epriestley mentioned this in T13680: Pagination in Conpherence may pass an int to `phutil_nonempty_string()`, throwing an exception.May 19 2022, 5:33 PM

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

query/

policy/

PhabricatorCursorPagedPolicyAwareQuery.php

11 lines

Diff 52011

src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php

Loading...