Differential D25210 Diff 759 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.