How is the ldap_servers_encryption_key intended to be exported? Since the variable gets set during ldap_servers_install, exporting the variable to a feature doesn't work unless you revert the strongarm component after the module is enabled.

Using $conf['ldap_servers_encryption_key'] is an alternative.

Comments

joelcollinsdc’s picture

Title: Difficult / confusing to export the encryption key » Features exported variables get overwritten during module installation

Same thing happens with ldap_authentication_conf in ldap_user... aarg.

johnbarclay’s picture

Priority: Normal » Major
johnbarclay’s picture

Status: Active » Needs review

Thanks. I committed this. http://drupalcode.org/project/ldap.git/commitdiff/9ee53d24aef486f205669e...

Some of the code was no longer used; other variable setting was replaced by correct defaults elsewhere in code.

Please test.

Note that 'ldap_servers_encrypt_key' should still be stored in $conf in settings.php. Otherwise it will default to drupal_get_hash_salt(0 after the next commit.

johnbarclay’s picture

The ldap server salt is also fixed by this patch. See http://drupalcode.org/project/ldap.git/commitdiff/4528533fece804190c5543...

So there are no more variables being set in *.intstall hooks. So exportables should behave better.

johnbarclay’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.