CODEWITHSUNDEEP
djangodjango-modelsdjango-viewsdjango-forms
igteraous
igteraous

Reputation: 117

Django forms - Allow a user to create a group for other users

I am trying to let a user create a "club" (basically a group) where the user later on can add users to.

Currently it does not create a field in the database somehow.

Any suggestions would be appreciated since I am fairly new to forms.

Model

class Club(models.Model):
    owner = models.CharField(max_length=30)
    topic = models.CharField(max_length=30)
    start = models.DateTimeField(verbose_name='start date', auto_now_add=False)
    end = models.DateTimeField(verbose_name='end date', auto_now_add=False)
    account = models.ManyToManyField(Account)

Views

@login_required
def add_club(request):

    if request.method == "POST":

        form = AddClubForm(request.POST, instance=request.user)
        print(form)

        if form.is_valid():
            form.save()

        return HttpResponseRedirect(request.path_info)

    else:
        form = AddClubForm(instance=request.user)

    return render(request, 'page/club.html', {
        "form": form,
    })

Form

class AddClubForm(forms.Model):
    
    owner = forms.CharField(required=True)
    topic = forms.CharField(required=False)
    start = forms.DateField(required=False)
    end = forms.DateField(required=False)

    class Meta:
        model = Club
        fields = (
            'owner',
            'topic',
            'start',
            'end',
        )

Template

<form method="POST" action="">
    {% csrf_token %}
    <div class="col-md-6">
        <label class="labels">Create a club</label>
        {{ form.owner }}
        <input class="btn" type="submit" value="Add club">
    </div>
</form>

Upvotes: 2

Views: 374

Answers (1)

Guillaume
Guillaume

Reputation: 2006

Since this answered your problem, I am posting the solution here:

You need to add blank=True, null=True to your fields in your model, otherwise it expects them when saving the form.

class Club(models.Model):
    owner = models.CharField(max_length=30)
    topic = models.CharField(max_length=30, blank=True, null=True)
    start = models.DateTimeField(verbose_name='start date', auto_now_add=False, blank=True, null=True)
    end = models.DateTimeField(verbose_name='end date', auto_now_add=False, blank=True, null=True)
    account = models.ManyToManyField(Account, blank=True)

Concerning the instance=request.user, I believe you misunderstanding the use of instance in a ModelForm.

If the request method is GET, the instance is used to populate a ModelForm with data from an existing Club object, and then pass it to your template to display the information. If the request method is POST (or PUT), instance represent the existing Club object you want to update with data received from the form.

You usually need to use the instance arg in a DetailView (either to update or retrieve one specific Club), never when creating an object.

That's why you need to remove the instance arg in your views.py:

form = AddClubForm(request.POST)

Upvotes: 1

Related Questions