M #~: fix re-render row (OpenNebula#3278)

FELDSAM-INC · Oct 25, 2024 · 315896d · 315896d
1 parent af002f2
commit 315896d
Showing 1 changed file with 41 additions and 20 deletions.
diff --git a/src/fireedge/src/client/components/Tables/Enhanced/WrapperRow.js b/src/fireedge/src/client/components/Tables/Enhanced/WrapperRow.js
@@ -158,6 +158,39 @@ CardWrapper.propTypes = {
 
 CardWrapper.displayName = 'CardWrapper'
 
+const SwitchRowComponent = memo(
+  ({ props, hasHeader, RowCardComponent, enabledFullScreen }) => {
+    const internalProps = { ...props }
+    let Component = ''
+
+    if (hasHeader) {
+      internalProps.enabledFullScreen = enabledFullScreen
+      Component = <RowStyle {...internalProps} />
+    } else {
+      Component = <RowCardComponent {...internalProps} />
+      if (enabledFullScreen) {
+        Component = (
+          <CardWrapper {...internalProps}>
+            <RowCardComponent {...internalProps} />
+          </CardWrapper>
+        )
+      }
+    }
+
+    return Component
+  },
+  (prev, next) => prev.RowCardComponent === next.RowCardComponent
+)
+
+SwitchRowComponent.propTypes = {
+  props: PropTypes.any,
+  RowCardComponent: PropTypes.any,
+  hasHeader: PropTypes.bool,
+  enabledFullScreen: PropTypes.bool,
+}
+
+SwitchRowComponent.displayName = 'SwitchRowComponent'
+
 /**
  * @param {ReactElement} RowCardComponent - Standard row component (Card).
  * @param {boolean} enabledFullScreen - to check if the datatable is in full screen mode
@@ -172,26 +205,14 @@ const WrapperRow = (RowCardComponent, enabledFullScreen) => {
   const header = data === 'list'
 
   const component = memo(
-    (props) => {
-      const internalProps = { ...props }
-      let Component = ''
-
-      if (header) {
-        internalProps.enabledFullScreen = enabledFullScreen
-        Component = <RowStyle {...internalProps} />
-      } else {
-        Component = <RowCardComponent {...internalProps} />
-        if (enabledFullScreen) {
-          Component = (
-            <CardWrapper {...internalProps}>
-              <RowCardComponent {...internalProps} />
-            </CardWrapper>
-          )
-        }
-      }
-
-      return Component
-    },
+    (props) => (
+      <SwitchRowComponent
+        props={props}
+        hasHeader={header}
+        RowCardComponent={RowCardComponent}
+        enabledFullScreen={enabledFullScreen}
+      />
+    ),
     (prev, next) => prev.className === next.className
   )