Ver Fonte

Delete unused canUserReviewProjectContent function (#23073)

GitOrigin-RevId: 5c2e17a44623916e494a24bf18dbff4e846734d3
Domagoj Kriskovic há 1 ano atrás
pai
commit
7e5a0a9bea

+ 0 - 15
services/web/app/src/Features/Authorization/AuthorizationManager.js

@@ -191,19 +191,6 @@ async function canUserReadProject(userId, projectId, token) {
   ].includes(privilegeLevel)
 }
 
-async function canUserReviewProjectContent(userId, projectId, token) {
-  const privilegeLevel = await getPrivilegeLevelForProject(
-    userId,
-    projectId,
-    token
-  )
-  return [
-    PrivilegeLevels.OWNER,
-    PrivilegeLevels.READ_AND_WRITE,
-    PrivilegeLevels.REVIEW,
-  ].includes(privilegeLevel)
-}
-
 async function canUserWriteProjectContent(userId, projectId, token) {
   const privilegeLevel = await getPrivilegeLevelForProject(
     userId,
@@ -302,7 +289,6 @@ async function canUserSendComment(userId, projectId, token) {
 module.exports = {
   canUserReadProject: callbackify(canUserReadProject),
   canUserWriteProjectContent: callbackify(canUserWriteProjectContent),
-  canUserReviewProjectContent: callbackify(canUserReviewProjectContent),
   canUserDeleteOrResolveThread: callbackify(canUserDeleteOrResolveThread),
   canUserSendComment: callbackify(canUserSendComment),
   canUserWriteProjectSettings: callbackify(canUserWriteProjectSettings),
@@ -315,7 +301,6 @@ module.exports = {
   promises: {
     canUserReadProject,
     canUserWriteProjectContent,
-    canUserReviewProjectContent,
     canUserDeleteOrResolveThread,
     canUserSendComment,
     canUserWriteProjectSettings,

+ 0 - 9
services/web/test/unit/src/Authorization/AuthorizationManagerTests.js

@@ -453,15 +453,6 @@ describe('AuthorizationManager', function () {
     tokenReadOnly: true,
   })
 
-  testPermission('canUserReviewProjectContent', {
-    siteAdmin: true,
-    owner: true,
-    readAndWrite: true,
-    review: true,
-    publicReadAndWrite: true,
-    tokenReadAndWrite: true,
-  })
-
   testPermission('canUserSendComment', {
     siteAdmin: true,
     owner: true,

+ 0 - 1
services/web/test/unit/src/Authorization/AuthorizationMiddlewareTests.js

@@ -29,7 +29,6 @@ describe('AuthorizationMiddleware', function () {
         canUserSendComment: sinon.stub(),
         canUserAdminProject: sinon.stub(),
         canUserRenameProject: sinon.stub(),
-        canUserReviewProjectContent: sinon.stub(),
         isUserSiteAdmin: sinon.stub(),
         isRestrictedUserForProject: sinon.stub(),
       },