diff --git a/src/applications/meta/controller/PhabricatorApplicationUninstallController.php b/src/applications/meta/controller/PhabricatorApplicationUninstallController.php index 72313a0048..c04426a502 100644 --- a/src/applications/meta/controller/PhabricatorApplicationUninstallController.php +++ b/src/applications/meta/controller/PhabricatorApplicationUninstallController.php @@ -43,7 +43,7 @@ final class PhabricatorApplicationUninstallController return id(new AphrontDialogResponse())->setDialog($dialog); } - if ($request->isDialogFormPost()) { + if ($request->isFormOrHisecPost()) { $xactions = array(); $template = $application->getApplicationTransactionTemplate(); $xactions[] = id(clone $template) @@ -54,6 +54,7 @@ final class PhabricatorApplicationUninstallController $editor = id(new PhabricatorApplicationEditor()) ->setActor($user) ->setContentSourceFromRequest($request) + ->setCancelURI($view_uri) ->setContinueOnNoEffect(true) ->setContinueOnMissingFields(true); diff --git a/src/applications/meta/xactions/PhabricatorApplicationUninstallTransaction.php b/src/applications/meta/xactions/PhabricatorApplicationUninstallTransaction.php index 6083738212..f8bc1271b3 100644 --- a/src/applications/meta/xactions/PhabricatorApplicationUninstallTransaction.php +++ b/src/applications/meta/xactions/PhabricatorApplicationUninstallTransaction.php @@ -81,4 +81,11 @@ final class PhabricatorApplicationUninstallTransaction } } + public function shouldTryMFA( + $object, + PhabricatorApplicationTransaction $xaction) { + // Installing or uninstalling applications is a big deal. + return true; + } + }