Mercurial > public > sg101
view gpp/forums/attachments.py @ 318:c550933ff5b6
Fix a bug where you'd get an error when trying to delete a forum thread (topic does not exist). Apparently when you call topic.delete() the posts would get deleted, but the signal handler for each one would run, and it would try to update the topic's post count or something, but the topic was gone? Reworked the code a bit and explicitly delete the posts first. I also added a sync() call on the parent forum since post counts were not getting adjusted.
author | Brian Neal <bgneal@gmail.com> |
---|---|
date | Sat, 05 Feb 2011 21:46:52 +0000 |
parents | 72fd300685d5 |
children |
line wrap: on
line source
""" This module contains a class for handling attachments on forum posts. """ from oembed.models import Oembed from forums.models import Attachment class AttachmentProcessor(object): """ This class is aggregated by various form classes to handle attachments on forum posts. New posts can receive attachments and edited posts can have their attachments replaced, augmented, or deleted. """ def __init__(self, ids): """ This class is constructed with a list of Oembed ids. We retrieve the actual Oembed objects associated with these keys for use in subsequent operations. """ # ensure all ids are integers self.pks = [] for pk in ids: try: pk = int(pk) except ValueError: continue self.pks.append(pk) self.embeds = [] if self.pks: self.embeds = Oembed.objects.in_bulk(self.pks) def save_attachments(self, post): """ Create and save attachments to the supplied post object. Any existing attachments on the post are removed first. """ post.attachments.clear() for n, pk in enumerate(self.pks): attachment = Attachment(post=post, embed=self.embeds[pk], order=n) attachment.save() def has_attachments(self): """ Return true if we have valid pending attachments. """ return len(self.embeds) > 0 def get_ids(self): """ Return the list of Oembed ids. """ return self.pks