fixed ldap form, it doesn't validate the baseDN until enable checkbox is set

rhodecode/controllers/admin/ldap_settings.py

@@ -68,10 +68,11 class LdapSettingsController(BaseControl
         """POST ldap create and store ldap settings"""
 
         settings_model = SettingsModel()
-        _form = LdapSettingsForm()()
+        post_data = dict(request.POST)
+        _form = LdapSettingsForm(post_data.get('ldap_active'))()
 
         try:
-            form_result = _form.to_python(dict(request.POST))
+            form_result = _form.to_python(post_data)
             try:
 
                 for k, v in form_result.items():

rhodecode/model/forms.py

@@ -300,10 +300,13 class LdapLibValidator(formencode.valida
             raise LdapImportError
         return value
 
-class BaseDnValidator(formencode.validators.FancyValidator):
+def BaseDnValidator(ldap_enable):
+    class _BaseDnValidator(formencode.validators.FancyValidator):
 
     def to_python(self, value, state):
 
+            if not ldap_enable:
+                return ''
         try:
             value % {'user':'valid'}
 
@@ -319,7 +322,7 class BaseDnValidator(formencode.validat
                                          value, state)
 
         return value
-
+    return _BaseDnValidator
 #===============================================================================
 # FORMS        
 #===============================================================================
@@ -467,7 +470,7 def DefaultPermissionsForm(perms_choices
     return _DefaultPermissionsForm
 
 
-def LdapSettingsForm():
+def LdapSettingsForm(ldap_enable):
     class _LdapSettingsForm(formencode.Schema):
         allow_extra_fields = True
         filter_extra_fields = True
@@ -478,6 +481,6 def LdapSettingsForm():
         ldap_ldaps = StringBoolean(if_missing=False)
         ldap_dn_user = UnicodeString(strip=True,)
         ldap_dn_pass = UnicodeString(strip=True,)
-        ldap_base_dn = All(BaseDnValidator, UnicodeString(strip=True,))
+        ldap_base_dn = All(BaseDnValidator(ldap_enable), UnicodeString(strip=True,))
 
     return _LdapSettingsForm