From 31cfdef0f782526bbc9abf7ef1d891b080128f9b Mon Sep 17 00:00:00 2001 From: Chad Little Date: Thu, 6 Aug 2015 17:30:47 -0700 Subject: [PATCH] Reduce colors in ApplicationTransactions for subscriptions Summary: Remove previously added colors. Test Plan: Load page, see less color (task) Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D13816 --- .../storage/PhabricatorApplicationTransaction.php | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php index d6d961c780..c7be284ecd 100644 --- a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php +++ b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php @@ -428,21 +428,6 @@ abstract class PhabricatorApplicationTransaction return 'red'; } break; - case PhabricatorTransactions::TYPE_SUBSCRIBERS: - $old = $this->getOldValue(); - $new = $this->getNewValue(); - $add = array_diff($new, $old); - $rem = array_diff($old, $new); - if ($add && $rem) { - return 'green'; - } else if ($add) { - return 'green'; - } else if ($rem) { - return 'black'; - } else { - return null; - } - break; } return null; }