mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-30 02:32:42 +01:00
Show only enabled providers on the account link screen
Summary: Ref T1536. Above, we load all providers, which is intentional (if a user has a link with a previously-enabled but now-disabled provider, we should enrich it with provider information). However, before showing linking options we should drop disabled providers. Test Plan: Disabled Disqus, reloaded, didn't see Disqus anymore. Reviewers: mbishopim3 Reviewed By: mbishopim3 CC: aran Maniphest Tasks: T1536 Differential Revision: https://secure.phabricator.com/D6219
This commit is contained in:
parent
c0cc7bbfdf
commit
ad8fe93cc7
1 changed files with 2 additions and 0 deletions
|
@ -104,6 +104,8 @@ final class PhabricatorSettingsPanelExternalAccounts
|
|||
pht('Your account is linked with all available providers.'));
|
||||
|
||||
$accounts = mpull($accounts, null, 'getProviderKey');
|
||||
|
||||
$providers = PhabricatorAuthProvider::getAllEnabledProviders();
|
||||
$providers = msort($providers, 'getProviderName');
|
||||
foreach ($providers as $key => $provider) {
|
||||
if (isset($accounts[$key])) {
|
||||
|
|
Loading…
Reference in a new issue