#9453 Minor authorisation fix and enable_review_round_history feature…

… switch removal
pkp · May 1, 2024 · 8b40f9f · 8b40f9f
1 parent 9c3d107
commit 8b40f9f
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 8 deletions.
diff --git a/api/v1/reviews/PKPReviewController.php b/api/v1/reviews/PKPReviewController.php
@@ -75,15 +75,10 @@ public function getGroupRoutes(): void
      */
     public function authorize(PKPRequest $request, array &$args, array $roleAssignments): bool
     {
-        $illuminateRequest = $args[0]; /** @var \Illuminate\Http\Request $illuminateRequest */
-        $actionName = static::getRouteActionName($illuminateRequest);
-
         $this->addPolicy(new UserRolesRequiredPolicy($request), true);
         $this->addPolicy(new ContextAccessPolicy($request, $roleAssignments));
 
-        if ($actionName === 'getHistory') {
-            $this->addPolicy(new SubmissionAccessPolicy($request, $args, $roleAssignments, 'submissionId', true));
-        }
+        $this->addPolicy(new SubmissionAccessPolicy($request, $args, $roleAssignments, 'submissionId', true));
 
         return parent::authorize($request, $args, $roleAssignments);
     }

diff --git a/pages/reviewer/PKPReviewerHandler.php b/pages/reviewer/PKPReviewerHandler.php
@@ -95,7 +95,6 @@ public function submission(array $args, PKPRequest $request): void
         ]);
 
         $templateMgr->setState([
-            'isReviewRoundHistoryEnabled' => Config::getVar('features', 'enable_review_round_history'),
             'pageInitConfig' => [
                 'reviewRoundHistories' => $reviewRoundHistories,
             ]

diff --git a/templates/reviewer/review/reviewStepHeader.tpl b/templates/reviewer/review/reviewStepHeader.tpl
@@ -15,7 +15,7 @@
 	</h1>
 
 	<!-- rest of this template will be moved inside this component in future -->
-	<reviewer-submission-page v-if="isReviewRoundHistoryEnabled" v-bind="pageInitConfig"></reviewer-submission-page>
+	<reviewer-submission-page v-bind="pageInitConfig"></reviewer-submission-page>
 
 	<script type="text/javascript">
 		// Attach the JS file tab handler.