From 5237fcbf42b33c60c5de471698428e270fdbc0ca Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Mon, 3 Nov 2008 17:18:16 -0500 Subject: Fix mark_entry_active() Reverse order of search filter so we return a useful NotFound exception --- ipa_server/plugins/b_ldap.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ipa_server/plugins/b_ldap.py b/ipa_server/plugins/b_ldap.py index c279ac70..a93f97f0 100644 --- a/ipa_server/plugins/b_ldap.py +++ b/ipa_server/plugins/b_ldap.py @@ -127,10 +127,10 @@ class ldap(CrudBackend): if not object_type: return None - search_filter = "(&(%s=%s)(objectclass=%s))" % ( + search_filter = "(&(objectclass=%s)(%s=%s))" % ( + object_type, key_attribute, - self.dn.escape_dn_chars(primary_key), - object_type + self.dn.escape_dn_chars(primary_key) ) search_base = "%s, %s" % (self.api.env.container_accounts, self.api.env.basedn) @@ -153,7 +153,7 @@ class ldap(CrudBackend): return servercore.get_ipa_config() def mark_entry_active(self, dn): - return servercore.mark_entry_inactive(dn) + return servercore.mark_entry_active(dn) def mark_entry_inactive(self, dn): return servercore.mark_entry_inactive(dn) -- cgit