Fix section error in auth_oidc confirmation page for new installations

Author: weilai-irl

classes/form/application.php

@@ -77,7 +77,8 @@ protected function definition() {
         $authmethodoptions = [
             AUTH_OIDC_AUTH_METHOD_SECRET => get_string('auth_method_secret', 'auth_oidc'),
         ];
-        if ($this->_customdata['oidcconfig']->idptype == AUTH_OIDC_IDP_TYPE_MICROSOFT) {
+        if (isset($this->_customdata['oidcconfig']->idptype) &&
+            $this->_customdata['oidcconfig']->idptype == AUTH_OIDC_IDP_TYPE_MICROSOFT) {
             $authmethodoptions[AUTH_OIDC_AUTH_METHOD_CERTIFICATE] = get_string('auth_method_certificate', 'auth_oidc');
         }
         $mform->addElement('select', 'clientauthmethod', auth_oidc_config_name_in_form('clientauthmethod'), $authmethodoptions);

lang/en/auth_oidc.php

@@ -147,7 +147,6 @@
 <li>If Azure AD is used as the IdP, additional profile data can be made available by installing and configuring the <a href="https://moodle.org/plugins/local_o365">Microsoft 365 integration plugin (local_o365)</a>.</li>
 <li>If SDS profile sync feature is enabled in the local_o365 plugin, certain profile fields can be synchronised from SDS to Moodle. when running the "Sync with SDS" scheduled task, and will not happen when running the "Sync users with Azure AD" scheduled task, nor when user logs in.</li>
 </ul>';
-$string['error_incomplete_basic_settings'] = 'Missing basic settings.';
 $string['settings_section_basic'] = 'Basic settings';
 $string['settings_section_authentication'] = 'Authentication';
 $string['settings_section_endpoints'] = 'Endpoints';

manageapplication.php

@@ -52,12 +52,6 @@
 
 $oidcconfig = get_config('auth_oidc');
 
-// If "idptype" or "clientid" is not set, we are not ready to configure application authentication.
-if (!isset($oidcconfig->idptype) || empty($oidcconfig->idptype) || !isset($oidcconfig->clientid) || empty($oidcconfig->clientid)) {
-    redirect(new moodle_url('/admin/settings.php', ['section' => 'auth_oidc_basic_settings']),
-        get_string('error_incomplete_basic_settings', 'auth_oidc'));
-}
-
 $form = new application(null, ['oidcconfig' => $oidcconfig]);
 
 $formdata = [];