Django CreateView: Comment faire pour effectuer une action sur enregistrer

Je suis en utilisant une mesure CreateView (CourseCreate) et UpdateView (CourseUpdate) la sauvegarde et la mise à jour d'un Cours. Je veux prendre une action lorsque le Cours est enregistré. Je vais créer une nouvelle plusieurs-à-plusieurs relation entre le formateur des nouveaux cours et de l'utilisateur (si elle n'existe pas déjà).

Donc, je veux sauver le Cours comme cours, et ensuite utiliser bien sûr.faculté de créer cette nouvelle relation. Où est le meilleur endroit pour ce faire?

Je suis en train de le faire dans form_valid dans les points de vue, mais j'ai des erreurs lorsque vous essayez d'accéder à un formulaire.exemple.faculté bc le parcours n'est pas encore créé (dans CourseCreate). Le message d'erreur est semblable:

"Cours: ..." doit avoir une valeur pour le champ "cours" avant cela, plusieurs-à-plusieurs relations peuvent être utilisés.

C'est aussi ne fonctionne pas dans CourseUpdate. L'Aide de la relation n'est pas créé. Devrais-je essayer ce dans le Formulaire? Mais je ne suis pas sûr de savoir comment obtenir les infos de l'utilisateur vers le Formulaire.
Merci.

models.py

class Faculty(models.Model):
    last_name = models.CharField(max_length=20)

class Course(models.Model):
    class_title = models.CharField(max_length=120)
    faculty = models.ManyToManyField(Faculty)

class UserProfile(models.Model):
    user = models.OneToOneField(User)
    faculty = models.ManyToManyField(Faculty, through='Assists')

class Assists(models.Model):
    user = models.ForeignKey(UserProfile)
    faculty = models.ForeignKey(Faculty)

views.py

class CourseCreate(CreateView):
    model = Course
    template_name = 'mcadb/course_form.html'
    form_class = CourseForm
    def form_valid(self, form):
        my_course = form.instance
        for f in my_course.faculty.all():
            a, created = Assists.objects.get_or_create(user=self.request.user.userprofile, faculty=f)
        return super(CourseCreate, self).form_valid(form)

class CourseUpdate(UpdateView):
    model = Course
    form_class = CourseForm
    def form_valid(self, form):
        my_course = form.instance
        for f in my_course.faculty.all():
            a, created = Assists.objects.get_or_create(user=self.request.user.userprofile, faculty=f)
        return super(CourseUpdate, self).form_valid(form)

Essayez de remplacer my_course = form.instance avec my_course = form.save(commit=False)
je vous remercie. Lorsque j'essaie que, après avoir cliqué sur Enregistrer btn lors de la création d'un nouveau cours, je vois toujours le même message d'erreur:" Le Parcours de l'objet doit avoir une valeur pour le champ "cours" avant cela, plusieurs-à-plusieurs relations peuvent être utilisés." Je n'arrive pas à comprendre d'où le "cours" de mentionner vient de... j'ai une classe nommée Sûr, mais il n'a pas un champ nommé "cours", et mon avis n'a aucune mention d'un "cours".

InformationsquelleAutor Carrie | 2015-10-07

django

43

La form_valid() méthode pour CreateView et UpdateView enregistre le formulaire, puis redirige vers la réussite de l'url. Il n'est pas possible de faire return super(), parce que vous voulez faire des trucs entre l'objet de la sauvegarde et de la redirection.

La première option est de ne pas appeler super(), et copier les deux lignes dans votre point de vue. L'avantage de cela est qu'il est très clair ce qui se passe.
```
def form_valid(self, form):
    self.object = form.save()
    # do something with self.object
    # remember the import: from django.http import HttpResponseRedirect
    return HttpResponseRedirect(self.get_success_url())
```
La deuxième option est de continuer à appeler super(), mais ne renvoie pas la réponse jusqu'à ce que après avoir mis à jour la relation. L'avantage, c'est que vous n'êtes pas dupliquer le code dans super(), mais l'inconvénient est qu'il n'est pas clair ce qu'il se passe, sauf si vous êtes familier avec ce que super() n'.
```
def form_valid(self, form):
    response = super(CourseCreate, self).form_valid(form)
    # do something with self.object
    return response
```
- Wow, merci, c'est le travail (je suis allé avec la première option pour la transparence). J'ai passé beaucoup de temps à la recherche par le biais de la documentation, mais encore du mal à résoudre ce genre de choses moi-même. Merci beaucoup @Alasdair!
- Personnellement je préfère la première option. Heureux, il a aidé 🙂
- Comment sur la substitution form_valid() et la mise à jour form.instance qui est généré à l'aide de données de POSTES, comme le résultat de form.is_valid appelé juste avant de form_valid(). Cette instance est ensuite utilisée pour définir object quand form.save() est appelé dans form_valid
- si je me souviens bien, form.instance est None pour CreateView. Vous pouvez définir instance à Course() dans get_form_kwargs, mais ce n'est pas beaucoup vous aider pour cette question en particulier, parce que l'instance doit être enregistré avant de pouvoir définir les valeurs de la plusieurs-à-plusieurs domaine.
InformationsquelleAutor Alasdair
1

Je suggère d'utiliser Django du Signal. C'est une action qui se déclenche quand quelque chose arrive à un Modèle, comme enregistrer ou mise à jour. De cette façon, votre code reste propre (pas de logique métier dans le formulaire de manutention), et de vous assurer qu'il ne se déclenche après enregistrer.
```
#views.py
from django.dispatch import receiver
...

@receiver(post_save, sender=Course)
def post_save_course_dosomething(sender,instance, **kwargs):
    the_faculty = instance.faculty
    #...etc  
```
- Cela ne fonctionnera pas dans ce cas car vous ne pouvez pas accéder au M2M champs comme faculty à l'aide de la post_save signal, voir cette réponse stackoverflow.com/a/23796604/263989
InformationsquelleAutor Alex

Vous devez vous connecter pour publier un commentaire.