From c03945cd6377999e32ddbaf0a0ec89f3a9b52bc8 Mon Sep 17 00:00:00 2001 From: Nick Liu Date: Fri, 22 May 2020 18:34:58 -0500 Subject: [PATCH] social: Add New Provider includes all known scopes by default Because it's probably quite common that the "email" scope be used by default See: https://github.com/e107inc/e107/issues/4115#issuecomment-622561908 --- e107_plugins/social/admin_config.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/e107_plugins/social/admin_config.php b/e107_plugins/social/admin_config.php index 1ed9400b0..0d67b2759 100644 --- a/e107_plugins/social/admin_config.php +++ b/e107_plugins/social/admin_config.php @@ -662,13 +662,14 @@ class social_ui extends e_admin_ui foreach ($fieldInfo as $fieldSlash => $description) { $field = str_replace("/", "][", $fieldSlash); + $placeholder = self::getPlaceholderFor($provider_name, $fieldSlash); $frm_options = [ 'size' => 'xxlarge', - 'placeholder' => self::getPlaceholderFor($provider_name, $fieldSlash), + 'placeholder' => $placeholder, ]; $text .= "".$this->getLabel($fieldSlash).""; - $text .= $frm->text("social_login[$provider_name][$field]", $slcm->getProviderConfig($provider_name, $fieldSlash), 256, $frm_options); + $text .= $frm->text("social_login[$provider_name][$field]", $placeholder, 256, $frm_options); $text .= "
$description
"; $text .= ""; }