How to initialize form with some parameters of model instance

Question

I would like be able to send SMS/Email notifications manually using the groups/users of a model instance. Let's say the model looks like this:

class Memo(models.Model):
    title = models.CharField(max_length=100)
    receiver = models.ManyToManyField(EmployeeType, related_name='memos_receiver')

I pass the object instance to the view:

path('<int:pk>/notify', NotificationView.as_view(), name='memos-notify'),

The form and the view are where I am having trouble. I figure I should be able to just pass the forms initial fields right there in the view:

class NotificationView(FormView):
    template_name = 'memos/notification_form.html'
    form_class = MemoNotificationForm
    success_url = reverse_lazy('overview')

    def get_initial(self):
        initial = super(NotificationView, self).get_initial()
        memo = Memo.objects.filter(id=id)
        initial['receiving_groups'] = memo.receiver.all()
        return initial

And the form looks like this:

class MemoNotificationForm(forms.Form):
    class Meta:
        fields = [
            'receiving_groups'
        ]

    receiving_groups = forms.MultipleChoiceField(
        required=False,
        widget=forms.CheckboxSelectMultiple)

*Note: the receiving_groups will be the ones receiving the notification. Once the form is valid I will apply a send_sms method send it.

TypeError: int() argument must be a string, a bytes-like object or a number, not 'builtin_function_or_method'

Do I need to initialize the queryset in the form? Would appreciate it if someone could paint me a clear picture of the why and how here. Thank you!

You need to pass the queryset and not just the related manager - initial['receiving_groups'] = memo.receiver.all() — Iain Shelvington
– Iain Shelvington, Commented Dec 10, 2019 at 3:05
I knew that looked wrong. That is one issue but still get the >TypeError — Nothing here
– Nothing here, Commented Dec 10, 2019 at 3:06

JPG · Accepted Answer · 2019-12-10 03:48:09Z

1

The error is because of this line,

memo = Memo.objects.filter(id=id)

Here, in your scope, the id becoming python's built-in fucntion and hence the error.

To access the URL parameter, you should use self.kwargs attribute, as below

class NotificationView(FormView):
    template_name = 'memos/notification_form.html'
    form_class = MemoNotificationForm
    success_url = reverse_lazy('overview')

    def get_initial(self):
        initial = super(NotificationView, self).get_initial()
        memo = Memo.objects.filter(id=self.kwargs['pk'])
        initial['receiving_groups'] = memo.receiver.all()
        return initial

You can find the working example from the official Django documentaion here, Dynamic filtering

edited Dec 10, 2019 at 3:48

answered Dec 10, 2019 at 3:45

JPG

89.4k24 gold badges164 silver badges235 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Nothing here Over a year ago

I have to give you a updoot because that handles the TypeError and you explained it well. Though now it is giving KeyError at /memos/1/notify 'id'

JPG Over a year ago

Sorry, my bad. It should be self.kwargs['pk'] (check my updated answer)

Nothing here Over a year ago

I am going to call this the correct answer to the specific question though I have other errors to deal with. Thanks and good job!

Collectives™ on Stack Overflow

How to initialize form with some parameters of model instance

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related