recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Conduit: Improve error handling calling phid.query without PHIDs

Summary:
Avoid an ugly TypeError exception. Throw a proper error code instead.

```
EXCEPTION: (TypeError) PhabricatorHandleQuery::withPHIDs(): Argument #1 ($phids) must be of type array, null given, called in /var/www/html/phorge/phorge/src/applications/phid/conduit/PHIDQueryConduitAPIMethod.php on line 28 at [<phorge>/src/applications/phid/query/PhabricatorHandleQuery.php:12]
```

Closes T16439

Test Plan: Go to http://phorge.localhost/conduit/method/phid.query/ and press "Call Method"

Reviewers: O1 Blessed Committers, mainframe98

Reviewed By: O1 Blessed Committers, mainframe98

Subscribers: mainframe98, tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Maniphest Tasks: T16439

Differential Revision: https://we.phorge.it/D26642

Andre Klapper 2b8a17e3 7d0bb15e

+10
+10
src/applications/phid/conduit/PHIDQueryConduitAPIMethod.php
··· 20 20 return 'nonempty dict<string, wild>'; 21 21 } 22 22 23 + protected function defineErrorTypes() { 24 + return array( 25 + 'ERR-BAD-PHID' => pht('Must pass PHIDs.'), 26 + ); 27 + } 28 + 23 29 protected function execute(ConduitAPIRequest $request) { 24 30 $phids = $request->getValue('phids'); 31 + 32 + if (!$phids) { 33 + throw new ConduitException('ERR-BAD-PHID'); 34 + } 25 35 26 36 $handles = id(new PhabricatorHandleQuery()) 27 37 ->setViewer($request->getUser())