Use transactions only in views that need them

boards/views.py

@@ -7,6 +7,7 b' from django.http.response import HttpRes'
 from django.template import RequestContext
 from django.shortcuts import render, redirect, get_object_or_404
 from django.utils import timezone
+from django.db import transaction
 
 from boards import forms
 import boards
@@ -61,6 +62,7 b' def index(request, page=0):'
                   context)
 
 
[e-mail]_on_success
 def _new_post(request, form, thread_id=boards.models.NO_PARENT):
     """Add a new post (in thread or as a reply)."""
 
@@ -226,6 +228,7 b' def settings(request):'
     is_moderator = user.is_moderator()
 
     if request.method == 'POST':
+	with transaction.commit_on_success():
         if is_moderator:
             form = ModeratorSettingsForm(request.POST,
                                          error_class=PlainErrorList)
@@ -288,6 +291,7 b' def authors(request):'
     return render(request, 'boards/authors.html', context)
 
 
[e-mail]_on_success
 def delete(request, post_id):
     """Delete post"""
 
@@ -304,6 +308,7 b' def delete(request, post_id):'
         return redirect(thread, post_id=post.thread.id)
 
 
[e-mail]_on_success
 def ban(request, post_id):
     """Ban user"""
 
@@ -331,6 +336,7 b' def page_404(request):'
     return render(request, 'boards/404.html', context)
 
 
[e-mail]_on_success
 def tag_subscribe(request, tag_name):
     """Add tag to favorites"""
 
@@ -343,6 +349,7 b' def tag_subscribe(request, tag_name):'
     return _redirect_to_next(request)
 
 
[e-mail]_on_success
 def tag_unsubscribe(request, tag_name):
     """Remove tag from favorites"""
 

neboard/settings.py

@@ -111,7 +111,6 b' MIDDLEWARE_CLASSES = ('
     'django.contrib.sessions.middleware.SessionMiddleware',
     'django.middleware.locale.LocaleMiddleware',
     'django.middleware.common.CommonMiddleware',
-    'django.middleware.transaction.TransactionMiddleware',
     'django.contrib.auth.middleware.AuthenticationMiddleware',
     'django.contrib.messages.middleware.MessageMiddleware',
     'boards.middlewares.BanMiddleware',