Opened 6 years ago

Closed 5 years ago

#14184 closed Bug (fixed)

Validators not called on MultiValueField

Reported by: anonymous Owned by: Paul Collins
Component: Forms Version: 1.2
Severity: Normal Keywords: MultiValueField Validators
Cc: Paul Collins Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Validators are not called for the compressed value of a MultiValueField.

If, say, I have a MultiValueField subclass which takes a first name and a last name and compresses the two into a "Lastname, Firstname" and this MultiValueField subclass was instantiated using validators = [disallow_john_doe] , one would expect that disallow_john_doe(compressed value) would be executed. However, this is not the case.

Looking at the source, it appears that while the clean() method of django.forms.fields.Field calls run_validators() , the clean() method of django.forms.fields.MultiValueField does not. This is different from expected behaviour and is not documented to my knowledge.

The workaround is to implement validate() in the subclass.

Attachments (2)

add_run_validators_call.patch (2.2 KB) - added by Paul Collins 5 years ago.
add_run_validators_call.2.patch (2.0 KB) - added by Paul Collins 5 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by dmoisset

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

Either the documentation or MultiValueField are wrong. (I'd say that MultiValueField is). Accepting

comment:2 Changed 5 years ago by Julien Phalip

Severity: Normal
Type: Bug

Changed 5 years ago by Paul Collins

comment:3 Changed 5 years ago by Paul Collins

Cc: Paul Collins added
Easy pickings: unset
Has patch: set
Owner: changed from nobody to Paul Collins
UI/UX: unset

Was a bit surprised by this one myself. I'm working on the idea that the MultiValueField is wrong rather than the docs since the validators arg is a core Field arg. Patch includes a regression test for this and a fix which appears to resolve the issue.

Version 0, edited 5 years ago by Paul Collins (next)

Changed 5 years ago by Paul Collins

comment:4 in reply to:  3 Changed 5 years ago by Paul Collins

Replying to paulcollins:

On a related note MultiValueField.validate is overridden to be a noop, but the parent class Field checks if the field is required. I've left it alone but, should MultiValueField.validate be removed so that the Field.validate gets called instead?

Never mind, eventually rereading the clean function for the MultiValueField answered my question. Seems the clean method already does the work of the Field.validate method.

comment:5 Changed 5 years ago by Claude Paroz

Triage Stage: AcceptedReady for checkin

comment:6 Changed 5 years ago by Jannis Leidel

Resolution: fixed
Status: newclosed

In [17430]:

Fixed #14184 -- Enabled running the validators in MultiValueFields. Thanks, paulcollins.

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