Merge pull request #87 from alecghica/master

Fix for issue #86

CHANGES.rst

@@ -5,6 +5,10 @@ History
 1.7.0 (unreleased)
 ------------------
 
+- Fixed error display for /plone_ldapcontrolpanel when a wrong value is
+  provided for the "Groups container DN" field.
+  [alecghica]
+
 - Fixed error adding a Plone user group.
   [iulianpetchesi]
 

src/pas/plugins/ldap/plugin.py

@@ -331,7 +331,11 @@ def getGroupsForPrincipal(self, principal, request=None):
         if self.groups:
             # XXX: provide group_ids function in UGM! Way too calculation-heavy
             #      now
-            return [_.id for _ in _principal.groups]
+            try:
+                return [_.id for _ in _principal.groups]
+            except Exception:
+                logger.exception("Problems with groups settings!")
+                return default
         return default
 
     # ##