Opened 14 years ago

Last modified 13 years ago

#13293 closed

r12900 breaks extra() with additional select clauses (Only applies to MySQL SQLCompiler) — at Version 2

Reported by: Matthias Kestenholz Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Keywords: MySQL
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Ramiro Morales)

[12900] breaks the following query:

Page.objects.filter(...).extra(select={'_url_length': 'LENGTH(_cached_url)'}).order_by('-_url_length')[0]

LENGTH(_cached_url) is prepended to the list of fields which should be selected, which makes the row tuple one element longer than the fields tuple:

http://code.djangoproject.com/browser/django/trunk/django/db/backends/mysql/compiler.py#L6

Note that this bug only manifests itself under the following conditions:

  1. The first model field is an AutoField
  2. The second field is a (Null)?BooleanField
  3. The retrieved model has a PK of 1L (0L would also work, but auto incrementing primary key fields never have a value of 0L)

The resulting model instance has True instead of 1L as its PK.

Change History (3)

by Matthias Kestenholz, 14 years ago

Attachment: 13293.patch added

Now with a regression test

comment:1 by Matthias Kestenholz, 14 years ago

Has patch: set
Keywords: MySQL added
Summary: r12900 breaks extra() with additional select clausesr12900 breaks extra() with additional select clauses (Only applies to MySQL SQLCompiler)

comment:2 by Ramiro Morales, 14 years ago

Description: modified (diff)

(reformatted description)

Note: See TracTickets for help on using tickets.
Back to Top