From 277117ba34204995efac766b715fe62270a5e908 Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Wed, 26 May 2010 09:30:43 -0500 Subject: Move package flagging URLs below the package We were still using the user-facing package IDs here which we have eliminated everywhere else Change it to use the same user-friendly URL pattern we are using elsewhere. Signed-off-by: Dan McGee --- packages/views.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'packages') diff --git a/packages/views.py b/packages/views.py index fc36698..682dded 100644 --- a/packages/views.py +++ b/packages/views.py @@ -205,8 +205,9 @@ def files(request, name='', repo='', arch=''): return render_to_response(template, RequestContext(request, {'pkg':pkg,'files':files})) @permission_required('main.change_package') -def unflag(request, pkgid): - pkg = get_object_or_404(Package, id=pkgid) +def unflag(request, name='', repo='', arch=''): + pkg = get_object_or_404(Package, + pkgname=name, repo__name__iexact=repo, arch__name=arch) pkg.needupdate = 0 pkg.save() return HttpResponseRedirect(pkg.get_absolute_url()) @@ -263,8 +264,9 @@ class FlagForm(forms.Form): widget=forms.TextInput(attrs={'style': 'display:none;'}), required=False) -def flag(request, pkgid): - pkg = get_object_or_404(Package, id=pkgid) +def flag(request, name='', repo='', arch=''): + pkg = get_object_or_404(Package, + pkgname=name, repo__name__iexact=repo, arch__name=arch) context = {'pkg': pkg} if pkg.needupdate == 1: # already flagged. do nothing. -- cgit v1.2.3-24-g4f1b