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.

Include all audit states in audit.query

Summary:
Additional audit states were made queryable for T5871.
Include them in Conduit's audit.query as well. In doing so corrects
references from "status-foo" to "audit-status-foo".

Depends on D10271

Test Plan: with an api, issues queries and got sensible results

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D10290

authored by

cburroughs and committed by
epriestley fd45f64e efadfbbc

+7 -3
+7 -3
src/applications/audit/conduit/AuditQueryConduitAPIMethod.php
··· 12 12 13 13 public function defineParamTypes() { 14 14 $statuses = array( 15 - 'status-any', 16 - 'status-open', 15 + DiffusionCommitQuery::AUDIT_STATUS_ANY, 16 + DiffusionCommitQuery::AUDIT_STATUS_OPEN, 17 + DiffusionCommitQuery::AUDIT_STATUS_CONCERN, 18 + DiffusionCommitQuery::AUDIT_STATUS_ACCEPTED, 19 + DiffusionCommitQuery::AUDIT_STATUS_PARTIAL, 17 20 ); 18 21 $status_const = $this->formatStringConstants($statuses); 19 22 20 23 return array( 21 24 'auditorPHIDs' => 'optional list<phid>', 22 25 'commitPHIDs' => 'optional list<phid>', 23 - 'status' => 'optional '.$status_const.' (default = "status-any")', 26 + 'status' => ('optional '.$status_const. 27 + ' (default = "audit-status-any")'), 24 28 'offset' => 'optional int', 25 29 'limit' => 'optional int (default = 100)', 26 30 );