modelformset_factory не сохраняет экземпляры в базу данных

Я пытаюсь создать набор форм с экземплярами на моей странице, но набор форм вообще не сохраняет экземпляры. Сохранение дополнительных полей работает хорошо, но я не могу понять, что не так с экземплярами. У меня много форм на странице, и это последняя, ​​которая не сохраняется: Мой вид:

def details(request, username, slug):
    sm = Social_media.objects.all()
    profile = get_object_or_404(Profiles, user__username=username)
    site = get_object_or_404(MySites, slug=slug, user__username=username)
    ftp = get_object_or_404(FTP, site=site)
    FtpPathFormSet = forms.modelformset_factory(FtpPath,form=FtpPathForm,extra=1)
    sjim_ops = SjimOperations.objects.filter(user=request.user).order_by('-date')
    sjim_det = SjimDetails.objects.all()

    if request.method == 'POST':

    // other forms and logic 

       if 'change3' in request.POST:
            newsite = NewSiteForm(instance=site)
            newftp = NewFtpForm(instance=ftp)
            ftppath = FtpPathFormSet(request.POST)
            if ftppath.is_valid:
                print('here')
                i = 0
                instances = ftppath.save(commit=False)
                for instance in instances:
                    print('here2')
                    instance.ftp = ftp
                    instance.recursive = request.POST.get('form-'+str(i)+'-recursive')

                    if instance.recursive == 'on':
                        instance.recursive = True
                    else:
                        instance.recursive = False

                    instance.period = request.POST.get('form-'+str(i)+'-period')
                    try:
                        testconnect = ftplibr(ftp.host)
                        testconnect.login(user=ftp.ftpuser, passwd=ftp.password)
                        testconnect.cwd(instance.path)
                        instance.find = True
                        testconnect.quit()
                    except:
                        instance.find = False

                    ftppath.save()
                    i =+ 1
                return redirect('main:details', username=username, slug=site.slug)

модель:

class FtpPath(models.Model):
    period = (
        ('Раз в сутки','Раз в сутки'),
        ('Раз в неделю','Раз в неделю'),
        ('Раз в 2 недели','Раз в 2 недели')
        )

    ftp = models.ForeignKey(FTP, on_delete=models.CASCADE)
    path = models.CharField(max_length=200, blank=True)
    period = models.CharField(choices=period, max_length=20, null=True, blank=True)
    find = models.BooleanField(null=True, default=False)
    recursive = models.BooleanField(null=True, default=False)

    class Meta:
        verbose_name = 'FTP path'
        verbose_name_plural = 'FTP paths'

Форма:

class FtpPathForm(forms.ModelForm):
    path = forms.CharField(widget=forms.TextInput(attrs={'type':'text','class':'effect-16'}), required=False)
    recursive = forms.BooleanField(widget=forms.CheckboxInput(attrs={}), required=False)
    period = forms.CharField(widget=forms.TextInput(attrs={'type':'text','style':'display:none','class':'period-input'}), required=False, label='')

    class Meta:
        model = FtpPath
        fields = ('path', 'recursive','period', 'find')

шаблон:

<form method="POST" id="path-form">
       {% csrf_token %}
       {{ ftppath.management_form }}
       <div class="info_tab_3col">
        <div class="form_title">Выбрать папки</div>
        <div class="form_folders">
        {% for ftppath in ftppath %}
        {% if ftppath.find.value == True %}
         <div class="form_folder">
         {% else %}
        <div class="form_folder form_folder_error">
        {% endif %}
        <div class="ftp_form_item">
        <div class="ftp_f_long ftp_f_long_i">
        <div class="input-effect">
        {{ ftppath.path }}
        <label>Путь на сервере</label>
        <span class="focus-border"></span>
        </div>
        </div>
        <div class="ftp_form_s ftp_form_s_i">
        <div class="checkbox_box">
        {{ ftppath.recursive }}
        <label for="id_form-{{ forloop.counter0 }}-recursive">
        <span><!-- This span is needed to create the "checkbox" element --></span>
        Рекурсивно
        </label>
        </div>
        </div>
        </div>
        <div class="ftp_form_item ftp_form_item_type">
        <div class="select_wrapper">
        {{ ftppath.period }}
        <div class="select_wrapper_val"></div>
        <span class="select_wrapper_label">Период</span>
        <span class="input_error error-ftppath"></span>
        <div class="select_list">
        <div class="select_list_item">Раз в сутки</div>
        <div class="select_list_item">Раз в неделю</div>
        <div class="select_list_item">Раз в 2 недели</div>
        </div>
        </div>
        </div>
           </div>
       {% endfor %}
       </div>
        <div class="form_button form_button_ftp">
               <button class="btn" type="submit" name="change3">
                   <span>Изменить</span>
                </button>
         </div>
      </div>
         </form>

Таким образом, набор форм отображается на странице нормально: у меня есть 1 экземпляр и одно дополнительное поле. Если я нажму поле отправки, которое уже экземпляр не меняется в базе данных и на странице. Если я изменю в своем представлении is_valid: на is_valid():, похоже, что форма сохраняется на странице, но не в базе данных, поэтому, если я перейду на другую страницу и вернусь, ничего не будет сохранено. Если я уберу всю логику, на мой взгляд, все будет работать так же, и я действительно не знаю, что я делаю неправильно, пожалуйста, помогите!


person Andrej Vilenskij    schedule 11.12.2018    source источник


Ответы (1)


Проблема в том, что вы должны вызывать instance.save() вместо ftppath.save() в конце вашего цикла.

Также обратите внимание, что is_valid — это метод, вам нужно вызвать его: if ftppath.is_valid():

И также обратите внимание, что очень не в Python хранить такой счетчик и увеличивать его вручную. Вместо этого используйте enumerate.

Но на самом деле вы все равно не хотите использовать ни один из них; вместо того, чтобы получать данные из POST путем объединения идентификатора формы, вы должны получить их из самой формы clean_data. Если вы это сделаете, вам не нужно вручную сравнивать его с «включено»; это то, что процесс очистки формы уже делает для вас.

Таким образом, вы должны перебирать формы, а затем сохранять каждую из них:

if ftppath.is_valid():
    for form in ftppath.forms:
        instance = form.save(commit=False)
        instance.ftp = ftp
        instance.recursive = form.cleaned_data['recursive']
        instance.period = form.cleaned_data['period']
        try:
           ...
        except ftplib.all_errors:   # always use a specific error
           ...
        instance.save()

Наконец, вы уверены, что вам нужно даже установить эти два значения вручную? Это поля модели, указанные в атрибуте полей формы, поэтому Django уже должен установить их для вас.

person Daniel Roseman    schedule 11.12.2018
comment
Большое спасибо за ваши советы о том, как сделать мой код лучше, и ваш ответ. Полностью проблема была в поле идентификатора, которое требуется на странице для каждой формы в наборе форм. Вот почему набор форм не может пройти проверку. - person Andrej Vilenskij; 11.12.2018