#30494 closed Bug (fixed)
Problem with ExtractYear()+1 in queries
Reported by: | Alexey Chernov | Owned by: | Simon Charette |
---|---|---|---|
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
It looks like django doesn't create a correct MySQL query when having a simple arithmetics with ExtractYear().
I have a model with two data fields:
class Event(models.Model): start_date = models.DateField() finish_date = models.DateField()
If I want to find all rows such that the years of start_date and finish_date are equal, I can do it so:
from django.db.models import F from django.db.models.functions import ExtractYear >>> print Event.objects.annotate(year=ExtractYear(F('start_date'))).filter(finish_date__year=F('year')).only('pk').query SELECT `event`.`id`, EXTRACT(YEAR FROM `event`.`start_date`) AS `year` FROM `event` WHERE EXTRACT(YEAR FROM `event`.`finish_date`) = (EXTRACT(YEAR FROM `event`.`start_date`))
But if I want to find events that start and finish in consequent years, I try the following filter, and it gives me an incorrect MySQL query:
>>> print Event.objects.annotate(year=ExtractYear(F('start_date'))).filter(finish_date__year=F('year')+1).only('pk').query SELECT `event`.`id`, EXTRACT(YEAR FROM `event`.`start_date`) AS `year` FROM `event` WHERE `event`.`finish_date` BETWEEN 0001-01-01 AND 0001-12-31
I tried to replace F('year')+1
with F('year')+Value(1)
but it didn't help.
Am I wrong somewhere, or does it look like a bug?
Change History (5)
comment:1 by , 5 years ago
Version: | 2.2 → 1.11 |
---|
comment:2 by , 5 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Triage Stage: | Unreviewed → Accepted |
Version: | 1.11 → master |
Note:
See TracTickets
for help on using tickets.
I confirmed this is still an issue at 1d0bab0bfd77edcf1228d45bf654457a8ff1890d.
The issue likely lies in
YearExact.as_sql
https://github.com/django/django/blob/1d0bab0bfd77edcf1228d45bf654457a8ff1890d/django/db/models/lookups.py#L507-L525