Merge pull request #864 from LiamEdwardsLamarche/fix/htmlRenderRollback

html render rollback due to too many bugs

Author: mariusconjeaud

src/chart/graph/component/GraphEntityInspectionTable.tsx

@@ -2,7 +2,7 @@ import React from 'react';
 import ShowMoreText from 'react-show-more-text';
 import { Checkbox, Table, TableBody, TableCell, TableContainer, TableHead, TableRow } from '@mui/material';
 import { TextLink } from '@neo4j-ndl/react';
-import DOMPurify from 'dompurify';
+// import DOMPurify from 'dompurify'; 
 
 export const formatProperty = (property) => {
   const str = property?.toString() || '';
@@ -13,8 +13,7 @@ export const formatProperty = (property) => {
       </TextLink>
     );
   }
-  const cleanValue = DOMPurify.sanitize(str);
-  return <div dangerouslySetInnerHTML={{ __html: cleanValue }} />;
+  return str; 
 };
 
 /**
@@ -91,7 +90,7 @@ export const GraphEntityInspectionTable = ({
                     {key}
                   </TableCell>
                   <TableCell align={'left'} style={{ color: tableTextColor }}>
-                    <ShowMoreText lines={2}>{formatProperty(entity?.properties[key])}</ShowMoreText>
+                    <ShowMoreText lines={2}>{formatProperty(entity?.properties[key])}</ShowMoreText> 
                   </TableCell>
                   {checklistEnabled ? (
                     <TableCell align={'center'}>

src/chart/table/TableChart.tsx

@@ -38,15 +38,6 @@ const fallbackRenderer = (value) => {
   return JSON.stringify(value);
 };
 
-function htmlToPlainText(html): string {
-  // Create a temporary div element to hold the sanitized HTML content
-  const tempElement = document.createElement('div');
-  // Set the HTML content directly as innerHTML of the temporary element
-  tempElement.innerHTML = html.props.dangerouslySetInnerHTML.__html;
-  // Extract plain text using textContent
-  return tempElement.textContent ?? '';
-}
-
 function renderAsButtonWrapper(renderer) {
   return function renderAsButton(value) {
     const outputValue = renderer(value, true);
@@ -59,7 +50,7 @@ function renderAsButtonWrapper(renderer) {
         style={{ width: '100%', marginLeft: '5px', marginRight: '5px' }}
         variant='contained'
         color='primary'
-      >{`${htmlToPlainText(outputValue)}`}</Button>
+      >{outputValue}</Button>
     );
   };
 }

src/report/ReportRecordProcessing.tsx

@@ -10,7 +10,7 @@ import {
   valueIsPath,
   valueIsRelationship,
 } from '../chart/ChartUtils';
-import DOMPurify from 'dompurify';
+// import DOMPurify from 'dompurify';
 
 /**
  * Collects all node labels and node properties in a set of Neo4j records.
@@ -267,8 +267,7 @@ function RenderString(value) {
       </TextLink>
     );
   }
-  const cleanValue = DOMPurify.sanitize(str);
-  return <div dangerouslySetInnerHTML={{ __html: cleanValue }} />;
+  return str;
 }
 
 function RenderLink(value, disabled = false) {