@bpmn-io/form-js-viewer 1.13.1 → 1.13.2

package/dist/index.cjs

@@ -5454,7 +5454,7 @@ function Table(props) {
     key
   }) => key);
   const evaluatedDataSource = useExpressionEvaluation(dataSource);
-  const data = Array.isArray(evaluatedDataSource) ? evaluatedDataSource.filter(i => i !== undefined) : [];
+  const data = Array.isArray(evaluatedDataSource) ? evaluatedDataSource.filter(entry => !minDash.isNil(entry) || typeof entry !== 'object') : [];
   const sortedData = sortBy === null ? data : sortByColumn(data, sortBy.key, sortBy.direction);
 
   /** @type {unknown[][]} */
@@ -5464,12 +5464,6 @@ function Table(props) {
   hooks.useEffect(() => {
     setCurrentPage(0);
   }, [rowCount, sortBy]);
-  const serializeCellData = cellData => {
-    if (cellData !== null && typeof cellData === 'object') {
-      return JSON.stringify(cellData);
-    }
-    return cellData;
-  };
 
   /** @param {string} key */
   function toggleSortBy(key) {
@@ -5740,6 +5734,17 @@ function getHeaderAriaLabel(sortBy, key, label) {
   return `Click to sort by ${label} ascending`;
 }
 
+/**
+ * @param {unknown} cellData
+ * @returns string
+ */
+function serializeCellData(cellData) {
+  if (cellData !== null && typeof cellData === 'object') {
+    return JSON.stringify(cellData);
+  }
+  return `${cellData || ''}`;
+}
+
 const FILE_PICKER_FILE_KEY_PREFIX = 'files::';
 
 const type$1 = 'filepicker';
@@ -5920,6 +5925,7 @@ const type = 'documentPreview';
 /**
  * @typedef DocumentMetadata
  * @property {string} documentId
+ * @property {string} contentHash
  * @property {Object} metadata
  * @property {string|undefined} [metadata.contentType]
  * @property {string} metadata.fileName
@@ -6079,7 +6085,11 @@ function DocumentRenderer(props) {
   const [hasError, setHasError] = hooks.useState(false);
   const ref = hooks.useRef(null);
   const isInViewport = useInViewport(ref);
-  const fullUrl = endpoint.replace(DOCUMENT_ID_PLACEHOLDER, documentMetadata.documentId);
+  const fullUrl = buildUrl({
+    baseUrl: endpoint,
+    documentId: documentMetadata.documentId,
+    contentHash: documentMetadata.contentHash
+  });
   const singleDocumentContainerClassName = `fjs-${type}-single-document-container`;
   const errorMessageId = `${domId}-error-message`;
   const errorMessage = 'Unable to download document';
@@ -6189,12 +6199,16 @@ function DownloadButton(props) {
 
 /**
  *
- * @param {import("preact").RefObject<HTMLElement>} ref
+ * @param {import("preact").RefObject<HTMLElement|null>} ref
  * @returns boolean
  */
 function useInViewport(ref) {
   const [isInViewport, setIsInViewport] = hooks.useState(false);
   hooks.useEffect(() => {
+    const container = ref.current;
+    if (!container) {
+      return;
+    }
     const observer = new IntersectionObserver(([entry]) => {
       if (entry.isIntersecting) {
         setIsInViewport(true);
@@ -6202,18 +6216,39 @@ function useInViewport(ref) {
     }, {
       threshold: 0
     });
-    if (ref.current) {
-      observer.observe(ref.current);
-    }
+    observer.observe(container);
     return () => {
-      if (ref.current) {
-        observer.unobserve(ref.current);
+      if (container) {
+        observer.unobserve(container);
       }
     };
   }, [ref]);
   return isInViewport;
 }
 
+/**
+ * This solution should be a temporary fix, we should try to remove it via: https://github.com/bpmn-io/form-js/issues/1341
+ *
+ * @param {Object} options
+ * @param {string} options.baseUrl
+ * @param {string} options.documentId
+ * @param {string} [options.contentHash]
+ *
+ * @returns {string}
+ */
+function buildUrl(options) {
+  const {
+    baseUrl,
+    documentId,
+    contentHash
+  } = options;
+  const finalUrl = new URL(baseUrl.replace(DOCUMENT_ID_PLACEHOLDER, documentId));
+  if (contentHash !== undefined) {
+    finalUrl.searchParams.set('contentHash', contentHash);
+  }
+  return decodeURI(finalUrl.toString());
+}
+
 /**
  * This file must not be changed or exchanged.
  *