Commit e7ef2f6e authored by Côme Chilliet's avatar Côme Chilliet
Browse files

Merge branch '5589-sasl-missing-from-password-default-hash-field-in-setup-wizard' into '1.3-dev'

Resolve "SASL missing from Password default hash field in setup wizard"

Closes #5589

See merge request !35
Showing with 12 additions and 0 deletions
+12 -0
......@@ -455,6 +455,9 @@ class configInLdap extends simplePlugin
/* Password methods */
$methods = passwordMethod::get_available_methods();
$methods = $methods['name'];
if (!in_array('sasl', $methods)) {
$methods[] = 'sasl';
}
$attributesInfo['password']['attrs'][0]->setChoices($methods);
$groupsAndRoles = array_merge(
......@@ -569,6 +572,15 @@ class configInLdap extends simplePlugin
return $this->dn;
}
function check()
{
$messages = parent::check();
if (($this->fdPasswordDefaultHash == 'sasl') && ($this->fdSaslRealm == '') && ($this->fdSaslExop == '')) {
$messages[] = _('You need to fill saslRealm or saslExop in the configuration screen in order to use SASL');
}
return $messages;
}
static function get_themes()
{
$themesdir = '../ihtml/themes/';
......
  • bmortier @bmortier

    mentioned in commit 1783e487

    By Côme Chilliet on 2017-10-24T09:27:38 (imported from GitLab)

    ·

    mentioned in commit 1783e487

    By Côme Chilliet on 2017-10-24T09:27:38 (imported from GitLab)

    Toggle commit list
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment