Skip to content

Commit 46b21a4

Browse files
committed
add login_required decorator to all views that use check_permissions decorator. Commit ready for merge
- Legacy-Id: 8640
1 parent 5016bae commit 46b21a4

2 files changed

Lines changed: 12 additions & 0 deletions

File tree

ietf/secr/proceedings/views.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
from django.conf import settings
1010
from django.contrib import messages
11+
from django.contrib.auth.decorators import login_required
1112
from django.core.exceptions import ObjectDoesNotExist
1213
from django.core.urlresolvers import reverse
1314
from django.db.models import Max
@@ -330,6 +331,7 @@ def build(request,meeting_num,acronym):
330331
url = reverse('proceedings_upload_unified', kwargs={'meeting_num':meeting_num,'acronym':acronym})
331332
return HttpResponseRedirect(url)
332333

334+
@login_required
333335
@check_permissions
334336
def delete_material(request,slide_id):
335337
'''
@@ -394,6 +396,7 @@ def delete_interim_meeting(request, meeting_num):
394396
url = reverse('proceedings_interim', kwargs={'acronym':group.acronym})
395397
return HttpResponseRedirect(url)
396398

399+
@login_required
397400
@check_permissions
398401
def edit_slide(request, slide_id):
399402
'''
@@ -528,6 +531,7 @@ def main(request):
528531
RequestContext(request,{}),
529532
)
530533

534+
@login_required
531535
@check_permissions
532536
def move_slide(request, slide_id, direction):
533537
'''
@@ -670,6 +674,7 @@ def recording_edit(request, meeting_num, name):
670674
RequestContext(request, {}),
671675
)
672676

677+
@login_required
673678
@check_permissions
674679
def replace_slide(request, slide_id):
675680
'''
@@ -848,6 +853,7 @@ def select_interim(request):
848853
RequestContext(request,{}),
849854
)
850855

856+
@login_required
851857
@check_permissions
852858
def upload_unified(request, meeting_num, acronym=None, session_id=None):
853859
'''

ietf/secr/sreq/views.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import datetime
22

33
from django.contrib import messages
4+
from django.contrib.auth.decorators import login_required
45
from django.db.models import Q
56
from django.http import Http404
67
from django.shortcuts import render_to_response, get_object_or_404, redirect
@@ -163,6 +164,7 @@ def session_conflicts_as_string(group, meeting):
163164
# -------------------------------------------------
164165
# View Functions
165166
# -------------------------------------------------
167+
@login_required
166168
@check_permissions
167169
def approve(request, acronym):
168170
'''
@@ -183,6 +185,7 @@ def approve(request, acronym):
183185
messages.error(request, 'Not authorized to approve the third session')
184186
return redirect('sessions_view', acronym=acronym)
185187

188+
@login_required
186189
@check_permissions
187190
def cancel(request, acronym):
188191
'''
@@ -328,6 +331,7 @@ def session_save(session):
328331
# send an email to iesg-secretariat to alert to change
329332
pass
330333

334+
@login_required
331335
@check_permissions
332336
def edit_mtg(request, num, acronym):
333337
'''
@@ -530,6 +534,7 @@ def main(request):
530534
RequestContext(request, {}),
531535
)
532536

537+
@login_required
533538
@check_permissions
534539
def new(request, acronym):
535540
'''
@@ -591,6 +596,7 @@ def new(request, acronym):
591596
RequestContext(request, {}),
592597
)
593598

599+
@login_required
594600
@check_permissions
595601
def no_session(request, acronym):
596602
'''

0 commit comments

Comments
 (0)