Skip to content

Commit 771591f

Browse files
committed
Remove use of obsolete generic delete view
- Legacy-Id: 6891
1 parent d1e60e5 commit 771591f

1 file changed

Lines changed: 17 additions & 18 deletions

File tree

ietf/nomcom/views.py

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@
22

33
import datetime, re, json
44

5-
from django.views.generic.create_update import delete_object
65
from django.conf import settings
76
from django.contrib.auth.decorators import login_required
8-
from django.contrib.messages.api import success, get_messages
7+
from django.contrib import messages
98
from django.core.urlresolvers import reverse
109
from django.http import HttpResponse, Http404, HttpResponseRedirect, HttpResponseForbidden
11-
from django.shortcuts import render_to_response, get_object_or_404
10+
from django.shortcuts import render_to_response, get_object_or_404, render, redirect
1211
from django.template import RequestContext
1312
from django.template.loader import render_to_string
1413
from django.utils.datastructures import SortedDict
@@ -539,7 +538,7 @@ def view_feedback_pending(request, year):
539538
nomcom = get_nomcom_by_year(year)
540539
extra_ids = None
541540
message = None
542-
for message in get_messages(request):
541+
for message in messages.get_messages(request):
543542
message = ('success', message.message)
544543
FeedbackFormSet = modelformset_factory(Feedback,
545544
form=PendingFeedbackForm,
@@ -561,7 +560,7 @@ def view_feedback_pending(request, year):
561560
formset = FeedbackFormSet(queryset=feedbacks)
562561
for form in formset.forms:
563562
form.set_nomcom(nomcom, request.user)
564-
success(request, 'Feedback saved')
563+
messages.success(request, 'Feedback saved')
565564
return HttpResponseRedirect(reverse('nomcom_view_feedback_pending', None, args=(year, )))
566565
elif request.method == 'POST' and request.POST.get('end'):
567566
extra_ids = request.POST.get('extra_ids', None)
@@ -583,7 +582,7 @@ def view_feedback_pending(request, year):
583582
form.set_nomcom(nomcom, request.user, extra)
584583
extra_ids = None
585584
else:
586-
success(request, 'Feedback saved')
585+
messages.success(request, 'Feedback saved')
587586
return HttpResponseRedirect(reverse('nomcom_view_feedback_pending', None, args=(year, )))
588587
elif request.method == 'POST':
589588
formset = FeedbackFormSet(request.POST)
@@ -617,7 +616,7 @@ def view_feedback_pending(request, year):
617616
if moved:
618617
message = ('success', '%s messages classified. You must enter more information for the following feedback.' % moved)
619618
else:
620-
success(request, 'Feedback saved')
619+
messages.success(request, 'Feedback saved')
621620
return HttpResponseRedirect(reverse('nomcom_view_feedback_pending', None, args=(year, )))
622621
else:
623622
formset = FeedbackFormSet(queryset=feedbacks)
@@ -739,17 +738,17 @@ def edit_nomcom(request, year):
739738
@role_required("Nomcom Chair", "Nomcom Advisor")
740739
def delete_nomcom(request, year):
741740
nomcom = get_nomcom_by_year(year)
742-
post_delete_redirect = reverse('nomcom_deleted')
743-
extra_context = {'year': year,
744-
'selected': 'edit_nomcom',
745-
'nomcom': nomcom}
746-
747-
return delete_object(request,
748-
model=NomCom,
749-
object_id=nomcom.id,
750-
post_delete_redirect=post_delete_redirect,
751-
template_name='nomcom/delete_nomcom.html',
752-
extra_context=extra_context)
741+
742+
if request.method == 'POST':
743+
nomcom.delete()
744+
messages.success(request, "Deleted NomCom data")
745+
return redirect('nomcom_deleted')
746+
747+
return render(request, 'nomcom/delete_nomcom.html', {
748+
'year': year,
749+
'selected': 'edit_nomcom',
750+
'nomcom': nomcom,
751+
})
753752

754753

755754
@role_required("Nomcom Chair", "Nomcom Advisor")

0 commit comments

Comments
 (0)