README.md file viewer not displaying content according to file extens…

…ion (#4380)
treeverse · Oct 18, 2022 · aef425e · aef425e
1 parent 7c967e6
commit aef425e
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/webui/src/pages/repositories/repository/objects.jsx b/webui/src/pages/repositories/repository/objects.jsx
@@ -297,20 +297,22 @@ const TreeContainer = ({
 }
 
 const ReadmeContainer = ({repo, reference, path='', refreshDep=''}) => {
+    let readmePath = '';
+
+    if (path) {
+        readmePath = path.endsWith('/') ? `${path}${README_FILE_NAME}` : `${path}/${README_FILE_NAME}`;
+    } else {
+        readmePath = README_FILE_NAME;
+    }
     const {response, error, loading} = useAPI(async () => {
-        if (path) {
-            path = path.endsWith('/') ? `${path}${README_FILE_NAME}` : `${path}/${README_FILE_NAME}`;
-        } else {
-            path = README_FILE_NAME;
-        }
-        return await objects.getWithHeaders(repo.id, reference.id, path);
+        return await objects.getWithHeaders(repo.id, reference.id, readmePath);
     }, [path, refreshDep]);
 
     if (loading || error) {
         return <></>;
     }
 
-    const fileExtension = getFileExtension(path);
+    const fileExtension = getFileExtension(readmePath);
     const contentType = getContentType(response?.headers);
 
     return (