Opened 9 years ago
Last modified 9 years ago
#27976 closed Cleanup/optimization
label_from_instance fails silently: form field disappears. — at Version 1
| Reported by: | Wim Feijen | Owned by: | nobody |
|---|---|---|---|
| Component: | Forms | Version: | 1.8 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description (last modified by )
Hi,
In a forms init method, I am using a foreign-key pulldown. I'd like to rename the labels, like this:
self.fields['lawyer'].label_from_instance = lambda obj: obj.userprofile.name
However, when obj has no userprofile, the code does not raise an Exception, as expected, but fails silently and the field disappears from the form output. I find this dangerous and would just expect the field to raise an Exception, so I can fix it, instead of causing abnormalities.
Part of my code in more detail is here. I'm using UserProfiles because I find them way easier to understand than extended user models.
class UserProfileForm(forms.ModelForm):
"""General form to add a user. Sanne-only."""
group = forms.ModelChoiceField(queryset=Group.objects.order_by('name'), required=True, widget=forms.RadioSelect, initial=lawyer_group)
email = forms.EmailField(required=True, label="Email address")
def __init__(self, *args, **kwargs):
super(UserProfileForm, self).__init__(*args, **kwargs)
lawyer_users = User.objects.filter(groups__name__in=['firm lawyer', 'client lawyer']).distinct().order_by('userprofile__name')
self.fields['lawyer'].queryset = lawyer_users
self.fields['lawyer'].label_from_instance = lambda obj: obj.userprofile.name
class Meta:
model = UserProfile
fields = ['group', 'name', 'email', 'vendor', 'customer', 'city', 'country', 'photo', 'tel', 'tel2', 'bar_city', 'bar_date_of_admission', 'lawyer']
The form is outputted as divs using crispy_forms