This is an automated email from the ASF dual-hosted git repository.
sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git
The following commit(s) were added to refs/heads/master by this push:
new ce29f790 Fix hasLDAP? for Person class
ce29f790 is described below
commit ce29f7901c4f85ce4d5ec47802f4b6b125c4c1a5
Author: Sebb <[email protected]>
AuthorDate: Thu Jan 12 14:01:21 2023 +0000
Fix hasLDAP? for Person class
---
lib/whimsy/asf/ldap.rb | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/lib/whimsy/asf/ldap.rb b/lib/whimsy/asf/ldap.rb
index 5b6c2a4a..0e43bf6a 100644
--- a/lib/whimsy/asf/ldap.rb
+++ b/lib/whimsy/asf/ldap.rb
@@ -738,6 +738,12 @@ module ASF
person.attrs['dn'] ? person : nil
end
+ # override the base version which does not work
+ # as it relies on search by 'cn' == id
+ def hasLDAP?
+ !attrs['dn'].nil?
+ end
+
# list of LDAP attributes for this person, populated lazily upon
# first reference.
def attrs
