From 8817b7536c97fe624cf8a243791360ee728e8c87 Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Fri, 21 Apr 2017 10:06:17 -0400 Subject: [PATCH] LP#1685232: fix egCore.pcrud.apply() This patch fixes egCore.pcrud.apply() method so that it behaves as intended (i.e., process an array of fieldmapper objects and either creates, updates, or deletes them based on the value of the isnew(), ischanged(), and isdeleted() flags). To test ------- Currently no code uses this method (but some will soon), but it can be tested by writing some code like this: var notes = []; var n1 = new egCore.idl.aun(); n1.usr(userid); // etc. n1.isnew(true); notes.push(n1); var n2 = new egCore.idl.aun(); n2.usr(userid); // etc. n2.isnew(true); notes.push(n1); egCore.pcrud.apply(notes).then(... Signed-off-by: Galen Charlton Signed-off-by: Bill Erickson --- Open-ILS/web/js/ui/default/staff/services/pcrud.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Open-ILS/web/js/ui/default/staff/services/pcrud.js b/Open-ILS/web/js/ui/default/staff/services/pcrud.js index 1f78b17ce2..66fdba928b 100644 --- a/Open-ILS/web/js/ui/default/staff/services/pcrud.js +++ b/Open-ILS/web/js/ui/default/staff/services/pcrud.js @@ -267,12 +267,12 @@ angular.module('egCoreMod') var action = this.cud_action; var fm_obj = this.cud_list[this.cud_idx++]; - if (action == 'auto') { + if (action == 'apply') { if (fm_obj.ischanged()) action = 'update'; if (fm_obj.isnew()) action = 'create'; if (fm_obj.isdeleted()) action = 'delete'; - if (action == 'auto') { + if (action == 'apply') { // object does not need updating; move along this._CUD_next_request(); } -- 2.43.2