sg101: forums/views/main.py comparison

comparison forums/views/main.py @ 1031:e1c03da72818

Get rid of some warnings in Django 1.8.

author	Brian Neal <bgneal@gmail.com>
date	Sun, 20 Dec 2015 22:18:59 -0600
parents	4619290d171d
children	a60aabced346

comparison

equal deleted inserted replaced

-:d9610b1e2a3d
+:e1c03da72818
 from django.http import HttpResponseForbidden
 from django.http import HttpResponseRedirect
 from django.core.urlresolvers import reverse
 from django.core.paginator import InvalidPage
 from django.shortcuts import get_object_or_404
-from django.shortcuts import render_to_response
+from django.shortcuts import render
 from django.template.loader import render_to_string
-from django.template import RequestContext
 from django.views.decorators.http import require_POST
 from django.db.models import F
 import antispam
 import antispam.utils
 cat['forums'].append(forum)
 cmpdef = lambda a, b: cmp(a['cat'].position, b['cat'].position)
 cats = sorted(cats.values(), cmpdef)
-return render_to_response('forums/index.html', {
+return render(request, 'forums/index.html', {
 'cats': cats,
 'feeds': feeds,
-},
+})
-context_instance=RequestContext(request))
 def forum_index(request, slug):
 """
 Displays all the topics in a forum.
 # we do this for the template since it is rendered twice
 page_nav = render_to_string('forums/pagination.html', {'page': page})
 can_moderate = perms.can_moderate(forum, request.user)
-return render_to_response('forums/forum_index.html', {
+return render(request, 'forums/forum_index.html', {
 'forum': forum,
 'feed': feed,
 'page': page,
 'page_nav': page_nav,
 'can_moderate': can_moderate,
-},
+})
-context_instance=RequestContext(request))
 def topic_index(request, id):
 """
 Displays all the posts in a topic.
 topic in request.user.favorite_topics.all())
 is_subscribed = request.user.is_authenticated() and (
 topic in request.user.subscriptions.all())
-return render_to_response('forums/topic.html', {
+return render(request, 'forums/topic.html', {
 'forum': topic.forum,
 'topic': topic,
 'page': page,
 'page_nav': page_nav,
 'last_page': last_page,
 'can_moderate': can_moderate,
 'can_reply': can_reply,
 'form': NewPostForm(initial={'topic_id': topic.id}),
 'is_favorite': is_favorite,
 'is_subscribed': is_subscribed,
-},
+})
-context_instance=RequestContext(request))
 def topic_unread(request, id):
 """
 This view redirects to the first post the user hasn't read, if we can
 return HttpResponseRedirect(reverse('forums-new_topic_thanks',
 kwargs={'tid': topic.pk}))
 else:
 form = NewTopicForm(request.user, forum)
-return render_to_response('forums/new_topic.html', {
+return render(request, 'forums/new_topic.html', {
 'forum': forum,
 'form': form,
-},
+})
-context_instance=RequestContext(request))
 @login_required
 def new_topic_thanks(request, tid):
 """
 This view displays the success page for a newly created topic.
 """
 topic = get_object_or_404(Topic.objects.select_related(), pk=tid)
-return render_to_response('forums/new_topic_thanks.html', {
+return render(request, 'forums/new_topic_thanks.html', {
 'forum': topic.forum,
 'topic': topic,
-},
+})
-context_instance=RequestContext(request))
 @require_POST
 def quick_reply_ajax(request):
 """
 post.unread = True
 post.attach_list = post.attachments.all()
 _bump_post_count(request.user)
 _update_last_visit(request.user, form.topic, datetime.datetime.now())
-return render_to_response('forums/display_post.html', {
+return render(request, 'forums/display_post.html', {
 'post': post,
 'can_moderate': perms.can_moderate(form.topic.forum, request.user),
 'can_reply': True,
-},
+})
-context_instance=RequestContext(request))
 # The client side javascript is pretty simplistic right now and we don't
 # want to change it yet. It is expecting a single error string. Just grab
 # the first error message and use that.
 errors = form.errors.as_data()
 return HttpResponseRedirect(post.get_absolute_url())
 else:
 form = PostForm(instance=post, topic_name=topic_name)
-return render_to_response('forums/edit_post.html', {
+return render(request, 'forums/edit_post.html', {
 'forum': post.topic.forum,
 'topic': post.topic,
 'post': post,
 'form': form,
 'can_moderate': can_moderate,
-},
+})
-context_instance=RequestContext(request))
 @require_POST
 def delete_post(request):
 """
 else:
 form = PostForm()
 else:
 form = None
-return render_to_response('forums/new_post.html', {
+return render(request, 'forums/new_post.html', {
 'forum': topic.forum,
 'topic': topic,
 'form': form,
 'can_post': can_post,
-},
+})
-context_instance=RequestContext(request))
 @login_required
 def mod_topic_stick(request, id):
 """
 _move_topic(topic, old_forum, new_forum)
 return HttpResponseRedirect(topic.get_absolute_url())
 else:
 form = MoveTopicForm(request.user)
