feat: pdf authoring

Author: Kelketek

package.json

@@ -20,6 +20,7 @@
     "start:with-theme": "paragon install-theme && npm start && npm install",
     "dev": "PUBLIC_PATH=/authoring/ MFE_CONFIG_API_URL='http://localhost:8000/api/mfe_config/v1' fedx-scripts webpack-dev-server --progress --host apps.local.openedx.io",
     "test": "TZ=UTC fedx-scripts jest --coverage --passWithNoTests",
+    "test:dev": "TZ=UTC fedx-scripts jest --coverage --watch --passWithNoTests",
     "test:ci": "TZ=UTC fedx-scripts jest --silent --coverage --passWithNoTests",
     "types": "tsc --noEmit"
   },

src/course-unit/__mocks__/courseSectionVertical.ts

@@ -82,6 +82,14 @@ export default {
           tab: 'common',
           support_level: true,
         },
+        {
+          display_name: 'PDF',
+          category: 'pdf',
+          boilerplate_name: null,
+          hinted: false,
+          tab: 'common',
+          support_level: true,
+        },
       ],
       display_name: 'Advanced',
       support_legend: {

src/course-unit/add-component/AddComponent.test.tsx

@@ -2,6 +2,7 @@
 /* eslint-disable react/prop-types */
 import userEvent from '@testing-library/user-event';
 
+import { mockWaffleFlags } from '@src/data/apiHooks.mock';
 import {
   act,
   render,
@@ -319,6 +320,55 @@ describe('<AddComponent />', () => {
     });
   });
 
