Data: Rename useSelect internals to fix React Compiler violations (#6…

…6807) Co-authored-by: tyxla <[email protected]> Co-authored-by: Mamaduka <[email protected]>
WordPress · Nov 7, 2024 · 69f71e1 · 69f71e1 · github-actions · Nov 7, 2024
1 parent b0d2041
commit 69f71e1
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/packages/data/src/components/use-select/index.js b/packages/data/src/components/use-select/index.js
@@ -209,11 +209,11 @@ function Store( registry, suspense ) {
 	};
 }
 
-function useStaticSelect( storeName ) {
+function _useStaticSelect( storeName ) {
 	return useRegistry().select( storeName );
 }
 
-function useMappingSelect( suspense, mapSelect, deps ) {
+function _useMappingSelect( suspense, mapSelect, deps ) {
 	const registry = useRegistry();
 	const isAsync = useAsyncMode();
 	const store = useMemo(
@@ -308,13 +308,11 @@ export default function useSelect( mapSelect, deps ) {
 		);
 	}
 
-	/* eslint-disable react-hooks/rules-of-hooks */
 	// `staticSelectMode` is not allowed to change during the hook instance's,
 	// lifetime, so the rules of hooks are not really violated.
 	return staticSelectMode
-		? useStaticSelect( mapSelect )
-		: useMappingSelect( false, mapSelect, deps );
-	/* eslint-enable react-hooks/rules-of-hooks */
+		? _useStaticSelect( mapSelect )
+		: _useMappingSelect( false, mapSelect, deps );
 }
 
 /**
@@ -337,5 +335,5 @@ export default function useSelect( mapSelect, deps ) {
  * @return {ReturnType<T>} Data object returned by the `mapSelect` function.
  */
 export function useSuspenseSelect( mapSelect, deps ) {
-	return useMappingSelect( true, mapSelect, deps );
+	return _useMappingSelect( true, mapSelect, deps );
 }