fix(ui): check for correct role defined before before calling into it (…

…#1785) close #1784
tchiotludo · May 23, 2024 · f75cb86 · f75cb86
1 parent d29c54d
commit f75cb86
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/client/src/utils/AkhqRoutes.jsx b/client/src/utils/AkhqRoutes.jsx
@@ -159,23 +159,23 @@ class AkhqRoutes extends Root {
                 <Route exact path="/ui/:clusterId/topic/create" element={<TopicCreate />} />
               )}
 
-              {roles && roles.TOPIC && roles.TOPIC_DATA.includes('CREATE') && (
+              {roles && roles.TOPIC_DATA && roles.TOPIC_DATA.includes('CREATE') && (
                 <Route
                   exact
                   path="/ui/:clusterId/topic/:topicId/produce"
                   element={<TopicProduce />}
                 />
               )}
 
-              {roles && roles.TOPIC && roles.TOPIC_DATA.includes('CREATE') && (
+              {roles && roles.TOPIC_DATA && roles.TOPIC_DATA.includes('CREATE') && (
                 <Route
                   exact
                   path="/ui/:clusterId/topic/:topicId/increasepartition"
                   element={<TopicIncreaseParition />}
                 />
               )}
 
-              {roles && roles.TOPIC && roles.TOPIC_DATA.includes('CREATE') && (
+              {roles && roles.TOPIC_DATA && roles.TOPIC_DATA.includes('CREATE') && (
                 <Route exact path="/ui/:clusterId/topic/:topicId/copy" element={<TopicCopy />} />
               )}
 
@@ -187,7 +187,7 @@ class AkhqRoutes extends Root {
                 />
               )}
 
-              {roles && roles.TOPIC && roles.TOPIC_DATA.includes('READ') && (
+              {roles && roles.TOPIC_DATA && roles.TOPIC_DATA.includes('READ') && (
                 <Route exact path="/ui/:clusterId/tail" element={<Tail />} />
               )}