altf4arnold/datatracker
1
0
Fork 0
Code Issues Pull requests Actions 3 Packages Projects Releases Wiki Activity

Fixes issue where authorized individual unable to add incoming liaison. Fixes problem with incoming liaison form requiring deadline date for information only statements. Commit ready for merge.

Browse source 
- Legacy-Id: 9595
This commit is contained in:
Ryan Cross 2015-05-01 18:11:00 +00:00
parent 9d6fc63581
commit fca04d97e5
2 changed files with 6 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
ietf/liaisons/views.py
View file

@ -22,6 +22,9 @@ from ietf.liaisons.fields import select2_id_liaison_json
@can_submit_liaison_required @can_submit_liaison_required
def add_liaison(request, liaison=None): def add_liaison(request, liaison=None):
if 'incoming' in request.GET.keys() and not can_add_incoming_liaison(request.user):
return HttpResponseForbidden("Restricted to users who are authorized to submit incoming liaison statements")
if request.method == 'POST': if request.method == 'POST':
form = liaison_form_factory(request, data=request.POST.copy(), form = liaison_form_factory(request, data=request.POST.copy(),
files = request.FILES, liaison=liaison) files = request.FILES, liaison=liaison)

13
static/js/liaisons.js
View file

@ -238,21 +238,14 @@ $(document).ready(function () {
var selected_id = purpose.val(); var selected_id = purpose.val();
if (selected_id == '1' || selected_id == '2' || selected_id == '5') { if (selected_id == '1' || selected_id == '2') {
deadline.prop('required',true);
deadlinecontainer.show(); deadlinecontainer.show();
} else { } else {
deadline.prop('required',false);
deadlinecontainer.hide(); deadlinecontainer.hide();
deadline.val(''); deadline.val('');
} }
if (selected_id == '5') {
othercontainer.show();
deadlinecontainer.find("label").removeClass("required");
} else {
othercontainer.hide();
other_purpose.val('');
deadlinecontainer.find("label").addClass("required");
}
}; };
var checkOtherSDO = function() { var checkOtherSDO = function() {