LP#1901247: skip unnecessary user/ws setting updates when the value is unchanged...

Signed-off-by: Jeff Davis <jeff.davis@bc.libraries.coop>

diff --git a/Open-ILS/web/js/ui/default/staff/services/hatch.js b/Open-ILS/web/js/ui/default/staff/services/hatch.js
index b7e0ff5..5863e6f 100644 (file)
--- a/Open-ILS/web/js/ui/default/staff/services/hatch.js
+++ b/Open-ILS/web/js/ui/default/staff/services/hatch.js
@@ -568,6 +568,12 @@ angular.module('egCoreMod')
                 console.warn('No server setting type exists for ' + key);
                 return service.setBrowserItem(key, value); 
             }
+        } else if (summary && summary.has_staff_setting) {
+            // This is a previously-retrieved server-side user/ws setting.
+            // Don't update the server unless the value has actually changed.
+            if (service.keyCache[key] && service.keyCache[key] === value) {
+                return $q.when();
+            }
         }
 
         var settings = {};