revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-24 22:40:55 +01:00
Code Issues Releases Wiki Activity

Don't require missing fields be present in order to subscribe to an object

Browse source 
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
This commit is contained in:
epriestley 2015-06-30 11:19:16 -07:00
parent 9eda21451a
commit 594690ade9
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php
View file

@ -95,6 +95,7 @@ final class PhabricatorSubscriptionsEditController
$editor = id($object->getApplicationTransactionEditor()) $editor = id($object->getApplicationTransactionEditor())
->setActor($user) ->setActor($user)
->setContinueOnNoEffect(true) ->setContinueOnNoEffect(true)
->setContinueOnMissingFields(true)
->setContentSourceFromRequest($request); ->setContentSourceFromRequest($request);
$editor->applyTransactions( $editor->applyTransactions(