From 0a3aae8362568ab470a7288829bd8e9678e67ee2 Mon Sep 17 00:00:00 2001 From: Luke Plant Date: Thu, 10 Mar 2011 01:15:19 +0000 Subject: [PATCH] Fixed #15559 - distinct queries introduced by [15607] cause errors with some custom model fields This patch just reverts [15607] until a more satisfying solution can be found. Refs #11707 git-svn-id: http://code.djangoproject.com/svn/django/trunk@15791 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- django/db/models/fields/related.py | 2 +- tests/regressiontests/model_fields/models.py | 5 ----- tests/regressiontests/model_fields/tests.py | 19 +------------------ 3 files changed, 2 insertions(+), 24 deletions(-) diff --git a/django/db/models/fields/related.py b/django/db/models/fields/related.py index 4c4e6f01e8..1318706040 100644 --- a/django/db/models/fields/related.py +++ b/django/db/models/fields/related.py @@ -910,7 +910,7 @@ class ForeignKey(RelatedField, Field): db = kwargs.pop('using', None) defaults = { 'form_class': forms.ModelChoiceField, - 'queryset': self.rel.to._default_manager.using(db).complex_filter(self.rel.limit_choices_to).distinct(), + 'queryset': self.rel.to._default_manager.using(db).complex_filter(self.rel.limit_choices_to), 'to_field_name': self.rel.field_name, } defaults.update(kwargs) diff --git a/tests/regressiontests/model_fields/models.py b/tests/regressiontests/model_fields/models.py index 7db1f8904a..1dc1649f13 100644 --- a/tests/regressiontests/model_fields/models.py +++ b/tests/regressiontests/model_fields/models.py @@ -29,11 +29,6 @@ class Bar(models.Model): b = models.CharField(max_length=10) a = models.ForeignKey(Foo, default=get_foo) -class Baz(models.Model): - a = models.CharField(max_length=5) - #Only Foos related to Bars starting with 'a' - foo = models.ForeignKey(Foo, limit_choices_to=models.Q(bar__b__startswith='a')) - class Whiz(models.Model): CHOICES = ( ('Group 1', ( diff --git a/tests/regressiontests/model_fields/tests.py b/tests/regressiontests/model_fields/tests.py index b6d9a3d713..3ec9d7a8b7 100644 --- a/tests/regressiontests/model_fields/tests.py +++ b/tests/regressiontests/model_fields/tests.py @@ -1,6 +1,5 @@ import datetime from decimal import Decimal -import re from django import test from django import forms @@ -9,7 +8,7 @@ from django.db import models from django.db.models.fields.files import FieldFile from django.utils import unittest -from models import Foo, Bar, Baz, Whiz, BigD, BigS, Image, BigInt, Post, NullBooleanModel, BooleanModel, Document +from models import Foo, Bar, Whiz, BigD, BigS, Image, BigInt, Post, NullBooleanModel, BooleanModel, Document # If PIL available, do these tests. if Image: @@ -96,10 +95,6 @@ class DecimalFieldTests(test.TestCase): # This should not crash. That counts as a win for our purposes. Foo.objects.filter(d__gte=100000000000) -class BazForm(forms.ModelForm): - class Meta: - model = Baz - class ForeignKeyTests(test.TestCase): def test_callable_default(self): """Test the use of a lazy callable for ForeignKey.default""" @@ -107,18 +102,6 @@ class ForeignKeyTests(test.TestCase): b = Bar.objects.create(b="bcd") self.assertEqual(b.a, a) - def test_distinct_choice_limit(self): - """Doesn't make sense to offer the same ForeignKey multiple times in a form""" - a = Foo.objects.create(a='a', d=Decimal("-1")) - b = Foo.objects.create(a='b', d=Decimal("1")) - bar_a = Bar.objects.create(b='ah', a=a) - bar_b = Bar.objects.create(b='aha', a=a) - bar_b = Bar.objects.create(b='bla', a=b) - form = BazForm() - fk_field = str(form['foo']) - self.assertEqual(len(re.findall(r'value="%d"' % b.pk, fk_field)), 0) - self.assertEqual(len(re.findall(r'value="%d"' % a.pk, fk_field)), 1) - class DateTimeFieldTests(unittest.TestCase): def test_datetimefield_to_python_usecs(self): """DateTimeField.to_python should support usecs"""