From 594690ade94ef652258d5ae39fbfeae9580c3943 Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 30 Jun 2015 11:19:16 -0700 Subject: [PATCH] Don't require missing fields be present in order to subscribe to an object Summary: Fixes T8719. It's fine to subscribe to objects that don't have all their required fields filled in. Test Plan: Subscribed/unsubscribed from an object. Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Maniphest Tasks: T8719 Differential Revision: https://secure.phabricator.com/D13484 --- .../controller/PhabricatorSubscriptionsEditController.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php b/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php index acc8f5ebe9..4519a1c902 100644 --- a/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php +++ b/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php @@ -95,6 +95,7 @@ final class PhabricatorSubscriptionsEditController $editor = id($object->getApplicationTransactionEditor()) ->setActor($user) ->setContinueOnNoEffect(true) + ->setContinueOnMissingFields(true) ->setContentSourceFromRequest($request); $editor->applyTransactions(