Opened 17 years ago

Closed 17 years ago

#5328 closed (wontfix)

Small ChoiceField.clean() fix

Reported by: Ramiro Morales Owned by: Adrian Holovaty
Component: Forms Version: dev
Severity: Keywords: choicefield clean newforms empty EMPTY_VALUES
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

While reviewing #5327 I spotted a slightly sub-optimal code snippet in the aforementioned method. The aattached patch fixes it.

Attachments (1)

newforms-choice-field-clean-r6040.diff (625 bytes ) - added by Ramiro Morales 17 years ago.

Download all attachments as: .zip

Change History (2)

by Ramiro Morales, 17 years ago

comment:1 by Chris Beaven, 17 years ago

Resolution: wontfix
Status: newclosed

But this changes the logic of the clean function slightly - value may be in EMPTY_VALUES after it has been parsed by smart_unicode

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