Commit e0459477 by Asad Azam Committed by AsadAzam

Fixed instructor organization switching between org and org override

parent 2156bbbb
...@@ -41,15 +41,13 @@ $(document).ready(function () { ...@@ -41,15 +41,13 @@ $(document).ready(function () {
addModalError(gettext("Please upload a instructor image.")); addModalError(gettext("Please upload a instructor image."));
return; return;
} }
var position = getFormInstructorPosition()
personData = { personData = {
'given_name': $('#given-name').val(), 'given_name': $('#given-name').val(),
'family_name': $('#family-name').val(), 'family_name': $('#family-name').val(),
'bio': $('#bio').val(), 'bio': $('#bio').val(),
'email': $('#email').val(), 'email': $('#email').val(),
'profile_image': $('.select-image').attr('src'), 'profile_image': $('.select-image').attr('src'),
'position': position, 'position': getFormInstructorPosition(),
'works': $('#majorWorks').val().split('\n'), 'works': $('#majorWorks').val().split('\n'),
'urls': { 'urls': {
facebook: $('#facebook').val(), facebook: $('#facebook').val(),
...@@ -103,15 +101,18 @@ $(document).ready(function () { ...@@ -103,15 +101,18 @@ $(document).ready(function () {
}); });
function getFormInstructorPosition () { function getFormInstructorPosition () {
if ($('#organization_override').val()) { var org_override_element_value = $('#organization_override').val(),
instructor_position = $('#title').val();
if (org_override_element_value && ($('#org_override_container').is(':visible'))) {
return { return {
title: $('#title').val(), title: instructor_position,
organization_override: $('#organization_override').val(), organization_override: org_override_element_value,
organization: null organization: null
}; };
} }
return { return {
title: $('#title').val(), title: instructor_position,
organization_override: null, organization_override: null,
organization: parseInt($('#id_organization').val()) organization: parseInt($('#id_organization').val())
}; };
...@@ -274,10 +275,12 @@ $(document).on('click', '.selected-instructor a.edit', function (e) { ...@@ -274,10 +275,12 @@ $(document).on('click', '.selected-instructor a.edit', function (e) {
if (data['position']['organization_id'] == null){ if (data['position']['organization_id'] == null){
$('#organization_override').val(data['position']['organization_override']); $('#organization_override').val(data['position']['organization_override']);
$('#org_container').hide(); $('#org_container').hide();
$('#org_override_container').show();
} }
else { else {
$('#id_organization').val(data['position']['organization_id']); $('#id_organization').val(data['position']['organization_id']);
$('#org_override_container').hide() $('#org_override_container').hide();
$('#org_container').show();
} }
$('.select-image').attr('src', data['profile_image_url']); $('.select-image').attr('src', data['profile_image_url']);
$('#given-name').val(data['given_name']); $('#given-name').val(data['given_name']);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment