chore: one more check for null (#602)

react-component · Dec 11, 2024 · 850dfdc · 850dfdc
1 parent d2a8db3
commit 850dfdc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/ref.ts b/src/ref.ts
@@ -36,6 +36,10 @@ export const useComposeRef = <T>(...refs: React.Ref<T>[]): React.Ref<T> => {
 };
 
 export const supportRef = (nodeOrComponent: any): boolean => {
+  if (!nodeOrComponent) {
+    return false;
+  }
+
   // React 19 no need `forwardRef` anymore. So just pass if is a React element.
   if (
     isReactElement(nodeOrComponent) &&

diff --git a/tests/ref.test.tsx b/tests/ref.test.tsx
@@ -178,6 +178,10 @@ describe('ref', () => {
       expect(supportRef(MemoFC)).toBeTruthy();
       expect(supportRef(holderRef.current.props.children)).toBeTruthy();
     });
+
+    it('skip null', () => {
+      expect(supportRef(null)).toBeFalsy();
+    });
   });
 
   describe('nodeSupportRef', () => {