fix: do not check ACL VIEW_HIERARCHY when access the Browse page

openequella · May 29, 2024 · 4a1abdb · 4a1abdb
1 parent 2f859a1
commit 4a1abdb
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/react-front-end/tsrc/mainui/IndexPage.tsx b/react-front-end/tsrc/mainui/IndexPage.tsx
@@ -259,13 +259,17 @@ export default function IndexPage() {
 
             // When the legacy path doesn't have query param 'topic or when it has but the value
             // is 'ALL', render 'BrowseHierarchyPage'.
-            return renderProtectedPage(
-              routeProps,
+            const page =
               topic === null || LegacyBrowseHierarchyLiteral.is(topic)
-                ? BrowseHierarchyPage
-                : RootHierarchyPage,
-              [isHierarchyPageACLGranted, isViewHierarchyTopicACLGranted],
-            );
+                ? renderProtectedPage(routeProps, BrowseHierarchyPage, [
+                    isHierarchyPageACLGranted,
+                  ])
+                : renderProtectedPage(routeProps, RootHierarchyPage, [
+                    isHierarchyPageACLGranted,
+                    isViewHierarchyTopicACLGranted,
+                  ]);
+
+            return page;
           }}
         />
         <Route render={renderLegacyContent} />