LP#1676608: fix glitch with egCore.pcrud.apply()
authorGalen Charlton <gmc@equinoxinitiative.org>
Wed, 7 Feb 2018 17:20:47 +0000 (12:20 -0500)
committerMike Rylander <mrylander@gmail.com>
Wed, 28 Feb 2018 15:26:42 +0000 (10:26 -0500)
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>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Open-ILS/web/js/ui/default/staff/services/pcrud.js

index 72cb94e..95e3aa2 100644 (file)
@@ -278,6 +278,7 @@ angular.module('egCoreMod')
                 if (action == 'apply') {
                     // object does not need updating; move along
                     this._CUD_next_request();
+                    return;
                 }
             }