(fix) requiredPrivilege should not display an error if privilege is u…

…ndefined (#504)
openmrs · Aug 15, 2022 · 144a312 · 144a312
1 parent 3038800
commit 144a312
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/framework/esm-api/src/shared-api-objects/current-user.ts b/packages/framework/esm-api/src/shared-api-objects/current-user.ts
@@ -123,7 +123,7 @@ function setUserLanguage(data: Session) {
 }
 
 function userHasPrivilege(
-  requiredPrivilege: string | string[],
+  requiredPrivilege: string | string[] | undefined,
   user: { privileges: Array<Privilege> }
 ) {
   if (typeof requiredPrivilege === "string") {
@@ -134,7 +134,7 @@ function userHasPrivilege(
     return requiredPrivilege.every(
       (rp) => !isUndefined(user.privileges.find((p) => rp === p.display))
     );
-  } else {
+  } else if (!isUndefined(requiredPrivilege)) {
     console.error(`Could not understand privileges "${requiredPrivilege}"`);
   }