From f260843deb371a7d2ef5a265f52e643fcf64f86f Mon Sep 17 00:00:00 2001 From: Dusty Phillips Date: Fri, 10 Oct 2008 18:51:21 -0400 Subject: use RequestContext because its standard --- TODO | 1 - devel/views.py | 13 +++++++------ main/utils.py | 4 ---- packages/views.py | 7 +++---- todolists/views.py | 18 +++++++++--------- 5 files changed, 19 insertions(+), 24 deletions(-) diff --git a/TODO b/TODO index 93dd1b85..b9c49464 100644 --- a/TODO +++ b/TODO @@ -1,6 +1,5 @@ TODO: - refactor stats by templates in dashboard, maybe a templatetag - - drop all references to render_response - move some of the custom widgets and fields to their own module - Look at every file and see if its still needed - eg: calendar css and js diff --git a/devel/views.py b/devel/views.py index 2b78e533..64d1531f 100644 --- a/devel/views.py +++ b/devel/views.py @@ -1,7 +1,6 @@ from django import forms from django.http import HttpResponseRedirect from django.contrib.auth.models import User -from archweb_dev.main.utils import render_response from archweb_dev.main.models import Package, Todolist from archweb_dev.main.models import Arch, Repo from archweb_dev.main.models import UserProfile, News @@ -13,14 +12,15 @@ def index(request): maintainer=request.user).filter( needupdate=True).order_by('repo', 'pkgname') - return render_response( - request, 'devel/index.html', - {'pkgs': pkgs, 'todos': todos, 'maint': request.user, + return render_to_response( + 'devel/index.html', + RequestContext(request, {'pkgs': pkgs, 'todos': todos, + 'maint': request.user, 'repos': Repo.objects.all(), 'arches': Arch.objects.all(), 'maintainers': [User(id=0, username="orphan", first_name="Orphans")] + list( User.objects.all()) - }) + })) def change_notify(request): maint = User.objects.get(username=request.user.username) @@ -58,7 +58,8 @@ def change_profile(request): return HttpResponseRedirect('/devel/') else: form = ProfileForm(initial={'email': request.user.email}) - return render_response(request, 'devel/profile.html', {'form': form}) + return render_to_response('devel/profile.html', + RequestContext(request, {'form': form})) def siteindex(request): news = News.objects.order_by('-postdate', '-id')[:10] diff --git a/main/utils.py b/main/utils.py index f31689f8..74ec50ef 100644 --- a/main/utils.py +++ b/main/utils.py @@ -11,10 +11,6 @@ def prune_cache(django_page_url): cache_postfix = '.d41d8cd98f00b204e9800998ecf8427e' cache.delete('%s%s%s' % (cache_prefix,django_page_url,cache_postfix)) -def render_response(req, *args, **kwargs): - kwargs['context_instance'] = RequestContext(req) - return render_to_response(*args, **kwargs) - #utility to make a pair of django choices make_choice = lambda l: [(str(m), str(m)) for m in l] diff --git a/packages/views.py b/packages/views.py index 69f97dbb..75bdc56f 100644 --- a/packages/views.py +++ b/packages/views.py @@ -9,7 +9,6 @@ from django.views.generic import list_detail from django.db.models import Q import datetime -from archweb_dev.main.utils import render_response from archweb_dev.main.models import Package, PackageFile from archweb_dev.main.models import Arch, Repo, Signoff from archweb_dev.main.utils import make_choice @@ -144,7 +143,7 @@ def search(request, page=None): def files(request, pkgid): pkg = get_object_or_404(Package, id=pkgid) files = PackageFile.objects.filter(pkg=pkgid) - return render_response(request, 'packages/files.html', {'pkg':pkg,'files':files}) + return render_to_response('packages/files.html', RequestContext(request, {'pkg':pkg,'files':files})) def unflag(request, pkgid): pkg = get_object_or_404(Package, id=pkgid) @@ -154,8 +153,8 @@ def unflag(request, pkgid): def signoffs(request): packages = Package.objects.filter(repo__name="Testing").order_by("pkgname") - return render_response(request, 'packages/signoffs.html', - {'packages': packages}) + return render_to_response('packages/signoffs.html', + RequestContext(request, {'packages': packages})) def signoff_package(request, arch, pkgname): pkg = get_object_or_404(Package, diff --git a/todolists/views.py b/todolists/views.py index cd02c865..70e55beb 100644 --- a/todolists/views.py +++ b/todolists/views.py @@ -1,12 +1,12 @@ from django import forms from django.http import HttpResponseRedirect +from django.template import RequestContext from django.core.mail import send_mail -from django.shortcuts import get_object_or_404 +from django.shortcuts import get_object_or_404, render_to_response from django.contrib.auth.decorators import permission_required from django.views.generic.create_update import delete_object from django.template import Context, loader -from archweb_dev.main.utils import render_response from archweb_dev.main.models import Todolist, TodolistPkg, Package class TodoListForm(forms.Form): @@ -36,17 +36,16 @@ def flag(request, listid, pkgid): def view(request, listid): list = get_object_or_404(Todolist, id=listid) - return render_response( - request, - 'todolists/view.html', - {'list':list}) + return render_to_response('todolists/view.html', + RequestContext(request, {'list':list})) def list(request): lists = Todolist.objects.order_by('-date_added') for l in lists: l.complete = TodolistPkg.objects.filter( list=l.id,complete=False).count() == 0 - return render_response(request, 'todolists/list.html', {'lists':lists}) + return render_to_response('todolists/list.html', + RequestContext(request, {'lists':lists})) @permission_required('main.add_todolist') def add(request): @@ -71,7 +70,8 @@ def add(request): 'form': form, 'submit_text': 'Create List' } - return render_response(request, 'general_form.html', page_dict) + return render_to_response('general_form.html', + RequestContext(request, page_dict)) @permission_required('main.change_todolist') def edit(request, list_id): @@ -109,7 +109,7 @@ def edit(request, list_id): 'form': form, 'submit_text': 'Save List' } - return render_response(request, 'general_form.html', page_dict) + return render_to_response('general_form.html', RequestContext(request, page_dict)) @permission_required('main.delete_todolist') def delete_todolist(request, object_id): -- cgit v1.2.3-54-g00ecf