+  it('adds a PDF block from the advanced selection in modal as an mfe-editable block', async () => {
+    const user = userEvent.setup();
+    const {
+      getByRole, queryAllByRole, queryAllByText,
+    } = renderComponent();
+    const advancedBtn = getByRole('button', {
+      name: new RegExp(`${messages.buttonText.defaultMessage} Advanced`, 'i'),
+    });
+
+    await user.click(advancedBtn);
+
+    const dialog = getByRole('dialog');
+    const pdfOption = within(dialog).getByLabelText('PDF');
+    await user.click(pdfOption);
+    const confirmation = within(dialog).getByText('Select');
+    await user.click(confirmation);
+    await waitFor(() => expect(queryAllByRole('dialog')).toEqual([]));
+    expect(queryAllByText('PDF')).toEqual([]);
+    expect(handleCreateNewCourseXBlockMock).toHaveBeenCalled();
+    expect(handleCreateNewCourseXBlockMock).toHaveBeenCalledWith({
+      parentLocator: '123',
+      type: COMPONENT_TYPES.pdf,
+    }, expect.any(Function));
+  });
+
+  it('adds a PDF block from the advanced selection in modal as a traditional block', async () => {
+    const user = userEvent.setup();
+    mockWaffleFlags({ useNewPdfEditor: false });
+    const { getByRole, queryAllByRole } = renderComponent();
+    const advancedBtn = getByRole('button', {
+      name: new RegExp(`${messages.buttonText.defaultMessage} Advanced`, 'i'),
+    });
+
+    await user.click(advancedBtn);
+
+    const dialog = getByRole('dialog');
+    const pdfOption = within(dialog).getByLabelText('PDF');
+    await user.click(pdfOption);
+    const confirmation = within(dialog).getByText('Select');
+    await user.click(confirmation);
+    await waitFor(() => expect(queryAllByRole('dialog')).toEqual([]));
+    expect(handleCreateNewCourseXBlockMock).toHaveBeenCalled();
+    expect(handleCreateNewCourseXBlockMock).toHaveBeenCalledWith({
+      parentLocator: '123',
+      type: COMPONENT_TYPES.pdf,
+      category: COMPONENT_TYPES.pdf,
+    });
+  });
+
   it('verifies "Text" component selection in modal', async () => {
     const user = userEvent.setup();
     const { getByRole, getByText } = renderComponent();

src/course-unit/add-component/AddComponent.tsx

@@ -83,7 +83,7 @@ const AddComponent = ({
   const [selectedComponents, setSelectedComponents] = useState<SelectedComponent[]>([]);
   const [usageId, setUsageId] = useState(null);
   const { sendMessageToIframe } = useIframe();
-  const { useVideoGalleryFlow } = useWaffleFlags(courseId ?? undefined);
+  const { useVideoGalleryFlow, useNewPdfEditor } = useWaffleFlags(courseId ?? undefined);
 
   const courseUnit = useSelector(getCourseUnitData);
   const sequenceId = courseUnit?.ancestorInfo?.ancestors?.[0]?.id;
@@ -170,7 +170,28 @@ const AddComponent = ({
         showAddLibraryContentModal();
         break;
       case COMPONENT_TYPES.advanced:
-        handleCreateNewCourseXBlock({ type: moduleName, category: moduleName, parentLocator: blockId });
+        // TODO: The 'advanced components' concept warrants examination.
+        // 'Advanced' is a bucket where we chuck all the blocks that are
+        // uncommon, or third-party installs. Until now, none of these have
+        // had special editors in this MFE. This is the first.
+        // The fact that advanced modules are handled as a special category
+        // *in code* and not just in UI seems like a mistake in retrospect.
+        //
+        // There will be more of these, and soon.
+        if (moduleName === COMPONENT_TYPES.pdf && useNewPdfEditor) {
+          handleCreateNewCourseXBlock(
+            { type: moduleName, parentLocator: blockId },
+            /* istanbul ignore next */
+            ({ courseKey, locator }) => {
+              setCourseId(courseKey);
+              setBlockType(moduleName);
+              setNewBlockId(locator);
+              showXBlockEditorModal();
+            },
+          );
+        } else {
+          handleCreateNewCourseXBlock({ type: moduleName, category: moduleName, parentLocator: blockId });
+        }
         break;
       case COMPONENT_TYPES.openassessment:
         handleCreateNewCourseXBlock({ boilerplate: moduleName, category: type, parentLocator: blockId });

src/course-unit/xblock-container-iframe/index.tsx

@@ -1,6 +1,6 @@
 import { getConfig } from '@edx/frontend-platform';
 import {
-  FC, useEffect, useState, useMemo, useCallback,
+  FC, useEffect, useState, useMemo, useCallback, Fragment,
 } from 'react';
 import { useIntl } from '@edx/frontend-platform/i18n';
 import { useToggle, Sheet, StandardModal } from '@openedx/paragon';

src/data/api.ts

@@ -85,6 +85,7 @@ export const waffleFlagDefaults = {
   useNewUnitPage: false,
   useNewCertificatesPage: true,
   useNewTextbooksPage: true,
+  useNewPdfEditor: true,
   useReactMarkdownEditor: true,
   useVideoGalleryFlow: false,
   enableAuthzCourseAuthoring: false,

src/editors/api.ts

@@ -0,0 +1,23 @@
+/* Shared react-query hooks for editors. */
+import { useSelector } from 'react-redux';
+import { EditorState, selectors } from '@src/editors/data/redux';
+import { useEditorContext } from '@src/editors/EditorContext';
+import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
+import { useMutation } from '@tanstack/react-query';
+import * as urls from '@src/editors/data/services/cms/urls';
+
+export const useCourseAssetUpload = () => {
+  const studioEndpointUrl = useSelector((state: EditorState) => selectors.app.studioEndpointUrl(state))!;
+  const { learningContextId } = useEditorContext();
+  const client = getAuthenticatedHttpClient();
+  return useMutation({
+    mutationFn: (file: File) => {
+      const data = new FormData();
+      data.append('file', file);
+      return client.post(
+        urls.courseAssets({ studioEndpointUrl, learningContextId }),
+        data,
+      );
+    },
+  });
+};

src/editors/containers/PdfEditor/api.ts

@@ -0,0 +1,27 @@
+import { useQuery } from '@tanstack/react-query';
+import { useSelector } from 'react-redux';
+import { EditorState, selectors } from '@src/editors/data/redux';
+import { camelizeKeys } from '@src/editors/utils';
+import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
+import type { AxiosResponse } from 'axios';
+import * as urls from '@src/editors/data/services/cms/urls';
+
+interface UseBlockDataParams {
+  blockId: string,
+  uniqueId: string,
+  handlerName: string,
+}
+
+// Unique ID required due to intractable race conditions. See ./contexts.tsx file.
+export const useBlockData = <T>({ blockId, uniqueId, handlerName }: UseBlockDataParams) => {
+  const studioEndpointUrl = useSelector((state: EditorState) => selectors.app.studioEndpointUrl(state))!;
+  const client = getAuthenticatedHttpClient();
+  return useQuery<T>({
+    queryKey: ['blockData', blockId, uniqueId],
+    staleTime: Infinity,
+    queryFn: ({ signal }) => client.get(
+      urls.xblockHandlerUrl({ blockId, studioEndpointUrl, handlerName }),
+      { cancelSource: signal },
+    ).then((res: AxiosResponse<unknown>) => camelizeKeys(res.data) as T),
+  });
+};

src/editors/containers/PdfEditor/components/PdfEditingModal.tsx

@@ -0,0 +1,77 @@
+import { EditorComponent } from '@src/editors/EditorComponent';
+import { useFormikContext } from 'formik';
+import React, {
+  PropsWithChildren, useContext, useEffect, useRef,
+} from 'react';
+import EditorContainer from '@src/editors/containers/EditorContainer';
+import { PdfBlockContext, PdfState } from '@src/editors/containers/PdfEditor/contexts';
+import { isEqual } from 'lodash';
+import DownloadOptions from '@src/editors/containers/PdfEditor/components/sections/DownloadOptions';
+import { UploadWidget } from '@src/editors/sharedComponents/UploadWidget';
+import { Spinner } from '@openedx/paragon';
+import { FormattedMessage, useIntl } from '@edx/frontend-platform/i18n';
+import messages from './messages';
+
+const EditorWrapper: React.FC<PropsWithChildren> = ({ children }) => {
+  const intl = useIntl();
+  const { isPending, fetchError } = useContext(PdfBlockContext);
+  if (fetchError) {
+    return (
+      <div className="text-center p-6">
+        <FormattedMessage {...messages.blockFailed} />
+      </div>
+    );
+  }
+  if (isPending) {
+    return (
+      <div className="text-center p-6">
+        <Spinner
+          animation="border"
+          className="m-3"
+          screenReaderText={intl.formatMessage(messages.blockLoading)}
+        />
+      </div>
+    );
+  }
+  return <>{children}</>; /* eslint-disable-line react/jsx-no-useless-fragment */
+};
+
+const PdfEditingModal: React.FC<EditorComponent> = (props) => {
+  const intl = useIntl();
+  const { fields } = useContext(PdfBlockContext);
+  const originalState = useRef({ ...fields });
+  const { values, setValues } = useFormikContext<PdfState>();
+
+  useEffect(() => {
+    // Form is initialized before we get these values, so we have to set them
+    // when they arrive.
+    void setValues(fields); // eslint-disable-line no-void
+  }, [fields]);
+
+  const isDirty = () => isEqual(originalState, values);
+
+  const getContent = () => {
+    const settings = { ...values };
+    // disableAllDownload is not a setting we control, but a backend flag. Have to remove it or the
+    // backend will reject.
+    return Object.fromEntries(Object.entries(settings).filter(([key]) => key !== 'disableAllDownload'));
+  };
+
+  return (
+    <EditorContainer {...props} isDirty={isDirty} getContent={getContent}>
+      <EditorWrapper>
+        <div className="mt-2">
+          <UploadWidget
+            supportedFileFormats="application/pdf"
+            urlFieldName="url"
+            label={intl.formatMessage(messages.urlFieldLabel)}
+            id="pdf-url"
+          />
+        </div>
+        <DownloadOptions />
+      </EditorWrapper>
+    </EditorContainer>
+  );
+};
+
+export default PdfEditingModal;

src/editors/containers/PdfEditor/components/PdfEditorContainer.tsx

@@ -0,0 +1,16 @@
+import React, { useContext } from 'react';
+import { PdfBlockContext } from '@src/editors/containers/PdfEditor/contexts';
+import { Formik } from 'formik';
+import { EditorComponent } from '@src/editors/EditorComponent';
+import PdfEditingModal from '@src/editors/containers/PdfEditor/components/PdfEditingModal';
+
+const PdfEditorContainer: React.FC<EditorComponent> = (props) => {
+  const { fields } = useContext(PdfBlockContext);
+  return (
+    <Formik initialValues={fields} onSubmit={() => undefined}>
+      <PdfEditingModal {...props} />
+    </Formik>
+  );
+};
+
+export default PdfEditorContainer;