From 65e953658a32c09c55f0df531e89327ad82fd217 Mon Sep 17 00:00:00 2001 From: epriestley Date: Thu, 25 Oct 2018 04:58:06 -0700 Subject: [PATCH] Expose Audit actions for "transaction.search" in a basic way Summary: Ref T13210. See PHI841. This mirrors D19509 for Differential. Test Plan: Called `transaction.search` on a commit with a bunch of audit activity, got appropriate labels in the results. Reviewers: amckinley Reviewed By: amckinley Maniphest Tasks: T13210 Differential Revision: https://secure.phabricator.com/D19760 --- .../xaction/DiffusionCommitAcceptTransaction.php | 8 ++++++++ .../xaction/DiffusionCommitConcernTransaction.php | 8 ++++++++ .../xaction/DiffusionCommitResignTransaction.php | 8 ++++++++ .../xaction/DiffusionCommitVerifyTransaction.php | 8 ++++++++ 4 files changed, 32 insertions(+) diff --git a/src/applications/diffusion/xaction/DiffusionCommitAcceptTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitAcceptTransaction.php index f7b919be8e..5ade7f3513 100644 --- a/src/applications/diffusion/xaction/DiffusionCommitAcceptTransaction.php +++ b/src/applications/diffusion/xaction/DiffusionCommitAcceptTransaction.php @@ -70,4 +70,12 @@ final class DiffusionCommitAcceptTransaction $this->renderObject()); } + public function getTransactionTypeForConduit($xaction) { + return 'accept'; + } + + public function getFieldValuesForConduit($object, $data) { + return array(); + } + } diff --git a/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php index 94590def9e..ffd084412e 100644 --- a/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php +++ b/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php @@ -76,4 +76,12 @@ final class DiffusionCommitConcernTransaction $this->renderObject()); } + public function getTransactionTypeForConduit($xaction) { + return 'concern'; + } + + public function getFieldValuesForConduit($object, $data) { + return array(); + } + } diff --git a/src/applications/diffusion/xaction/DiffusionCommitResignTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitResignTransaction.php index 103d0fabfe..8adc8346dc 100644 --- a/src/applications/diffusion/xaction/DiffusionCommitResignTransaction.php +++ b/src/applications/diffusion/xaction/DiffusionCommitResignTransaction.php @@ -63,4 +63,12 @@ final class DiffusionCommitResignTransaction $this->renderObject()); } + public function getTransactionTypeForConduit($xaction) { + return 'resign'; + } + + public function getFieldValuesForConduit($object, $data) { + return array(); + } + } diff --git a/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php index f1ec5834b1..f9f19f4290 100644 --- a/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php +++ b/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php @@ -68,4 +68,12 @@ final class DiffusionCommitVerifyTransaction $this->renderObject()); } + public function getTransactionTypeForConduit($xaction) { + return 'request-verification'; + } + + public function getFieldValuesForConduit($object, $data) { + return array(); + } + }