Differential D25210 Diff 757 src/applications/settings/panel/PhabricatorEmailAddressesSettingsPanel.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/settings/panel/PhabricatorEmailAddressesSettingsPanel.php
Show First 20 Lines • Show All 166 Lines • ▼ Show 20 Lines | $token = id(new PhabricatorAuthSessionEngine())->requireHighSecuritySession( | ||||
$viewer, | $viewer, | ||||
$request, | $request, | ||||
$this->getPanelURI()); | $this->getPanelURI()); | ||||
$e_email = true; | $e_email = true; | ||||
$email = null; | $email = null; | ||||
$errors = array(); | $errors = array(); | ||||
if ($request->isDialogFormPost()) { | if ($request->isDialogFormPost()) { | ||||
$email = trim($request->getStr('email')); | $email = trim($request->getStr('email', '')); | ||||
valerio.bozzolan: Probably the exact minimal fix is just that.
Kind of "don't care" logic. | |||||
if ($new == 'verify') { | if ($new == 'verify') { | ||||
// The user clicked "Done" from the "an email has been sent" dialog. | // The user clicked "Done" from the "an email has been sent" dialog. | ||||
return id(new AphrontReloadResponse())->setURI($uri); | return id(new AphrontReloadResponse())->setURI($uri); | ||||
} | } | ||||
PhabricatorSystemActionEngine::willTakeAction( | PhabricatorSystemActionEngine::willTakeAction( | ||||
array($viewer->getPHID()), | array($viewer->getPHID()), | ||||
▲ Show 20 Lines • Show All 232 Lines • Show Last 20 Lines |
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0
Probably the exact minimal fix is just that.
Kind of "don't care" logic.