refactor: use same path sufix

openedx · Apr 12, 2024 · 53d70e3 · 53d70e3
1 parent f4cf7fd
commit 53d70e3
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/search-modal/SearchResult.jsx b/src/search-modal/SearchResult.jsx
@@ -75,12 +75,13 @@ const ItemIcon = {
 const getContextUrl = (hit, newWindow, libraryAuthoringMfeUrl) => {
   const { context_key: contextKey, usage_key: usageKey } = hit;
   if (contextKey.startsWith('course-v1:')) {
+    const courseSufix = `course/${contextKey}?show=${encodeURIComponent(usageKey)}`;
     if (newWindow) {
-      return `${getPath(getConfig().PUBLIC_PATH)}course/${contextKey}?show=${encodeURIComponent(usageKey)}`;
+      return `${getPath(getConfig().PUBLIC_PATH)}${courseSufix}`;
     }
-    return `/course/${contextKey}?show=${encodeURIComponent(usageKey)}`;
+    return `/${courseSufix}`;
   }
-  if (usageKey.includes('lb:')) {
+  if (usageKey.startsWith('lb:')) {
     return `${libraryAuthoringMfeUrl}library/${contextKey}`;
   }
   return '#';