Aufschalten speichern Methodenliste in der viele zu viele Feld hinzufügen

Question

Ich habe dieses Modell Post von den Benutzern sparen:

class Tag(models.Model): 
    name = models.CharField(max_length=255, unique=True) 

def add_tags(obj_id, body): 
    object = Post.objects.get(id=obj_id) 
    tag_list = [Tag.objects.create(name=word) for word in body.split()] 
    for tag in tag_list: 
     object.tags.add(tag) 

class Post(models.Model): 
    user = models.ForeignKey(User) 
    body = models.TextField() 
    tags = models.ManyToManyField(Tag, blank=True) 
    pub_date = models.DateTimeField(default=timezone.now) 
    activity = GenericRelation(Activity, related_query_name="posts") 

    def save(self, *args, **kwargs): 
     super(Post, self).save(*args, **kwargs) 
     if self.body: 
      body = self.body 
      obj_id = self.id 
      add_tags(obj_id, body) 

Also, wenn ein Benutzer Post etwas, würde ich prüfen möchte, ob es einen Hash ist Etikett innerhalb des Körpers verwendet. Wenn Tags vorhanden sind, rufen Sie die Tags in der Liste ab.

Aber wenn ich bin Entsendung werden die Tag-Objekte erstellt, aber sie sind nicht das Hinzufügen für die Post.tags Felder aus.

post.body Beispiel = Überprüfen Sie #von http://somesitedotcom/page#idtop #hell yeah!

Was mache ich hier falsch?

Answer 1

Sie können nicht zuordnen Post mit einem tag bis es gespeichert:

def save(self, *args, **kwargs): 
    super(Post, self).save(*args, **kwargs) 
    if self.body: 
     tag_list = [word for word in self.body.split() if word.startswith("#")] 
     self.body = tag_list 
     self.tags.add(*tag_list) 

Answer 2

Dies ist ein häufiger von Menschen gemacht Fehler. Um die m2m-Beziehung eines Modellobjekts zu aktualisieren, muss das Objekt zuerst in der Datenbank gespeichert werden. Django m2m verwendet im Wesentlichen eine Zwischentabelle, um die Beziehung zu speichern, sodass ohne die Speicherung der beiden Enden die Beziehung nicht gebildet werden kann.

Ich würde nicht auf save() Methode überschreiben, um dies zu erreichen, sondern eher mit diesem Prozess separat in Views-Methode, weil save() kann in jeder Situation passieren, aber Sie könnten nicht immer was dieser Prozess passieren. Aus Gründen der Richtigkeit sollten Sie es tun, nachdem Sie super(Post, self).save(*args, **kwargs) in der save Methode aufrufen.

Edit:

Klingt wie Ihr Code ist auch fehlerhaft. tag_list ist eine Liste von Zeichenfolgen, aber Ihr Code self.tags.add(*tag_list) behandelt sie als Tag Objekte. Sie müssen Tag Objekte erstellen und dann add verwenden, um die Beziehung zu verknüpfen.