From c0e7804ccd5a73da2966c85b5eff0e2c882815d3 Mon Sep 17 00:00:00 2001
From: Michael Rogers <contact@mhrogers.com>
Date: Wed, 14 Jun 2023 11:45:40 -0500
Subject: [PATCH] Store status roles in an array instead of a singular value

---
 example/exampleUser/ExampleUserProvider.js | 24 ++++++++++++++--------
 src/api/user/StatusAPI.js                  |  4 ++--
 2 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/example/exampleUser/ExampleUserProvider.js b/example/exampleUser/ExampleUserProvider.js
index cb54f81a8c..3afd23760e 100644
--- a/example/exampleUser/ExampleUserProvider.js
+++ b/example/exampleUser/ExampleUserProvider.js
@@ -68,7 +68,10 @@ export default class ExampleUserProvider extends EventEmitter {
         this.user = undefined;
         this.loggedIn = false;
         this.autoLoginUser = undefined;
-        this.status = STATUSES[0];
+        this.statusRoleValues = statusRoles.map(x => ({
+            role: x,
+            status: STATUSES[0]
+        }));
         this.pollQuestion = undefined;
         // this.defaultStatusRole = defaultStatusRole;
         this.statusRoles = statusRoles;
@@ -109,13 +112,13 @@ export default class ExampleUserProvider extends EventEmitter {
         return Promise.resolve(this.user.getRoles().includes(roleId));
     }
 
-    getActiveRole() {
-        if (!this.loggedIn) {
-            Promise.resolve(undefined);
-        }
+    // getActiveRole() {
+    //     if (!this.loggedIn) {
+    //         Promise.resolve(undefined);
+    //     }
 
-        return Promise.resolve(this.selectedStatusRole);
-    }
+    //     return Promise.resolve(this.selectedStatusRole);
+    // }
     getPossibleRoles() {
         return this.user.getRoles();
     }
@@ -130,7 +133,9 @@ export default class ExampleUserProvider extends EventEmitter {
     }
 
     getStatusForRole(role) {
-        return Promise.resolve(this.status);
+        const statusForRole = this.statusRoleValues.find(x => x.role === role);
+
+        return Promise.resolve(statusForRole?.status);
     }
 
     async getDefaultStatusForRole(role) {
@@ -141,7 +146,8 @@ export default class ExampleUserProvider extends EventEmitter {
 
     setStatusForRole(role, status) {
         status.timestamp = Date.now();
-        this.status = status;
+        const matchingIndex = this.statusRoleValues.findIndex(x => x.role === role);
+        this.statusRoleValues[matchingIndex].status = status;
         this.emit('statusChange', {
             role,
             status
diff --git a/src/api/user/StatusAPI.js b/src/api/user/StatusAPI.js
index acae0d54d1..b367246f4e 100644
--- a/src/api/user/StatusAPI.js
+++ b/src/api/user/StatusAPI.js
@@ -151,11 +151,11 @@ export default class StatusAPI extends EventEmitter {
      * @param {Status} status The status to set for the provided role
      * @returns {Promise<Boolean>} true if operation was successful, otherwise false.
      */
-    async setStatusForRole(role, status) {
+    setStatusForRole(_role, status) {
         const provider = this.#userAPI.getProvider();
 
         if (provider.setStatusForRole) {
-            const activeRole = await provider.getActiveRole();
+            const activeRole = this.#userAPI.getActiveRole();
             if (!provider.canProvideStatusForRole(activeRole)) {
                 return false;
             }