diff --git a/services/web/app/src/Features/Authorization/AuthorizationManager.js b/services/web/app/src/Features/Authorization/AuthorizationManager.js index 04291cfac1..612d98585e 100644 --- a/services/web/app/src/Features/Authorization/AuthorizationManager.js +++ b/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, diff --git a/services/web/test/unit/src/Authorization/AuthorizationManagerTests.js b/services/web/test/unit/src/Authorization/AuthorizationManagerTests.js index 02ae7c1912..5d88a319fc 100644 --- a/services/web/test/unit/src/Authorization/AuthorizationManagerTests.js +++ b/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, diff --git a/services/web/test/unit/src/Authorization/AuthorizationMiddlewareTests.js b/services/web/test/unit/src/Authorization/AuthorizationMiddlewareTests.js index bc3535c3c9..b4d6cc264f 100644 --- a/services/web/test/unit/src/Authorization/AuthorizationMiddlewareTests.js +++ b/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(), },