[mui-lab] Fix issues reported by react-compiler in mui-lab (#42880)

mui · Jul 8, 2024 · 043bb21 · 043bb21
1 parent edb100c
commit 043bb21
Show file tree

Hide file tree

Showing 14 changed files with 105 additions and 52 deletions.
diff --git a/packages/mui-lab/src/Alert/Alert.js b/packages/mui-lab/src/Alert/Alert.js
@@ -4,10 +4,7 @@ import Alert from '@mui/material/Alert';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedAlert(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedAlert(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedAlert(props, ref) {
+  warn();
 
   return <Alert ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/AlertTitle/AlertTitle.js b/packages/mui-lab/src/AlertTitle/AlertTitle.js
@@ -4,10 +4,7 @@ import AlertTitle from '@mui/material/AlertTitle';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedAlertTitle(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedAlertTitle(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedAlertTitle(props, ref) {
+  warn();
 
   return <AlertTitle ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/Autocomplete/Autocomplete.js b/packages/mui-lab/src/Autocomplete/Autocomplete.js
@@ -4,10 +4,7 @@ import Autocomplete from '@mui/material/Autocomplete';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedAutocomplete(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedAutocomplete(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedAutocomplete(props, ref) {
+  warn();
 
   return <Autocomplete ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/AvatarGroup/AvatarGroup.js b/packages/mui-lab/src/AvatarGroup/AvatarGroup.js
@@ -4,10 +4,7 @@ import AvatarGroup from '@mui/material/AvatarGroup';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedAvatarGroup(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedAvatarGroup(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedAvatarGroup(props, ref) {
+  warn();
 
   return <AvatarGroup ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/Pagination/Pagination.js b/packages/mui-lab/src/Pagination/Pagination.js
@@ -4,10 +4,7 @@ import Pagination from '@mui/material/Pagination';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedPagination(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedPagination(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedPagination(props, ref) {
+  warn();
 
   return <Pagination ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/PaginationItem/PaginationItem.js b/packages/mui-lab/src/PaginationItem/PaginationItem.js
@@ -4,10 +4,7 @@ import PaginationItem from '@mui/material/PaginationItem';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedPaginationItem(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedPaginationItem(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedPaginationItem(props, ref) {
+  warn();
 
   return <PaginationItem ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/Rating/Rating.js b/packages/mui-lab/src/Rating/Rating.js
@@ -4,10 +4,7 @@ import Rating from '@mui/material/Rating';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedRating(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedRating(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedRating(props, ref) {
+  warn();
 
   return <Rating ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/Skeleton/Skeleton.js b/packages/mui-lab/src/Skeleton/Skeleton.js
@@ -4,10 +4,7 @@ import Skeleton from '@mui/material/Skeleton';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedSkeleton(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedSkeleton(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedSkeleton(props, ref) {
+  warn();
 
   return <Skeleton ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/SpeedDial/SpeedDial.js b/packages/mui-lab/src/SpeedDial/SpeedDial.js
@@ -4,10 +4,7 @@ import SpeedDial from '@mui/material/SpeedDial';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedSpeedDial(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedSpeedDial(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedSpeedDial(props, ref) {
+  warn();
 
   return <SpeedDial ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/SpeedDialAction/SpeedDialAction.js b/packages/mui-lab/src/SpeedDialAction/SpeedDialAction.js
@@ -4,10 +4,7 @@ import SpeedDialAction from '@mui/material/SpeedDialAction';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedSpeedDialAction(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedSpeedDialAction(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedSpeedDialAction(props, ref) {
+  warn();
 
   return <SpeedDialAction ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/SpeedDialIcon/SpeedDialIcon.js b/packages/mui-lab/src/SpeedDialIcon/SpeedDialIcon.js
@@ -4,10 +4,7 @@ import SpeedDialIcon from '@mui/material/SpeedDialIcon';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedSpeedDialIcon(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedSpeedDialIcon(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedSpeedDialIcon(props, ref) {
+  warn();
 
   return <SpeedDialIcon ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/TabContext/TabContext.test.js b/packages/mui-lab/src/TabContext/TabContext.test.js
@@ -11,6 +11,7 @@ describe('<TabContext />', () => {
   it('is null by default', () => {
     let value;
     function Tabs() {
+      // TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- value is used outside of component
       value = useTabContext();
       return null;
     }

diff --git a/packages/mui-lab/src/ToggleButton/ToggleButton.js b/packages/mui-lab/src/ToggleButton/ToggleButton.js
@@ -4,10 +4,7 @@ import ToggleButton from '@mui/material/ToggleButton';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedToggleButton(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedToggleButton(props, ref) {
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedToggleButton(props, ref) {
+  warn();
 
   return <ToggleButton ref={ref} {...props} />;
 });
diff --git a/packages/mui-lab/src/ToggleButtonGroup/ToggleButtonGroup.js b/packages/mui-lab/src/ToggleButtonGroup/ToggleButtonGroup.js
@@ -4,10 +4,7 @@ import ToggleButtonGroup from '@mui/material/ToggleButtonGroup';
 
 let warnedOnce = false;
 
-/**
- * @ignore - do not document.
- */
-export default React.forwardRef(function DeprecatedToggleButtonGroup(props, ref) {
+const warn = () => {
   if (!warnedOnce) {
     console.warn(
       [
@@ -20,6 +17,13 @@ export default React.forwardRef(function DeprecatedToggleButtonGroup(props, ref)
 
     warnedOnce = true;
   }
+};
+
+/**
+ * @ignore - do not document.
+ */
+export default React.forwardRef(function DeprecatedToggleButtonGroup(props, ref) {
+  warn();
 
   return <ToggleButtonGroup ref={ref} {...props} />;
 });