From c532b0a376be74859824b211685b99e99102a7b5 Mon Sep 17 00:00:00 2001 From: Jason Etheridge Date: Wed, 2 Nov 2016 14:28:08 -0400 Subject: [PATCH] lp1261835 stop colliding bill UI refreshes for want of promises in Angular :) population of the bill list in this implementation is asynchronous, but the rest of the refresh action is not, and it's sadly being invoked multiple times. There are different ways to fix this, but using a semaphore seems to work. Signed-off-by: Jason Etheridge Signed-off-by: Dawn Dale Signed-off-by: Kathy Lussier --- Open-ILS/xul/staff_client/server/patron/bill2.js | 23 +++++++++++++++-------- 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/Open-ILS/xul/staff_client/server/patron/bill2.js b/Open-ILS/xul/staff_client/server/patron/bill2.js index 8f0c19beaa..bb9d8c0a10 100644 --- a/Open-ILS/xul/staff_client/server/patron/bill2.js +++ b/Open-ILS/xul/staff_client/server/patron/bill2.js @@ -20,6 +20,8 @@ function my_init() { g.funcs = []; g.bill_map = {}; g.row_map = {}; g.check_map = {}; + g.safe_for_refresh = false; + g.patron_id = xul_param('patron_id'); $('circulating_hint').hidden = true; @@ -63,6 +65,8 @@ function my_init() { ); } + g.funcs.push( function() { g.safe_for_refresh = true; } ); + } catch(E) { var err_msg = $("commonStrings").getFormattedString('common.exception', ['patron/bill2.xul', E]); try { g.error.sdump('D_ERROR',err_msg); } catch(E) { dump(err_msg); } @@ -519,11 +523,12 @@ function retrieve_mbts_for_list() { } else { g.mbts_ids.reverse(); - + g.safe_for_refresh = false; for (var i = 0; i < g.mbts_ids.length; i++) { dump('i = ' + i + ' g.mbts_ids[i] = ' + g.mbts_ids[i] + '\n'); g.funcs.push( gen_list_append_func(g.mbts_ids[i]) ); } + g.funcs.push( function() { g.safe_for_refresh = true; } ); } } @@ -1070,14 +1075,16 @@ function pay(payment_blob) { function refresh(params) { try { - if (params && params.clear_voided_summary) { - g.data.voided_billings = []; g.data.stash('voided_billings'); + if (g.safe_for_refresh) { + if (params && params.clear_voided_summary) { + g.data.voided_billings = []; g.data.stash('voided_billings'); + } + refresh_patron(); + g.bill_list.clear(); + retrieve_mbts_for_list(); + tally_voided(); + distribute_payment(); } - refresh_patron(); - g.bill_list.clear(); - retrieve_mbts_for_list(); - tally_voided(); - distribute_payment(); } catch(E) { alert('Error in bill2.js, refresh(): ' + E); } -- 2.11.0