Opened 5 years ago
Closed 5 years ago
#31767 closed Bug (fixed)
QuerySet.none() on combined queries returns all results.
| Reported by: | Tom Carrick | Owned by: | Mariusz Felisiak |
|---|---|---|---|
| Component: | Database layer (models, ORM) | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
I came across this issue on Stack Overflow. I'm not 100% sure it's a bug, but it does seem strange. With this code (excuse the bizarre example filtering):
class Publication(models.Model):
pass
class Article(models.Model):
publications = models.ManyToManyField(to=Publication, blank=True, null=True)
class ArticleForm(forms.ModelForm):
publications = forms.ModelMultipleChoiceField(
Publication.objects.filter(id__lt=2) | Publication.objects.filter(id__gt=5),
required=False,
)
class Meta:
model = Article
fields = ["publications"]
class ArticleAdmin(admin.ModelAdmin):
form = ArticleForm
This works well. However, changing the ModelMultipleChoiceField queryset to use union() breaks things.
publications = forms.ModelMultipleChoiceField(
Publication.objects.filter(id__lt=2).union(
Publication.objects.filter(id__gt=5)
),
required=False,
)
The form correctly shows only the matching objects. However, if you submit this form while empty (i.e. you didn't select any publications), ALL objects matching the queryset will be added. Using the OR query, NO objects are added, as I'd expect.
Change History (4)
comment:1 by , 5 years ago
| Component: | Forms → Database layer (models, ORM) |
|---|---|
| Summary: | Strange behaviour with QuerySet unions with ModelMultipleChoiceField → QuerySet.none() on combined queries returns all results. |
| Triage Stage: | Unreviewed → Accepted |
comment:2 by , 5 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
Thanks for the report.
QuerySet.none()doesn't work properly on combined querysets, it returns all results instead of an empty queryset.