From 5b59a81ab9b3d407fd2b4ce221e62c33de805bad Mon Sep 17 00:00:00 2001 From: Bob Trahan Date: Tue, 9 Dec 2014 10:14:14 -0800 Subject: [PATCH] Projects - fix missing subscriber handling code in the editor Summary: Fixes T6719. At some point, we added automagical subscriptions via @mentions, and these were failing in project descriptions from a lack of an implementation in the editor. Said "implementation" is to do nothing, but it needs to be there nonetheless. Test Plan: updated a project mentioning someone in the description and it worked. also saw a 'subscriber added' transaction Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T6719 Differential Revision: https://secure.phabricator.com/D10951 --- .../project/editor/PhabricatorProjectTransactionEditor.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/applications/project/editor/PhabricatorProjectTransactionEditor.php b/src/applications/project/editor/PhabricatorProjectTransactionEditor.php index 759fe4cb1b..8a72821478 100644 --- a/src/applications/project/editor/PhabricatorProjectTransactionEditor.php +++ b/src/applications/project/editor/PhabricatorProjectTransactionEditor.php @@ -101,6 +101,7 @@ final class PhabricatorProjectTransactionEditor case PhabricatorProjectTransaction::TYPE_LOCKED: $object->setIsMembershipLocked($xaction->getNewValue()); return; + case PhabricatorTransactions::TYPE_SUBSCRIBERS: case PhabricatorTransactions::TYPE_EDGE: return; case PhabricatorTransactions::TYPE_VIEW_POLICY: @@ -165,6 +166,7 @@ final class PhabricatorProjectTransactionEditor } return; + case PhabricatorTransactions::TYPE_SUBSCRIBERS: case PhabricatorTransactions::TYPE_VIEW_POLICY: case PhabricatorTransactions::TYPE_EDIT_POLICY: case PhabricatorTransactions::TYPE_JOIN_POLICY: