Page MenuHomePhabricator

Raise a tailored error when calling "transaction.search" with empty "phids" constraint
ClosedPublic

Authored by epriestley on Aug 3 2018, 8:04 PM.
Tags
None
Referenced Files
F15456785: D19562.id46766.diff
Sun, Mar 30, 12:11 PM
F15439209: D19562.id.diff
Wed, Mar 26, 6:21 AM
F15431936: D19562.diff
Mon, Mar 24, 3:25 PM
F15413504: D19562.id.diff
Wed, Mar 19, 7:17 PM
F15412384: D19562.diff
Wed, Mar 19, 12:47 PM
F15400560: D19562.id46769.diff
Mon, Mar 17, 12:31 PM
Unknown Object (File)
Feb 9 2025, 6:15 AM
Unknown Object (File)
Feb 9 2025, 6:15 AM
Subscribers
None

Details

Summary

Ref T13164. See PHI725. For real "*.search" methods, parameters get validated and you get an error if you use an empty list as a constraint.

Since "transaction.search" isn't really a normal "*.search" method, it doesn't benefit from this. Just do the check manually for now.

Test Plan

Made transaction.search calls with no constraints (got results); a valid costraint (got fewer results); and an invalid empty constraint (got an exception).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable