Variables in ldap_public not always expanded

[fisher]

I have configured an address book for the plugin new_user_identity. The address book configuration allows to use variables in the base_dn, search_base_dn and so, but the plugin (err, actually I believe the roundcube_plugin object, but I am not really familiar with php) does not resolve them. The config line:

'search_base_dn' => 'dc=%d,ou=domains,dc=server,dc=hu'

Became:

dc=,ou=domains,dc=server,dc=hu'

instead

dc=client.hu,ou=domains,dc=server,dc=hu'

The LDAP log contains 'S: Invalid DN syntax' message.

Tested in 5.2, 5.3 and the trunk version (yesterday :D)

[alec]

Right. The domain is not set in the moment when new_user_identity is doing its job. Try a fix, go to rcube_ldap.php and change line 184:

$fu = $RCMAIL->user->get_username();

to

$fu = $RCMAIL->user->get_username();
if (!$fu)
    $fu = $this->mail_domain;

[fisher]

I have tired the trick above but did not help.

Anyway tt is not big problem for me, I have changed the configuration and replaced the %d with the actual domain name.

[thomasb]

$this->mail_domain is empty because $_SESSION['imap_host'] isn't set yet when the create_user hook is triggered.

Added host argument to hook parameters. Should now be fixed in [94a5a24f] and r5163/svn.