-return render_to_response('forums/move_topic.html', {
+return render(request, 'forums/move_topic.html', {
 'forum': topic.forum,
 'topic': topic,
 'form': form,
-},
+})
-context_instance=RequestContext(request))
 @login_required
 def mod_forum(request, slug):
 """
 return HttpResponseRedirect(url)
 if form is None:
 form = MoveTopicForm(request.user, hide_label=True)
-return render_to_response('forums/mod_forum.html', {
+return render(request, 'forums/mod_forum.html', {
 'forum': forum,
 'page': page,
 'page_nav': page_nav,
 'form': form,
-},
+})
-context_instance=RequestContext(request))
 @login_required
 @require_POST
 def catchup_all(request):
 """
 This view marks all forums as being read.
 """
 forum_ids = Forum.objects.forum_ids_for_user(request.user)
-tlvs = TopicLastVisit.objects.filter(user=request.user,
+TopicLastVisit.objects.filter(
+user=request.user,
 topic__forum__id__in=forum_ids).delete()
 now = datetime.datetime.now()
 ForumLastVisit.objects.filter(user=request.user,
 forum__in=forum_ids).update(begin_date=now, end_date=now)
 else:
 form = SplitTopicForm(request.user)
 posts = topic.posts.select_related()
-return render_to_response('forums/mod_split_topic.html', {
+return render(request, 'forums/mod_split_topic.html', {
 'forum': topic.forum,
 'topic': topic,
 'posts': posts,
 'form': form,
-},
+})
-context_instance=RequestContext(request))
 @login_required
 def unread_topics(request):
 """Displays the topics with unread posts for a given user."""
 attach_topic_page_ranges(page.object_list)
 # we do this for the template since it is rendered twice
 page_nav = render_to_string('forums/pagination.html', {'page': page})
-return render_to_response('forums/topic_list.html', {
+return render(request, 'forums/topic_list.html', {
 'title': 'Topics With Unread Posts',
 'page': page,
 'page_nav': page_nav,
 'unread': True,
-},
+})
-context_instance=RequestContext(request))
 def unanswered_topics(request):
 """Displays the topics with no replies."""
 attach_topic_page_ranges(page.object_list)
 # we do this for the template since it is rendered twice
 page_nav = render_to_string('forums/pagination.html', {'page': page})
-return render_to_response('forums/topic_list.html', {
+return render(request, 'forums/topic_list.html', {
 'title': 'Unanswered Topics',
 'page': page,
 'page_nav': page_nav,
 'unread': False,
-},
+})
-context_instance=RequestContext(request))
 def active_topics(request, num):
 """Displays the last num topics that have been posted to."""
 # we do this for the template since it is rendered twice
 page_nav = render_to_string('forums/pagination.html', {'page': page})
 title = 'Last %d Active Topics' % num
-return render_to_response('forums/topic_list.html', {
+return render(request, 'forums/topic_list.html', {
 'title': title,
 'page': page,
 'page_nav': page_nav,
 'unread': False,
-},
+})
-context_instance=RequestContext(request))
 @login_required
 def my_posts(request):
 """Displays a list of posts the requesting user made."""
 return HttpResponseForbidden("You don't have permission for this post.")
 ip_users = sorted(set(Post.objects.filter(
 user_ip=post.user_ip).values_list('user__username', flat=True)))
-return render_to_response('forums/post_ip.html', {
+return render(request, 'forums/post_ip.html', {
 'post': post,
 'ip_users': ip_users,
-},
+})
-context_instance=RequestContext(request))
 def _user_posts(request, target_user, req_user, page_title):
 """Displays a list of posts made by the target user.
 req_user is the user trying to view the posts. Only the forums
 raise Http404
 # we do this for the template since it is rendered twice
 page_nav = render_to_string('forums/pagination.html', {'page': page})
-return render_to_response('forums/post_list.html', {
+return render(request, 'forums/post_list.html', {
 'title': page_title,
 'page': page,
 'page_nav': page_nav,
-},
+})
-context_instance=RequestContext(request))
 def _bump_post_count(user):
 """
 Increments the forum_post_count for the given user.

Mercurial > public > sg101

comparison forums/views/main.py @ 1031:e1c03da72818