avoid reloading users on every user change (#5010)

SDCP-1037

Author: petrjasek

scripts/core/helpers/data-provider.ts

@@ -32,10 +32,16 @@ export class DataProvider<T extends IBaseRestApiResponse> implements IDataProvid
     private scheduleUpdate: () => void;
     private cache: IRestApiResponse<T>;
 
-    constructor(requestFactory: IRequestFactory, responseHandler: IResponseHandler<T>, listenTo: IListenTo) {
+    constructor(
+        requestFactory: IRequestFactory,
+        responseHandler: IResponseHandler<T>,
+        listenTo: IListenTo,
+        updateTimeout?: number,
+    ) {
         this.requestFactory = requestFactory;
         this.responseHandler = responseHandler;
         this.listenTo = listenTo || {};
+        this.updateTimeout = updateTimeout ?? this.updateTimeout;
         this.scheduleUpdate = debounce(() => this.fetch(), this.updateTimeout);
 
         // init

scripts/data-store.ts

@@ -11,6 +11,8 @@ class DataStore {
     public contentProfiles: OrderedMap<IContentProfile['_id'], IContentProfile>;
     public users: OrderedMap<IUser['_id'], IUser>;
 
+    private initializePromise: Promise<any>;
+
     constructor() {
         this.contentProfiles = OrderedMap();
     }
@@ -81,13 +83,22 @@ class DataStore {
 
                     resolveOnce();
                 },
-                {'users': true},
+                {'users': {
+                    create: true,
+                    delete: true,
+                    update: ['display_name'],
+                }},
+                2000,
             );
         });
     }
 
     initialize() {
-        return Promise.all([this.loadContentProfiles(), this.loadUsers()]);
+        if (this.initializePromise == null) {
+            this.initializePromise = Promise.all([this.loadContentProfiles(), this.loadUsers()]);
+        }
+
+        return this.initializePromise;
     }
 }