Skip to content

Commit 174fa1f

Browse files
darrenohcolans
authored andcommitted
Issue #1921484 by Darren Oh: Fixed simplesamlphp_auth_allowdefaultloginroles default value.
1 parent c9fb0c1 commit 174fa1f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

simplesamlphp_auth.module

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,7 @@ function simplesamlphp_auth_init() {
310310
$strAllwDefLogUsers = variable_get('simplesamlphp_auth_allowdefaultloginusers', '');
311311
$arrAllwDefLogUsers = array();
312312
// See if we limit this privilege to specified roles.
313-
$arrAllwDefLogRoles = variable_get('simplesamlphp_auth_allowdefaultloginroles', FALSE);
313+
$arrAllwDefLogRoles = variable_get('simplesamlphp_auth_allowdefaultloginroles', array());
314314

315315
// If user IDs or roles are specified, we let them in, but everyone else gets logged out.
316316
if (drupal_strlen($strAllwDefLogUsers) || $arrAllwDefLogRoles) {
@@ -559,7 +559,7 @@ function simplesamlphp_auth_settings() {
559559
'#options' => $roles,
560560
'#multiple' => TRUE,
561561
'#title' => t('Which ROLES should be allowed to login with local accounts?'),
562-
'#default_value' => variable_get('simplesamlphp_auth_allowdefaultloginroles', ''),
562+
'#default_value' => variable_get('simplesamlphp_auth_allowdefaultloginroles', array()),
563563
'#description' => t('Roles that should be allowed to login without simpleSAMLphp. Examples are dev/admin roles or guest roles.'),
564564
);
565565
$form['simplesamlphp_auth_grp_auth']['simplesamlphp_auth_allowdefaultloginusers'] = array(

0 commit comments

Comments
 (0)