【问题标题】:Save Many-To-Many Field Django Forms保存多对多字段 Django 表单
【发布时间】:2018-09-30 15:03:00
【问题描述】:

我的课程和导师模型之间存在多对多关系。我正在努力做到这一点,以便当您使用表格为新导师创建记录时,您可以选择他们可以教授的课程。我尽我所能尝试使用和修改this previous question 中的代码,但我并不完全理解它,所以我确定我没有做对。提交表单时,不会保存课程和导师之间的链接。我需要对代码进行哪些更改才能保存?

models.py

class Tutor(models.Model):
    FirstName = models.CharField(max_length=50)
    LastName = models.CharField(max_length=50)
    Email = models.EmailField(max_length=100)
    PhoneNumber = models.CharField(max_length=10, )
    RequestedHours = models.DecimalField(max_digits=3, decimal_places=1)

class Course(models.Model):
    SubjectID = models.ForeignKey(Subject, related_name='subjectCourse')
    Department = models.CharField(max_length=4)
    Number = models.CharField(max_length=4)
    Name = models.CharField(max_length=200)
    TutorForCourse = models.ManyToManyField(Tutor)

forms.py

class TutorForm(forms.ModelForm):
    class Meta:
        model = Tutor
        fields = ('FirstName', 'LastName', 'Email', 'PhoneNumber', 
           'RequestedHours',)
        labels = {
            'FirstName': 'First Name',
            'LastName': 'Last Name',
            'Email': 'Email',
            'PhoneNumber': 'Phone Number',
            'RequestedHours': 'Requested Hours',
        }

    courses = forms.ModelMultipleChoiceField(queryset=Course.objects.all())

    def __init__(self, *args, **kwargs):
        # Only in case we build the form from an instance
        # (otherwise, 'toppings' list should be empty)
        if kwargs.get('instance'):
            # We get the 'initial' keyword argument or initialize it
            # as a dict if it didn't exist.
            initial = kwargs.setdefault('initial', {})
            # The widget for a ModelMultipleChoiceField expects
            # a list of primary key for the selected data.
            initial['courses'] = [t.pk for t in 
                kwargs['instance'].course_set.all()]

        forms.ModelForm.__init__(self, *args, **kwargs)

    # Overriding save allows us to process the value of 'toppings' field
    def save(self, commit=True):
        # Get the unsaved Pizza instance
        instance = forms.ModelForm.save(self, False)

        # Prepare a 'save_m2m' method for the form,
        old_save_m2m = self.save_m2m

        def save_m2m():
            old_save_m2m()
            # This is where we actually link the pizza with toppings
            instance.course_set.clear()
            for course in self.cleaned_data['courses']:
                instance.course_set.add(course)

        self.save_m2m = save_m2m

        # Do we need to save all changes now?
        if commit:
            instance.save()
            self.save_m2m()

        return instance

views.py

@login_required
def tutor_new(request):
    if request.method == "POST":
        form = TutorForm(request.POST)
        if form.is_valid():
            tutor = form.save(commit=False)
            tutor.save()
            tutors = Tutor.objects.filter()
            return render(request, 'portfolio/tutor_list.html',
                          {'tutors': tutors})
    else:
        form = TutorForm()
        # print("Else")
    return render(request, 'portfolio/tutor_new.html', {'form': form})

【问题讨论】:

  • 如果这真的是你的代码、缩进和所有,你有一个缩进问题:save 嵌套在__init__ 中。如果您也发布完整的源代码会很有帮助。
  • 你对缩进问题是对的。我修复了它并编辑了帖子,但缩进似乎并没有解决问题。我想我不确定我还应该发布什么以使其成为完整的来源。你有什么建议?
  • 可以给个错误信息吗?
  • @seuling 我希望我可以,但它没有给我一个。该应用程序允许我提交表格,但在提交过程中没有将导师和课程链接在一起
  • 你的意思是没有错误,对吧?然后你尝试调试吗?比如IPython debug或者……

标签: python django django-models django-forms django-views


【解决方案1】:

更新答案

问题出在save 方法中。 您使用commit=False 保存实例,但您的实例在if commit: 时保存,所以现在没有保存。

刚刚注释掉了if commit:,会好好保存的。

def save(self, commit=True):
    # Get the unsaved Pizza instance
    instance = forms.ModelForm.save(self, False)

    # Prepare a 'save_m2m' method for the form,
    old_save_m2m = self.save_m2m

    def save_m2m():
        old_save_m2m()
        # This is where we actually link the pizza with toppings
        instance.course_set.clear()
        for course in self.cleaned_data['courses']:
            instance.course_set.add(course)

    self.save_m2m = save_m2m

    # Do we need to save all changes now?
    # Just like this
    # if commit:
    instance.save()
    self.save_m2m()

    return instance

【讨论】:

  • 很高兴为您提供帮助 :) 与 django 共度美好的一天!
最近更新 更多