Revert "Fix bug to allow any contributor to see preprint rejection co…

…mments (CenterForOpenScience#2482)" This reverts commit a2eb33f.
cslzchen · Jan 20, 2025 · 57fcd09 · 57fcd09
1 parent a2eb33f
commit 57fcd09
Showing 1 changed file with 12 additions and 14 deletions.
diff --git a/app/preprints/detail/route.ts b/app/preprints/detail/route.ts
@@ -96,22 +96,20 @@ export default class PreprintsDetail extends Route {
             let hasPendingWithdrawal = false;
             let latestWithdrawalRequest = null;
             let latestAction = null;
-            if (preprint.currentUserPermissions.length > 0) {
+            if (preprintWithdrawableState && preprint.currentUserIsAdmin) {
                 const reviewActions = await preprint?.queryHasMany('reviewActions');
                 latestAction = reviewActions.firstObject;
-                if (preprintWithdrawableState) {
-                    const withdrawalRequests = await preprint?.queryHasMany('requests');
-                    latestWithdrawalRequest = withdrawalRequests.firstObject;
-                    if (latestWithdrawalRequest) {
-                        hasPendingWithdrawal = latestWithdrawalRequest.machineState === 'pending';
-                        const requestActions = await withdrawalRequests.firstObject?.queryHasMany('actions', {
-                            sort: '-modified',
-                        });
-                        latestAction = requestActions.firstObject;
-                        // @ts-ignore: ActionTrigger is never
-                        if (latestAction && latestAction.actionTrigger === 'reject') {
-                            isWithdrawalRejected = true;
-                        }
+                const withdrawalRequests = await preprint?.queryHasMany('requests');
+                latestWithdrawalRequest = withdrawalRequests.firstObject;
+                if (latestWithdrawalRequest) {
+                    hasPendingWithdrawal = latestWithdrawalRequest.machineState === 'pending';
+                    const requestActions = await withdrawalRequests.firstObject?.queryHasMany('actions', {
+                        sort: '-modified',
+                    });
+                    latestAction = requestActions.firstObject;
+                    // @ts-ignore: ActionTrigger is never
+                    if (latestAction && latestAction.actionTrigger === 'reject') {
+                        isWithdrawalRejected = true;
                     }
                 }
             }