Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed #15559 - distinct queries introduced by [15607] cause errors wi…

…th 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
  • Loading branch information...
commit 0a3aae8362568ab470a7288829bd8e9678e67ee2 1 parent f17fc56
Luke Plant spookylukey authored
2  django/db/models/fields/related.py
View
@@ -910,7 +910,7 @@ def formfield(self, **kwargs):
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)
5 tests/regressiontests/model_fields/models.py
View
@@ -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', (
19 tests/regressiontests/model_fields/tests.py
View
@@ -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.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 @@ def test_lookup_really_big_value(self):
# 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 @@ def test_callable_default(self):
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"""
Please sign in to comment.
Something went wrong with that request. Please try again.