From: Galen Charlton <gmc@equinoxinitiative.org>
Date: Wed, 7 Feb 2018 17:20:47 +0000 (-0500)
Subject: LP#1676608: fix glitch with egCore.pcrud.apply()
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=dc1ed4eec371efe05c17c951e8b3974a5a40cb0f;p=working%2FEvergreen.git

LP#1676608: fix glitch with egCore.pcrud.apply()

pcrud.apply() is supposed to skip any entries in the list
of CUD actions that don't explicitly mark themselves as
being a creation, update, or deletion, but didn't manage
to fully skip them. As a consequence, the browser console
would note complaints that open-ils.pcrud.apply.$IDLCLASS
methods do not exist.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
---

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 72cb94ec84..95e3aa24b9 100644
--- a/Open-ILS/web/js/ui/default/staff/services/pcrud.js
+++ b/Open-ILS/web/js/ui/default/staff/services/pcrud.js
@@ -278,6 +278,7 @@ angular.module('egCoreMod')
                 if (action == 'apply') {
                     // object does not need updating; move along
                     this._CUD_next_request();
+                    return;
                 }
             }