diff --git a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php index a27606e5e7..f9f51f1d8f 100644 --- a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php +++ b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php @@ -429,9 +429,12 @@ abstract class PhabricatorApplicationTransactionEditor $phids = array_diff($phids, $this->subscribers); } - if (!$phids) { - return null; + foreach ($phids as $key => $phid) { + if ($object->isAutomaticallySubscribed($phid)) { + unset($phids[$key]); + } } + $phids = array_values($phids); $xaction = newv(get_class(head($xactions)), array()); $xaction->setTransactionType(PhabricatorTransactions::TYPE_SUBSCRIBERS); diff --git a/src/view/layout/PhabricatorTimelineEventView.php b/src/view/layout/PhabricatorTimelineEventView.php index e113de5f49..463f89452e 100644 --- a/src/view/layout/PhabricatorTimelineEventView.php +++ b/src/view/layout/PhabricatorTimelineEventView.php @@ -163,7 +163,7 @@ final class PhabricatorTimelineEventView extends AphrontView { $classes = array(); $classes[] = 'phabricator-timeline-event-view'; $classes[] = 'phabricator-timeline-border'; - if ($content) { + if (!$this->isEmptyContent($content)) { $classes[] = 'phabricator-timeline-major-event'; $content = phutil_tag( 'div',