X-Git-Url: https://git.stderr.nl/gitweb?p=matthijs%2Fupstream%2Fdjango-ldapdb.git;a=blobdiff_plain;f=ldapdb%2Fmodels%2Fquery.py;h=129e5056df58b23009d5eb8939728e6b849dbdc0;hp=1fddeeeb1aea3b8bb0ece0f4a7b079fa8d802b84;hb=671b41b55bc07b0aa986aea0953620e34218d8e8;hpb=641160553d6db632259d325be2b22ee7c3059f3c diff --git a/ldapdb/models/query.py b/ldapdb/models/query.py index 1fddeee..129e505 100644 --- a/ldapdb/models/query.py +++ b/ldapdb/models/query.py @@ -56,8 +56,12 @@ class Constraint(BaseConstraint): """ An object that can be passed to WhereNode.add() and knows how to pre-process itself prior to including in the WhereNode. + + NOTES: + - we redefine this class, because when self.field is None calls + Field().get_db_prep_lookup(), which short-circuits our LDAP-specific code. """ - def process(self, lookup_type, value): + def process(self, lookup_type, value, connection): """ Returns a tuple of data suitable for inclusion in a WhereNode instance. @@ -67,10 +71,12 @@ class Constraint(BaseConstraint): try: if self.field: - params = self.field.get_db_prep_lookup(lookup_type, value) + params = self.field.get_db_prep_lookup(lookup_type, value, + connection=connection, prepared=True) db_type = self.field.db_type() else: - params = CharField().get_db_prep_lookup(lookup_type, value) + params = CharField().get_db_prep_lookup(lookup_type, value, + connection=connection, prepared=True) db_type = None except ObjectDoesNotExist: raise EmptyShortCircuit @@ -173,22 +179,11 @@ class WhereNode(BaseWhereNode): constraint, lookup_type, y, values = item comp = get_lookup_operator(lookup_type) - if hasattr(constraint, "col"): - # django 1.2 - column = constraint.col - if lookup_type == 'in': - equal_bits = [ "(%s%s%s)" % (column, comp, value) for value in values ] - clause = '(|%s)' % ''.join(equal_bits) - else: - clause = "(%s%s%s)" % (constraint.col, comp, values) + if lookup_type == 'in': + equal_bits = [ "(%s%s%s)" % (constraint.col, comp, value) for value in values ] + clause = '(|%s)' % ''.join(equal_bits) else: - # django 1.1 - (table, column, db_type) = constraint - equal_bits = [ "(%s%s%s)" % (column, comp, value) for value in values ] - if len(equal_bits) == 1: - clause = equal_bits[0] - else: - clause = '(|%s)' % ''.join(equal_bits) + clause = "(%s%s%s)" % (constraint.col, comp, values) bits.append(clause) @@ -220,7 +215,7 @@ class Query(BaseQuery): filterstr += sql return '(&%s)' % filterstr - def get_count(self, using=None): + def get_count(self, using): try: vals = ldapdb.connection.search_s( self.model.base_dn, @@ -244,24 +239,13 @@ class Query(BaseQuery): return Compiler(self, ldapdb.connection, using) def has_results(self, using): - return self.get_count() != 0 - - def results_iter(self): - "For django 1.1 compatibility" - return self.get_compiler().results_iter() + return self.get_count(using) != 0 class QuerySet(BaseQuerySet): def __init__(self, model=None, query=None, using=None): if not query: - import inspect - spec = inspect.getargspec(BaseQuery.__init__) - if len(spec[0]) == 3: - # django 1.2 - query = Query(model, WhereNode) - else: - # django 1.1 - query = Query(model, None, WhereNode) - super(QuerySet, self).__init__(model=model, query=query) + query = Query(model, WhereNode) + super(QuerySet, self).__init__(model=model, query=query, using=using) def delete(self): "Bulk deletion."