X-Git-Url: https://git.stderr.nl/gitweb?a=blobdiff_plain;f=ldapdb%2Fmodels%2Fquery.py;h=020d4eeb7d4a9bf3d03c6242ad296ade8789e0a2;hb=3912bd6478e2c17a4ea171840a50d1d78ca2694c;hp=f430286b65be5d14531e46a86b540eaa3a851ab6;hpb=aa99c514ad0457d05e95f48de4b0c855a75b363e;p=matthijs%2Fupstream%2Fdjango-ldapdb.git diff --git a/ldapdb/models/query.py b/ldapdb/models/query.py index f430286..020d4ee 100644 --- a/ldapdb/models/query.py +++ b/ldapdb/models/query.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- # # django-ldapdb -# Copyright (C) 2009 Bolloré telecom +# Copyright (C) 2009-2010 Bolloré telecom # See AUTHORS file for a full list of contributors. # # This program is free software: you can redistribute it and/or modify @@ -21,7 +21,6 @@ from copy import deepcopy import ldap -from django.db.models.fields import Field from django.db.models.query import QuerySet as BaseQuerySet from django.db.models.query_utils import Q from django.db.models.sql import Query as BaseQuery @@ -29,6 +28,16 @@ from django.db.models.sql.where import WhereNode as BaseWhereNode, Constraint as import ldapdb +from ldapdb.models.fields import CharField + +def get_lookup_operator(lookup_type): + if lookup_type == 'gte': + return '>=' + elif lookup_type == 'lte': + return '<=' + else: + return '=' + class Constraint(BaseConstraint): """ An object that can be passed to WhereNode.add() and knows how to @@ -42,21 +51,12 @@ class Constraint(BaseConstraint): # Because of circular imports, we need to import this here. from django.db.models.base import ObjectDoesNotExist - if lookup_type == 'endswith': - params = ["*%s" % value] - elif lookup_type == 'startswith': - params = ["%s*" % value] - elif lookup_type == 'exact': - params = [value] - elif lookup_type == 'in': - params = [v for v in value] - else: - raise TypeError("Field has invalid lookup: %s" % lookup_type) - try: if self.field: + params = self.field.get_db_prep_lookup(lookup_type, value) db_type = self.field.db_type() else: + params = CharField().get_db_prep_lookup(lookup_type, value) db_type = None except ObjectDoesNotExist: raise EmptyShortCircuit @@ -81,17 +81,26 @@ class WhereNode(BaseWhereNode): if isinstance(item, WhereNode): bits.append(item.as_sql()) continue - if len(item) == 4: - # django 1.1 - (table, column, type), x, y, values = item - else: - # django 1.0 - table, column, type, x, y, values = item - equal_bits = [ "(%s=%s)" % (column, value) for value in values ] - if len(equal_bits) == 1: - clause = equal_bits[0] + + 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) else: - clause = '(|%s)' % ''.join(equal_bits) + # 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) + if self.negated: bits.append('(!%s)' % clause) else: @@ -149,8 +158,15 @@ class Query(BaseQuery): yield row class QuerySet(BaseQuerySet): - def __init__(self, model=None, query=None): + def __init__(self, model=None, query=None, using=None): if not query: - query = Query(model, None, WhereNode) + import inspect + spec = inspect.getargspec(Query.__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, query)