Skip to content
Snippets Groups Projects

Issue #2991181 by bluegeek9: Properties are also shown as available field mappings

Merged Steven Ayers requested to merge issue/ldap-2991181:2991181-properties-are-also into 8.x-4.x
2 files
+ 35
0
Compare changes
  • Side-by-side
  • Inline
Files
2
@@ -67,6 +67,17 @@ abstract class LdapUserMappingBaseForm extends ConfigFormBase implements LdapUse
*/
protected $server;
/**
* Fields and properties to exclude from the mapping form.
*/
protected $exclude = [
'[field.mail]',
'[field.name]',
'[field.pass]',
'[field.roles]',
'[field.status]',
];
/**
* {@inheritdoc}
*/
@@ -156,6 +167,11 @@ abstract class LdapUserMappingBaseForm extends ConfigFormBase implements LdapUse
}
}
$params = [$direction, $sid];
foreach ($this->exclude as $exclude) {
if (array_key_exists($exclude, $attributes)) {
unset($attributes[$exclude]);
}
}
$this->moduleHandler->alter(
'ldap_user_attributes',
Loading