More clean up of the JS code.
This commit is contained in:
@@ -112,8 +112,7 @@ async function render_event_presenter_records(events) {
|
||||
console.log('Trying to remove an old presenter LI node if it exists...');
|
||||
try {
|
||||
document.getElementById('event_presenter_'+tbl_presenter_id).remove();
|
||||
}
|
||||
catch(err) {
|
||||
} catch(err) {
|
||||
//console.log('A node with the ID of event_presenter_'+tbl_presenter_id+' was not found.');
|
||||
console.log('This presenter list item node was not found. In most cases this is expected.');
|
||||
console.log(err.message);
|
||||
@@ -152,23 +151,28 @@ async function render_event_presenter_records(events) {
|
||||
remove_presenter_id = false;
|
||||
|
||||
console.log('XXXXX ******** STARTING SORT ******** XXXXX');
|
||||
var categoryItems = document.getElementById('event_presentation_presenters_'+tbl_presentation_id).childNodes;
|
||||
console.log(categoryItems);
|
||||
|
||||
var categoryItemsArray = Array.from(categoryItems);
|
||||
|
||||
function sorter(a, b) {
|
||||
if (a.dataset.family_name < b.dataset.family_name) return -1;
|
||||
if (a.dataset.family_name > b.dataset.family_name) return 1;
|
||||
try {
|
||||
var categoryItems = document.getElementById('event_presentation_presenters_'+tbl_presentation_id).childNodes;
|
||||
console.log(categoryItems);
|
||||
|
||||
var categoryItemsArray = Array.from(categoryItems);
|
||||
|
||||
function sorter(a, b) {
|
||||
if (a.dataset.family_name < b.dataset.family_name) return -1;
|
||||
if (a.dataset.family_name > b.dataset.family_name) return 1;
|
||||
}
|
||||
|
||||
let sorted = categoryItemsArray.sort(sorter);
|
||||
|
||||
function update_li_order(item, index) {
|
||||
document.getElementById('event_presentation_presenters_'+tbl_presentation_id).appendChild(item);
|
||||
}
|
||||
|
||||
sorted.forEach(update_li_order);
|
||||
} catch(err) {
|
||||
console.log(err);
|
||||
}
|
||||
|
||||
let sorted = categoryItemsArray.sort(sorter);
|
||||
|
||||
function update_li_order(item, index) {
|
||||
document.getElementById('event_presentation_presenters_'+tbl_presentation_id).appendChild(item);
|
||||
}
|
||||
|
||||
sorted.forEach(update_li_order);
|
||||
console.log('******** FINISHED SORT ********');
|
||||
|
||||
} else {
|
||||
@@ -185,8 +189,7 @@ async function render_event_presenter_records(events) {
|
||||
console.log('Trying to remove a presenter LI node...');
|
||||
try {
|
||||
document.getElementById('event_presenter_'+tbl_presenter_id).remove();
|
||||
}
|
||||
catch(err) {
|
||||
} catch(err) {
|
||||
//console.log('A node with the ID of event_presenter_'+tbl_presenter_id+' was not found.');
|
||||
console.log('This presenter list item node was not found. In some cases this is expected.');
|
||||
console.log(err.message);
|
||||
|
||||
Reference in New Issue
Block a user