FormCollection submission returns status 200 but no data is saved in the database.

[DallyMorgan]

Hey geeks,

I’m learning Django-Formset, and I’m facing an issue when saving data from a FormCollection. When sending the AJAX request, I receive a 200 status, but no data is saved in the database.

views.py
class addprojet(FormCollectionView,FormViewMixin,IncompleteSelectResponseMixin):
collection_class = ProjetCollection

template_name = 'app/add_projet.html'
success_url = reverse_lazy("projets")

models

class Perimetre(models.Model):

code = models.CharField(max_length=50, unique=True)
intitule = models.CharField(max_length=255)
ministere = models.ForeignKey(Ministere, on_delete=models.CASCADE)
plan_strategique = models.ForeignKey(Plan_strat, on_delete=models.CASCADE)
description = models.TextField()
objectif_general = models.TextField()
objectifs_specifiques = models.TextField()
province = models.ForeignKey(Province, on_delete=models.SET_NULL, null=True)
departement = models.ForeignKey(Departement, on_delete=models.SET_NULL, null=True)
canton = models.ForeignKey(Canton, on_delete=models.SET_NULL, null=True)
regroupement_village = models.ForeignKey(RegroupementVillage, on_delete=models.SET_NULL, null=True)
social = models.TextField()
economique = models.TextField()
environnemental = models.TextField()
date_demarrage = models.DateField()
date_effective = models.DateField()
date_fin = models.DateField()
duree = models.CharField(max_length=50)
maitre_ouvrage = models.CharField(max_length=255)
maitre_ouvrage_delegue = models.CharField(max_length=255)
maitre_oeuvre = models.CharField(max_length=255)
unite_gestion_projet = models.CharField(max_length=255)
parties_prenantes = models.TextField()
observations_perimetre = models.TextField()
status = models.CharField(max_length=50,choices=[('En cours','En cours'),('Termine','Termine'),('Suspendu','Suspendu')],default='En cours')

class Meta:
    
    verbose_name = 'Plan de gestion du perimetre'
    verbose_name_plural = "Plan de gestion du perimetre"

    indexes = [
        models.Index(fields=['code'], name='code_index'),
        models.Index(fields=['intitule'], name='intitule_index'),
    ]

class Integration(models.Model):

perimetre = models.OneToOneField(Perimetre, on_delete=models.CASCADE)
rapport_demarrage = models.BooleanField()
date_pub_rapp = models.DateField("Date de publication du rapport",null=True, blank=True)
observation_rapport = models.TextField(null=True, blank=True)
document_prealable = models.ManyToManyField(Document, related_name="projets_prealables")
document_demarrage = models.ManyToManyField(Document, related_name="projets_demarrage")
document_travaux = models.ManyToManyField(Document, related_name="projets_travaux")
document_cloture = models.ManyToManyField(Document, related_name="projets_cloture")
observations_integration = models.TextField()

class PerimetreForm(FormMixin, forms.ModelForm):

def __init__(self, *args, **kwargs):
    super(PerimetreForm, self).__init__(*args, **kwargs)
    for field_name, field in self.fields.items():
        field.widget.attrs['class'] = field.widget.attrs.get('class', '') + ' form-control form-control-sm'

step_label = "Plan de gestion de périmètre "
induce_activate = 'integration.previous:active'
next = Activator(
    label="prochain",
    widget=Button(action='submitPartial -> activate("apply")')
)


default_renderer = FormRenderer(
    form_css_classes='row',
    field_css_classes={
        '*': 'mb-2 col-12 ',
        'code': 'col-6',
        'intitule': 'col-6',
        'ministere': 'col-6',
        'plan_strategique': 'col-6',
        'description': 'col-12',
        'objectif_general': 'col-6',
        'objectifs_specifiques': 'col-6',
        'province': 'col-3',
        'departement': 'col-3',
        'canton': 'col-3',
        'regroupement_village': 'col-3',
        'social': 'col-4',
        'economique': 'col-4',
        'environnemental': 'col-4',
        'date_demarrage': 'col-3',
        'date_effective': 'col-3',
        'date_fin': 'col-3',
        'duree': 'col-3',
        'maitre_ouvrage': 'col-3',
        'maitre_ouvrage_delegue': 'col-3',
        'maitre_oeuvre': 'col-3',
        'unite_gestion_projet': 'col-3',
        'parties_prenantes': 'col-6',
        'observations_perimetre': 'col-6',
        'status': 'col-12',
      
    },
)

