search

plone / plone.app.ldap

Plone UI and integration of the functionality provided by LDAPMultiPlugins, LDAPUserFolder and PloneLDAP
3 stars 10 forks source link

The 'activated_plugins' default value changed to empty list #2

Closed radekj closed 12 years ago

radekj commented 12 years ago

The 'activated_plugins' default value should be an empty list instead of None, becouse it's expected to be an iterable type in other places in the code (for example: https://github.com/plone/plone.app.ldap/blob/master/plone/app/ldap/ploneldap/util.py#L173)

radekj commented 12 years ago

I just noticed that there already is an issue duplicating this problem (https://github.com/plone/plone.app.ldap/issues/1) so this pull request would also fix that issue.