From 62e052f3fc3511c7b6b159e18c48fc68d3e6eadd Mon Sep 17 00:00:00 2001 From: Chad Little Date: Tue, 28 Jul 2015 07:44:55 -0700 Subject: [PATCH] Remove View Log action from Herald List Summary: Fixes T8979. I beleive this explicit page was merged into view, at least I can't find any place to link it now. Test Plan: View a Herald list, no icon. Reviewers: epriestley Reviewed By: epriestley Subscribers: epriestley, Korvin Maniphest Tasks: T8979 Differential Revision: https://secure.phabricator.com/D13743 --- src/applications/herald/query/HeraldRuleSearchEngine.php | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/applications/herald/query/HeraldRuleSearchEngine.php b/src/applications/herald/query/HeraldRuleSearchEngine.php index 63e6efc818..04b7a5d852 100644 --- a/src/applications/herald/query/HeraldRuleSearchEngine.php +++ b/src/applications/herald/query/HeraldRuleSearchEngine.php @@ -197,12 +197,6 @@ final class HeraldRuleSearchEngine extends PhabricatorApplicationSearchEngine { $item->addIcon('fa-lock grey', pht('Disabled')); } - $item->addAction( - id(new PHUIListItemView()) - ->setHref($this->getApplicationURI("history/{$id}/")) - ->setIcon('fa-file-text-o') - ->setName(pht('Edit Log'))); - $content_type_name = idx($content_type_map, $rule->getContentType()); $item->addAttribute(pht('Affects: %s', $content_type_name));