Skip to content

Commit fca04d9

Browse files
committed
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.
- Legacy-Id: 9595
1 parent 9d6fc63 commit fca04d9

2 files changed

Lines changed: 6 additions & 10 deletions

File tree

ietf/liaisons/views.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@
2222

2323
@can_submit_liaison_required
2424
def add_liaison(request, liaison=None):
25+
if 'incoming' in request.GET.keys() and not can_add_incoming_liaison(request.user):
26+
return HttpResponseForbidden("Restricted to users who are authorized to submit incoming liaison statements")
27+
2528
if request.method == 'POST':
2629
form = liaison_form_factory(request, data=request.POST.copy(),
2730
files = request.FILES, liaison=liaison)

static/js/liaisons.js

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -238,21 +238,14 @@ $(document).ready(function () {
238238

239239
var selected_id = purpose.val();
240240

241-
if (selected_id == '1' || selected_id == '2' || selected_id == '5') {
241+
if (selected_id == '1' || selected_id == '2') {
242+
deadline.prop('required',true);
242243
deadlinecontainer.show();
243244
} else {
245+
deadline.prop('required',false);
244246
deadlinecontainer.hide();
245247
deadline.val('');
246248
}
247-
248-
if (selected_id == '5') {
249-
othercontainer.show();
250-
deadlinecontainer.find("label").removeClass("required");
251-
} else {
252-
othercontainer.hide();
253-
other_purpose.val('');
254-
deadlinecontainer.find("label").addClass("required");
255-
}
256249
};
257250

258251
var checkOtherSDO = function() {

0 commit comments

Comments
 (0)