@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

Remove an unused variable in PhabricatorDaemonLogViewController

Summary:
While poking a CSS bug in Diviner I was checking where else `PHUITagView::TYPE_STATE` is used.
Turns out that this use can be removed, as any reading of the $tag variable got removed in rP27e13ea0.

Test Plan: Search for `$tag` within that very file.

Reviewers: O1 Blessed Committers, mainframe98

Reviewed By: O1 Blessed Committers, mainframe98

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D26757

-3
-3
src/applications/daemon/controller/PhabricatorDaemonLogViewController.php
··· 24 24 ->setHeader($log->getDaemon()) 25 25 ->setHeaderIcon('fa-pied-piper-alt'); 26 26 27 - $tag = id(new PHUITagView()) 28 - ->setType(PHUITagView::TYPE_STATE); 29 - 30 27 $status = $log->getStatus(); 31 28 switch ($status) { 32 29 case PhabricatorDaemonLog::STATUS_UNKNOWN: