[Koha-bugs] [Bug 28385] ldap server configuration broken since migration from XML::Simple

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28385

--- Comment #7 from Dobrica Pavlinusic <dpavlin at rot13.org> ---
I was wondering about dropping id and there is also ticket about multiple ldap
servers which would have different ids, but we need to support id="ldapserver"
for upgraded instances, so I decided to start here.

I also considered adding something like:

foreach my $ldap_id ( keys %$ldap ) {
 if ( exists($ldap->{$ldap_id}->{hostname}) ) {
   $ldap = $ldap->{$ldap_id};
   last;
 }
}

but it seemed like too much code. It would support <ldapserver> without any id
and different id from ldapserver, but since we suggest id="ldapserver" is
included in documentation and koha-conf.xml template I decided against it.

Was I wrong about this? ;-)

-- 
You are receiving this mail because:
You are watching all bug changes.