Download & Extend

Features exported variables get overwritten during module installation

Project:Lightweight Directory Access Protocol (LDAP)
Version:7.x-2.x-dev
Component:Code
Category:bug report
Priority:major
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

#1

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.

#2

Priority:normal» major

#3

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.

#4

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.

#5

Status:needs review» fixed

#6

Status:fixed» closed (fixed)

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

nobody click here