Delete thread when the OP is deleted. Removed old post deleter

boards/models/post.py

@@ -95,14 +95,14 b' class PostManager(models.Manager):'
 
         logger = logging.getLogger('boards.post.create')
 
-        logger.info('Created post #%d with title "%s"' % (post.id,
-                                                        post.title))
+        logger.info('Created post #{} with title "{}" by {}'.format(
+            post.id, post.title, post.poster_ip))
 
         if image:
             post_image = PostImage.objects.create(image=image)
             post.images.add(post_image)
-            logger.info('Created image #%d for post #%d' % (post_image.id,
-                                                            post.id))
+            logger.info('Created image #{} for post #{}'.format(
+                post_image.id, post.id))
 
         thread.replies.add(post)
         list(map(thread.add_tag, tags))
@@ -111,28 +111,8 b' class PostManager(models.Manager):'
             Thread.objects.process_oldest_threads()
         self.connect_replies(post)
 
-
         return post
 
-    def delete_post(self, post):
-        """
-        Deletes post and update or delete its thread
-        """
-
-        post_id = post.id
-
-        thread = post.get_thread()
-
-        if post.is_opening():
-            thread.delete()
-        else:
-            thread.last_edit_time = timezone.now()
-            thread.save()
-
-            post.delete()
-
-        logging.getLogger('boards.post.delete').info('Deleted post #%d (%s)' % (post_id, post.get_title()))
-
     def delete_posts_by_ip(self, ip):
         """
         Deletes all posts of the author with same IP
@@ -140,7 +120,7 b' class PostManager(models.Manager):'
 
         posts = self.filter(poster_ip=ip)
         for post in posts:
-            self.delete_post(post)
+            post.delete()
 
     def  connect_replies(self, post):
         """
@@ -376,12 +356,23 b' class Post(models.Model, Viewable):'
 
     def delete(self, using=None):
         """
-        Deletes all post images and the post itself.
+        Deletes all post images and the post itself. If the post is opening,
+        thread with all posts is deleted.
         """
 
         self.images.all().delete()
 
-        super(Post, self).delete(using)
+        if self.is_opening():
+            self.get_thread().delete()
+        else:
+            thread = self.get_thread()
+            thread.last_edit_time = timezone.now()
+            thread.save()
+
+            super(Post, self).delete(using)
+
+        logging.getLogger('boards.post.delete').info(
+            'Deleted post P#{}/{}'.format(self.id, self.get_title()))
 
     def get_post_data(self, format_type=DIFF_TYPE_JSON, request=None,
                       include_last_update=False):

boards/tests/test_post.py

@@ -26,7 +26,7 b' class PostTests(TestCase):'
         post = self._create_post()
         post_id = post.id
 
-        Post.objects.delete_post(post)
+        post.delete()
 
         self.assertFalse(Post.objects.filter(id=post_id).exists())
 

boards/urls.py

@@ -5,7 +5,6 b' from boards.rss import AllThreadsFeed, T'
 from boards.views import api, tag_threads, all_threads, \
     settings, all_tags
 from boards.views.authors import AuthorsView
-from boards.views.delete_post import DeletePostView
 from boards.views.ban import BanUserView
 from boards.views.search import BoardSearchView
 from boards.views.static import StaticPageView
@@ -38,8 +37,6 b" urlpatterns = patterns('',"
     url(r'^settings/$', settings.SettingsView.as_view(), name='settings'),
     url(r'^tags/$', all_tags.AllTagsView.as_view(), name='tags'),
     url(r'^authors/$', AuthorsView.as_view(), name='authors'),
-    url(r'^delete/(?P<post_id>\w+)/$', DeletePostView.as_view(),
-            name='delete'),
     url(r'^ban/(?P<post_id>\w+)/$', BanUserView.as_view(), name='ban'),
 
     url(r'^banned/$', views.banned.BannedView.as_view(), name='banned'),

boards/views/api.py

@@ -113,8 +113,6 b' def get_post(request, post_id):'
     in threads list with 'truncated' get parameter.
     """
 
-    logger.info('Getting post #%s' % post_id)
-
     post = get_object_or_404(Post, id=post_id)
 
     context = RequestContext(request)