@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
Load audit requests when querying audits
Summary: Fixes T9434. I'm not sure exactly what changed behavior here, but we need a `needAuditRequests()`.
Test Plan: Ran a query which hit the exception (empty query was good enough, locally), then applied this patch; saw exception go away.
Reviewers: chad
Reviewed By: chad
Maniphest Tasks: T9434
Differential Revision: https://secure.phabricator.com/D14162
+2
-1
+2
-1
src/applications/audit/conduit/AuditQueryConduitAPIMethod.php
+2
-1
src/applications/audit/conduit/AuditQueryConduitAPIMethod.php
···
37
37
protected function execute(ConduitAPIRequest $request) {
38
38
39
39
$query = id(new DiffusionCommitQuery())
40
-
->setViewer($request->getUser());
40
+
->setViewer($request->getUser())
41
+
->needAuditRequests(true);
41
42
42
43
$auditor_phids = $request->getValue('auditorPHIDs', array());
43
44
if ($auditor_phids) {