class Meta:
    model = Perimetre
    fields = [
        'code', 'intitule', 'ministere', 'plan_strategique', 'description',
        'objectif_general', 'objectifs_specifiques', 'province', 'departement', 'canton', 'regroupement_village',
        'social', 'economique', 'environnemental', 'date_demarrage', 'date_effective', 'date_fin', 'duree',
        'maitre_ouvrage', 'maitre_ouvrage_delegue', 'maitre_oeuvre', 'unite_gestion_projet',
        'parties_prenantes', 'status', 'observations_perimetre'
    ]

    widgets = {
        'ministere': Selectize(search_lookup='nom__icontains'),
        'date_demarrage': DateInput(),
        'date_effective': DateInput(),
        'date_fin': DateInput(),
        'province': Selectize(search_lookup='nom__icontains'),
        'departement': Selectize(search_lookup='nom__icontains'),
        'canton': Selectize(search_lookup='nom__icontains'),
        'regroupement_village': Selectize(search_lookup='nom__icontains'),
        
    }

class IntegrationForm(FormMixin, forms.ModelForm):

def __init__(self, *args, **kwargs):
    super(IntegrationForm, self).__init__(*args, **kwargs)
    for field_name, field in self.fields.items():
        field.widget.attrs['class'] = field.widget.attrs.get('class', '') + ' form-control form-control-sm'

step_label = "Plan de gestion de l'integration"
induce_activate = 'perimetre.next:active'
previous = Activator(
    label="Precedent",
    widget=Button(action='activate("apply")'))

submit = Activator(
        label="Submit",
        widget=Button(action='submit -> intercept("#submit-data") !~ intercept("#submit-data")')
    )

default_renderer = FormRenderer(
    form_css_classes='row',
    field_css_classes={
        '*': 'mb-2 col-12 ',
        'code': 'col-6',
        'intitule': 'col-6',
        'ministere': 'col-6',
        'plan_strategique': 'col-6',
        'description': 'col-12',
        'objectif_general': 'col-6',
        'objectifs_specifiques': 'col-6',
        'province': 'col-3',
        'departement': 'col-3',
        'canton': 'col-3',
        'regroupement_village': 'col-3',
        'social': 'col-4',
        'economique': 'col-4',
        'environnemental': 'col-4',
        'date_demarrage': 'col-3',
        'date_effective': 'col-3',
        'date_fin': 'col-3',
        'duree': 'col-3',
        'maitre_ouvrage': 'col-3',
        'maitre_ouvrage_delegue': 'col-3',
        'maitre_oeuvre': 'col-3',
        'unite_gestion_projet': 'col-3',
        'parties_prenantes': 'col-6',
        'status': 'col-6',
        'observations_perimetre': 'col-12',
    },
)

class Meta:
    model = Integration
    fields = [ 'rapport_demarrage', 'date_pub_rapp', 'observation_rapport', 'observations_integration','document_prealable', 'document_demarrage','document_travaux','document_cloture' ]   

    
    widgets = {
    
        'date_pub_rapp': DateInput(attrs={'df-show': ".rapport_demarrage=='on'"}),
        'observation_rapport': TextInput(attrs={'df-show': ".rapport_demarrage=='null'"}),
        
        'observation_rapport': TextInput(), 
        'document_prealable': SelectizeMultiple(search_lookup='nom__icontains'),
        'document_demarrage': SelectizeMultiple(search_lookup='nom__icontains'),
        'document_travaux': SelectizeMultiple(search_lookup='nom__icontains'),
        'document_cloture': SelectizeMultiple(search_lookup='nom__icontains'),
    }


def model_to_dict(self, perimetre):
    try:
        return model_to_dict(perimetre.integration, fields=self._meta.fields, exclude=self._meta.exclude)
    except Integration.DoesNotExist:
        return {}    
    
def construct_instance(self, perimetre):
    try:
        integration = perimetre.integration
    except Integration.DoesNotExist:
        integration = Integration(perimetre=perimetre)
    form = IntegrationForm(data=self.cleaned_data, instance=integration)
    if form.is_valid():
        construct_instance(form, integration)
        form.save()

class ProjetCollection(StepperCollection):
legend = "Enregister un nouveau projet"
perimetre = PerimetreForm()
integration = IntegrationForm()

[jrief]

Why did you use FormCollectionView instead of formset.views.EditCollectionView as described in https://django-formset.fly.dev/model-collections/?

Please read the contribution guidelines on how to create issues. Your's isn't even formatted properly.

[DallyMorgan]

"I thought EditCollectionView was for updates and FormCollectionView for creation.
I modified the view to

class addprojet(EditCollectionView):
model = Perimetre
collection_class = ProjetCollection
template_name = 'app/add_projet.html'
success_url = reverse_lazy("projets")

and now I get this error "Generic detail view addprojet must be called with either an object pk or a slug in the URLconf." I also note that my form uses StepperCollection."

[jrief]

I now updated the documentation which hopefully explains this pattern better.
https://django-formset.fly.dev/model-collections/

You can also have a look at this example https://django-formset.fly.dev/bootstrap/company whose code can be found here:
https://github.com/jrief/django-formset/blob/releases/1.7/testapp/forms/company.py