Differential D20023 Diff 47820 src/applications/auth/xaction/PhabricatorAuthContactNumberStatusTransaction.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/auth/xaction/PhabricatorAuthContactNumberStatusTransaction.php
| Show All 40 Lines | foreach ($xactions as $xaction) { | ||||
| 'Status ("%s") is not a valid contact number status. Valid '. | 'Status ("%s") is not a valid contact number status. Valid '. | ||||
| 'status constants are: %s.', | 'status constants are: %s.', | ||||
| $new_value, | $new_value, | ||||
| implode(', ', array_keys($map))), | implode(', ', array_keys($map))), | ||||
| $xaction); | $xaction); | ||||
| continue; | continue; | ||||
| } | } | ||||
| $mfa_error = $this->newContactNumberMFAError($object, $xaction); | |||||
| if ($mfa_error) { | |||||
| $errors[] = $mfa_error; | |||||
| continue; | |||||
| } | |||||
| // NOTE: Enabling a contact number may cause us to collide with another | // NOTE: Enabling a contact number may cause us to collide with another | ||||
| // active contact number. However, there might also be a transaction in | // active contact number. However, there might also be a transaction in | ||||
| // this group that changes the number itself. Since we can't easily | // this group that changes the number itself. Since we can't easily | ||||
| // predict if we'll collide or not, just let the duplicate key logic | // predict if we'll collide or not, just let the duplicate key logic | ||||
| // handle it when we do. | // handle it when we do. | ||||
| } | } | ||||
| return $errors; | return $errors; | ||||
| } | } | ||||
| } | } | ||||