From 2c68af50e0cc51248ec162945f65fb1802b568f7 Mon Sep 17 00:00:00 2001
From: afc163 <afc163@gmail.com>
Date: Thu, 8 Feb 2024 11:19:12 +0800
Subject: [PATCH] fix: code scanning alerts (#45)

---
 src/Overflow.tsx   | 43 +++++++++++++++++--------------------------
 tests/ssr.spec.tsx |  1 -
 2 files changed, 17 insertions(+), 27 deletions(-)

diff --git a/src/Overflow.tsx b/src/Overflow.tsx
index de6241b..acc850c 100644
--- a/src/Overflow.tsx
+++ b/src/Overflow.tsx
@@ -357,7 +357,6 @@ function Overflow<ItemType = any>(
       };
 
   // >>>>> Rest node
-  let restNode: React.ReactNode;
   const restContextProps = {
     order: displayRest ? mergedDisplayCount : Number.MAX_SAFE_INTEGER,
     className: `${itemPrefixCls}-rest`,
@@ -365,10 +364,18 @@ function Overflow<ItemType = any>(
     display: displayRest,
   };
 
-  if (!renderRawRest) {
-    const mergedRenderRest = renderRest || defaultRenderRest;
+  const mergedRenderRest = renderRest || defaultRenderRest;
 
-    restNode = (
+    const restNode = renderRawRest ? (
+      <OverflowContext.Provider
+        value={{
+          ...itemSharedProps,
+          ...restContextProps,
+        }}
+      >
+        {renderRawRest(omittedItems)}
+      </OverflowContext.Provider>
+    ) : (
       <Item
         {...itemSharedProps}
         // When not show, order should be the last
@@ -379,20 +386,8 @@ function Overflow<ItemType = any>(
           : mergedRenderRest}
       </Item>
     );
-  } else if (renderRawRest) {
-    restNode = (
-      <OverflowContext.Provider
-        value={{
-          ...itemSharedProps,
-          ...restContextProps,
-        }}
-      >
-        {renderRawRest(omittedItems)}
-      </OverflowContext.Provider>
-    );
-  }
 
-  let overflowNode = (
+  const overflowNode = (
     <Component
       className={classNames(!invalidate && prefixCls, className)}
       style={style}
@@ -422,15 +417,11 @@ function Overflow<ItemType = any>(
     </Component>
   );
 
-  if (isResponsive) {
-    overflowNode = (
-      <ResizeObserver onResize={onOverflowResize} disabled={!shouldResponsive}>
-        {overflowNode}
-      </ResizeObserver>
-    );
-  }
-
-  return overflowNode;
+  return isResponsive ? (
+    <ResizeObserver onResize={onOverflowResize} disabled={!shouldResponsive}>
+      {overflowNode}
+    </ResizeObserver>
+  ) : overflowNode;
 }
 
 const ForwardOverflow = React.forwardRef(Overflow);
diff --git a/tests/ssr.spec.tsx b/tests/ssr.spec.tsx
index 75e5038..50c9798 100644
--- a/tests/ssr.spec.tsx
+++ b/tests/ssr.spec.tsx
@@ -1,6 +1,5 @@
 import React from 'react';
 import { render } from 'enzyme';
-import { act } from 'react-dom/test-utils';
 import Overflow from '../src';
 
 interface ItemType {