Opened 13 years ago

Last modified 13 years ago

#17325 closed New feature

Add **kwargs to is_valid and several validation methods — at Initial Version

Reported by: jorgeecardona Owned by: nobody
Component: Forms Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I like to use forms a lot, but sometimes I have to overwrite init to accept some parameters that change the behaviour of the validation, I was thinking that maybe is better just to add a way to pass some data to the validation methods of the form in order to validate it.

Imagine that I want to create a form to change the email of an user and I don't want to use an argument in the request for "username" and use the username from the session framework (I just want to avoid to send the username in the request), I will have to overwrite the init of the form in order to accept "user" as an argument and store it in self.user to be used later in clean_email. Instead, if we have this new argument in is_valid is just to do: form.is_valid(user=user).

This new argument doesn't hurt anyone, and It will help a lot to some of us that use form in this way.

Change History (0)

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