Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Iterate over the form using form.__iter__ to retrieve a more complete field #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bartek
Copy link
Contributor

@bartek bartek commented Jan 15, 2013

Hi there!

Not sure if this is necessary but I found that field.label in the existing instance would always be empty, unless specified, where-as normally Django prettifies this if its empty. I noticed that if you iterate over the fields in the form using the forms __iter__ method you are returned a BoundField, which does some niceties like set the label to a pretty_name

There may be some more things that can be done given this field is now passed around, perhaps improving the widget rendering :)

Hmm .. need to write some tests eventually D:

mgarcian referenced this pull request in A3Sec/django-remote-forms Aug 6, 2021
Filter form data to include only form fields data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant