Opened 12 years ago

Closed 12 years ago

Last modified 9 years ago

#10549 closed (fixed)

BooleanField.formfield() does not handle choices correct. [PATCH]

Reported by: Sebastian Noack Owned by: nobody
Component: Database layer (models, ORM) Version: 1.0
Severity: 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

If you have a BooleanField, with choices (as in the example below), the formfield() method always returns a TypedChoicesField, with a blank choice at the beginning. This is because of this entry is included always if the blank is True and BooleanFields are always blank=True. So we have to check for null instead of blank, when using BooleanFields, since the blank flag in the BooleanfField does not specify if the field can be empty in forms in contrast to other fields.

FLAG_CHOICES = (
    (0, _('No'),
    (1, _('Yes'),
)

class Foo(models.Model):
    flag = models.BooleanField(choices=FLAG_CHOICES)

Attachments (1)

fixed-choices-handling-of-BooleanFields.patch​ (770 bytes) - added by Sebastian Noack 12 years ago.

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by Sebastian Noack

Attachment: fixed-choices-handling-of-BooleanFields.patch​ added

comment:1 Changed 12 years ago by Jacob

milestone: → 1.1
Triage Stage: Unreviewed → Accepted

comment:2 Changed 12 years ago by Ramiro Morales

See also #9640

comment:3 Changed 12 years ago by Jacob

Resolution: → fixed
Status: new → closed

(In [10500]) Fixed #9640, #10549: BooleanFields with choices, a default, and null=False now correctly doesn't generate a blank option.

comment:4 Changed 12 years ago by Chris Beaven

I must say, it is rather amusing how this tests for something which you can't actually do, due to r10456

comment:5 Changed 9 years ago by Jacob

milestone: 1.1

Milestone 1.1 deleted

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