fix: add optional library context hook

openedx · Oct 21, 2024 · 6a52f3c · 6a52f3c
1 parent 56ff971
commit 6a52f3c
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/src/editors/containers/TextEditor/index.jsx b/src/editors/containers/TextEditor/index.jsx
@@ -18,7 +18,7 @@ import * as hooks from './hooks';
 import messages from './messages';
 import TinyMceWidget from '../../sharedComponents/TinyMceWidget';
 import { prepareEditorRef, replaceStaticWithAsset } from '../../sharedComponents/TinyMceWidget/hooks';
-import { useLibraryContext } from '../../../library-authoring/common/context';
+import { useOptionalLibraryContext } from '../../../library-authoring/common/context';
 
 const TextEditor = ({
   onClose,
@@ -43,9 +43,9 @@ const TextEditor = ({
   });
   const editorContent = newContent || initialContent;
   let documentURL;
-  const { sidebarComponentUsageKey: usageKey } = useLibraryContext();
-  if (isLibrary) {
-    documentURL = `${getConfig().STUDIO_BASE_URL }/library_assets/blocks/${ usageKey }/`;
+  const ctx = useOptionalLibraryContext();
+  if (isLibrary && ctx !== undefined) {
+    documentURL = `${getConfig().STUDIO_BASE_URL }/library_assets/blocks/${ ctx.sidebarComponentUsageKey }/`;
   }
 
   if (!refReady) { return null; }

diff --git a/src/editors/sharedComponents/TinyMceWidget/__snapshots__/index.test.jsx.snap b/src/editors/sharedComponents/TinyMceWidget/__snapshots__/index.test.jsx.snap
@@ -19,6 +19,7 @@ exports[`TinyMceWidget snapshots ImageUploadModal is not rendered 1`] = `
       {
         "clearSelection": [MockFunction hooks.selectedImage.clearSelection],
         "content": undefined,
+        "documentURL": undefined,
         "editorContentHtml": undefined,
         "editorRef": {
           "current": {
@@ -87,6 +88,7 @@ exports[`TinyMceWidget snapshots SourcecodeModal is not rendered 1`] = `
       {
         "clearSelection": [MockFunction hooks.selectedImage.clearSelection],
         "content": undefined,
+        "documentURL": undefined,
         "editorContentHtml": undefined,
         "editorRef": {
           "current": {
@@ -166,6 +168,7 @@ exports[`TinyMceWidget snapshots renders as expected with default behavior 1`] =
       {
         "clearSelection": [MockFunction hooks.selectedImage.clearSelection],
         "content": undefined,
+        "documentURL": undefined,
         "editorContentHtml": undefined,
         "editorRef": {
           "current": {

diff --git a/src/library-authoring/common/context.tsx b/src/library-authoring/common/context.tsx
@@ -199,3 +199,7 @@ export function useLibraryContext(): LibraryContextData {
   }
   return ctx;
 }
+
+export function useOptionalLibraryContext(): LibraryContextData | undefined {
+  return useContext(LibraryContext);
+}