Skip to content

Commit 7de2fae

Browse files
author
epriestley
committed
Link Herald rules to rule detail pages in Herald transcripts
Summary: Fixes T9410. Depends on D16382. Since all users can now view all Herald rules, we can link them in the transcripts. Test Plan: Viewed a transcript, clicked rule names, reviewed rules. Reviewers: chad Reviewed By: chad Maniphest Tasks: T9410 Differential Revision: https://secure.phabricator.com/D16383
1 parent 78ea664 commit 7de2fae

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/applications/herald/controller/HeraldTranscriptController.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -249,9 +249,13 @@ private function buildActionTranscriptPanel(HeraldTranscript $xscript) {
249249
foreach ($rule_xscripts as $rule_xscript) {
250250
$rule_id = $rule_xscript->getRuleID();
251251

252+
$rule_monogram = pht('H%d', $rule_id);
253+
$rule_uri = '/'.$rule_monogram;
254+
252255
$rule_item = id(new PHUIObjectItemView())
253-
->setObjectName(pht('H%d', $rule_id))
254-
->setHeader($rule_xscript->getRuleName());
256+
->setObjectName($rule_monogram)
257+
->setHeader($rule_xscript->getRuleName())
258+
->setHref($rule_uri);
255259

256260
if (!$rule_xscript->getResult()) {
257261
$rule_item->setDisabled(true);

0 commit comments

Comments
 (0)