Django m2m form save "сквозной" стол - PullRequest
17 голосов
/ 19 июня 2010

У меня проблемы с сохранением данных m2m, содержащих таблицу классов "через". Я хочу сохранить все выбранные элементы (выбранные в форме) в сквозной таблице. Но я не знаю, как инициализировать сквозную таблицу в представлении.

мой код:

class Classroom(models.Model):
     user = models.ForeignKey(User, related_name = 'classroom_creator')
     classname = models.CharField(max_length=140, unique = True)
     date = models.DateTimeField(auto_now=True)
     open_class = models.BooleanField(default=True)
     members = models.ManyToManyField(User,related_name="list of invited members", through = 'Membership')

class Membership(models.Model): 
      accept = models.BooleanField(User)
      date = models.DateTimeField(auto_now = True) 
      classroom = models.ForeignKey(Classroom, related_name = 'classroom_membership')
      member = models.ForeignKey(User, related_name = 'user_membership')

и в представлении:

def save_classroom(request):
   classroom_instance = Classroom()
   if request.method == 'POST':
        form = ClassroomForm(request.POST, request.FILES, user = request.user) 
        if form.is_valid():
           new_obj = form.save(commit=False)
           new_obj.user = request.user 
           new_obj.save()
           membership = Membership(member = HERE SELECTED ITEMS FROM FORM,classroom=new_obj)

           membership.save() 

Как правильно инициализировать членство в таблице членства?

Ответы [ 4 ]

22 голосов
/ 27 июня 2010

В случае использования нормального отношения m2m (не через промежуточную таблицу) вы можете заменить:

membership = Membership(member = HERE SELECTED ITEMS FROM FORM,classroom=new_obj)
membership.save()

с

form.save_m2m()

Но в случае использования промежуточных таблиц вам нужно вручную обрабатывать данные POST и создавать объекты Membership со всеми обязательными полями ( аналогичная проблема ). Самое основное решение состоит в том, чтобы изменить ваш взгляд на что-то вроде:

def save_classroom(request):
    if request.method == 'POST':
        form = ClassroomForm(request.POST, request.FILES)

        if form.is_valid():
           new_obj = form.save(commit=False)
           new_obj.user = request.user 
           new_obj.save()

           for member_id in request.POST.getlist('members'):
                membership = Membership.objects.create(member_id = int(member_id), classroom = new_obj)
           return HttpResponseRedirect('/')
    else:
        form = ClassroomForm()
    return render_to_response('save_classroom.html', locals())

Обратите внимание, как обрабатывается request.POST (.getlist). Это потому, что post и get являются QueryDict объектами, которые имеют некоторые последствия (request.POST ['members'] всегда будет возвращать один объект!).

Вы можете изменить этот код, чтобы сделать его более надежным (обработка ошибок и т. Д.) И более подробным, например:

member = get_object_or_404(User, pk = member_id)
membership = Membership.objects.create(member = member , classroom = new_obj)

Но обратите внимание, что вы выполняете некоторые дБ-запросы в цикле, что не очень хорошая идея в целом (с точки зрения производительности).

4 голосов
/ 26 ноября 2016

Как то, что сделал Дзида, но используйте form.cleaned_data вместо request.post:

def save_classroom(request):
    if request.method == 'POST':
        form = ClassroomForm(request.POST, request.FILES)

        if form.is_valid():
           new_obj = form.save(commit=False)
           new_obj.user = request.user
           new_obj.save()

           for member in form.cleaned_data['members'].all():
                Membership.objects.create(member = member,  classroom = new_obj)

           return HttpResponseRedirect('/')
    else:
        form = ClassroomForm()
    return render_to_response('save_classroom.html', locals())

Вы также должны учитывать, что некоторые членства могут быть удалены, поэтому:

def save_classroom(request):
    if request.method == 'POST':
        form = ClassroomForm(request.POST, request.FILES)

        if form.is_valid():
           new_obj = form.save(commit=False)
           new_obj.user = request.user
           new_obj.save()

           final_members = form.cleaned_data['members'].all()
           initial_members = form.initial['members'].all()

           # create and save new members
           for member in final_members:
                if member not in initial_members:
                    Membership.objects.create(member = member,  classroom = new_obj)

           # delete old members that were removed from the form
           for member in initial_members:
               if member not in final_members:
                   Membership.objects.filter(member = member, classroom = new_obj).delete()

           return HttpResponseRedirect('/')
    else:
        form = ClassroomForm()
    return render_to_response('save_classroom.html', locals())

Если вы используете формы моделей (как в общем CBV: form_class=ClassroomForm), переопределите и поместите логику сохранения выше в методе save, что-то вроде:

ClassroomForm(forms.ModelForm):
    members = ModelMultipleChoiceField(
        queryset=Classroom.objects.all(),
        widget=SelectMultiple
    )

    def save(self, commit=True):
        classroom = super().save(commit=False)
            if commit:
                classroom.save()
                if 'members' in self.changed_data:
                   final_members = self.cleaned_data['members'].all()
                   initial_members = self.initial['members']

                   # create and save new members
                   for member in final_members:
                        if member not in initial_members:
                            Membership.objects.create(member = member,  classroom = new_obj)

                   # delete old members that were removed from the form
                   for member in initial_members:
                       if member not in final_members:
                           Membership.objects.filter(member = member, classroom = new_obj).delete()

         return classroom
1 голос
/ 19 июня 2010

Вам также необходимо указать класс для членства:

membership = Membership(member = request.user,
                        classroom=new_obj) #if new_obj if your classroom
membership.save()

Полагаю, вам также следует удалить User в accept = models.BooleanField(User). Нет необходимости устанавливать дату при сохранении, если вы используете auto_now! Но, возможно, `auto_now_add более вероятно, что вам нужно (http://docs.djangoproject.com/en/dev/ref/models/fields/#django.db.models.DateField)

0 голосов
/ 31 декабря 2015

Вот как я это сделал в универсальном представлении на основе классов UpdateForm (django 1.8) для аналогичного, но другого приложения, использующего метод form_valid.

def form_valid(self, form):
    """
    If the form is valid, save the associated model.
    """
    self.object.members.clear()
    self.object = form.save(commit=False)
    self.object.user = self.request.user 
    self.object.save()

    list_of_members = form.cleaned_data['members']

    ClassRoom.objects.bulk_create([
            Membership(
                Course=self.object,
                member=member_person,
                order=num)
            for num, member_person in enumerate(list_of_members)
        ])
    return super(ModelFormMixin, self).form_valid(form)
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...