Skip to content

Commit f67a853

Browse files
committed
Audit - add ability to add a package as an auditor
Summary: Fixes T4687. This was also pretty easy...! Test Plan: made a package with a test user as owner. added package as owner. looked right on commit page. logged in as test user and verified audit showed up on home page. Reviewers: epriestley Reviewed By: epriestley Subscribers: chad, epriestley, Korvin Maniphest Tasks: T4687 Differential Revision: https://secure.phabricator.com/D8705
1 parent 496a7d8 commit f67a853

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

src/applications/diffusion/controller/DiffusionCommitController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -778,9 +778,9 @@ private function renderAddCommentPanel(
778778
'dynamic' => array(
779779
'add-auditors-tokenizer' => array(
780780
'actions' => array('add_auditors' => 1),
781-
'src' => '/typeahead/common/usersorprojects/',
781+
'src' => '/typeahead/common/usersprojectsorpackages/',
782782
'row' => 'add-auditors',
783-
'placeholder' => pht('Type a user or project name...'),
783+
'placeholder' => pht('Type a user, project, or package name...'),
784784
),
785785
'add-ccs-tokenizer' => array(
786786
'actions' => array('add_ccs' => 1),

src/applications/typeahead/controller/PhabricatorTypeaheadCommonDatasourceController.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,11 @@ public function processRequest() {
7373
$need_users = true;
7474
$need_projs = true;
7575
break;
76+
case 'usersprojectsorpackages':
77+
$need_users = true;
78+
$need_projs = true;
79+
$need_packages = true;
80+
break;
7681
case 'repositories':
7782
$need_repos = true;
7883
break;
@@ -309,6 +314,7 @@ public function processRequest() {
309314
$packages = id(new PhabricatorOwnersPackage())->loadAll();
310315
foreach ($packages as $package) {
311316
$results[] = id(new PhabricatorTypeaheadResult())
317+
->setIcon('pl-testplan')
312318
->setName($package->getName())
313319
->setURI('/owners/package/'.$package->getID().'/')
314320
->setPHID($package->getPHID());

src/view/form/control/AphrontFormTokenizerControl.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,9 +94,11 @@ private function getDefaultPlaceholder() {
9494
'projects' => pht('Type a project name...'),
9595
'repositories' => pht('Type a repository name...'),
9696
'packages' => pht('Type a package name...'),
97+
'macros' => pht('Type a macro name...'),
9798
'arcanistproject' => pht('Type an arc project name...'),
9899
'accountsorprojects' => pht('Type a user or project name...'),
99-
'macros' => pht('Type a macro name...'),
100+
'usersprojectsorpackages' =>
101+
pht('Type a user, project, or package name...'),
100102
);
101103

102104
return idx($map, $request);

0 commit comments

Comments
 (0)