useEvent: Only check for calls during render in development (#1269)

mui · Nov 3, 2022 · ef5ad1b · ef5ad1b
1 parent 4e34e6e
commit ef5ad1b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/toolpad-app/src/utils/useEvent.ts b/packages/toolpad-app/src/utils/useEvent.ts
@@ -3,6 +3,7 @@ import * as React from 'react';
 let dispatcher: any = null;
 
 function getCurrentDispatcher() {
+  // Only used to provide debug info during development.
   // eslint-disable-next-line no-underscore-dangle
   return (React as any).__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED.ReactCurrentDispatcher
     .current;
@@ -29,8 +30,8 @@ export default function useEvent<F extends (...args: any[]) => void>(handler: F)
   });
   // @ts-expect-error
   return React.useCallback((...args) => {
-    if (isInRender()) {
-      throw new Error('Cannot call event in Render!');
+    if (process.env.NODE_ENV !== 'production' && isInRender()) {
+      console.error(`Functions returned by useEvent can't be called during a React render.`);
     }
     const fn = ref.current;
     fn(...args);