How to set a field of the model in view using generic views?

Question

I have a model, which has an author ForeignKey, as such:

class Appointment(models.Model):
    # ...
    author = models.ForeignKey(User)

I want the author field to be set automatically when creating appointment to currently logged-in user. In other words, the author field should not appear in my Form class:

class AppointmentCreateForm(ModelForm):
    class Meta:
        model = Appointment
        exclude = ('author')

There are two problems:

How to access the form in generic CreateView and set the author?
How to tell the form to save the excluded field along with the values read from user input?

score 7 · Answer 1 · answered Jul 29 '14 at 15:31

7

The following seem slightly simpler. Note that self.request is set in View.as_view

class AppointmentCreateView(CreateView):        
    model=Appointment
    form_class = AppointmentCreateForm

    def get_form(self, form_class):
        form = super(AppointmentCreateView, self).get_form(form_class)
        # the actual modification of the form
        form.instance.author = self.request.user
        return form

answered Jul 29 '14 at 15:31

Robert Jørgensgaard Engdahl

3,234
22
34

1

the right signature woudn't be `def get_form(self, form_class**=None**): ` ? – Allan Vital Jun 02 '17 at 16:52
1

You are absolutly right @AllanVital, it has been like you write since Django 1.8 (I stopped using Django right about when 1.7 came out) – Robert Jørgensgaard Engdahl Jun 03 '17 at 18:35

score 2 · Accepted Answer · answered Dec 01 '12 at 12:35

I've modified my generic view subclass as such:

class AppointmentCreateView(CreateView):        
    model=Appointment
    form_class = AppointmentCreateForm

    def post(self, request, *args, **kwargs):
        self.object = None
        form_class = self.get_form_class()
        form = self.get_form(form_class)

        # the actual modification of the form
        form.instance.author = request.user

        if form.is_valid():
            return self.form_valid(form)
        else:
            return self.form_invalid(form)

There are few important parts here:

I modified form instance field, which holds the actual model that's going to be saved.
You can of course get rid of the form_class
The post method that I've needed to modify was two classes above in hierarchy, so I needed to incorporate the base code and the self.object = None line, merging the overload and base into one function (I'm not calling super in post).

I think it's a good way to solve pretty common problem, and once again I don't have to write my own custom view.

How to set a field of the model in view using generic views?

2 Answers2

Linked