Revert usePatternsState to return an array instead of an object (#54582)

WordPress · Sep 19, 2023 · 23bb930 · 23bb930
1 parent a2e4ac3
commit 23bb930
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 11 deletions.
diff --git a/packages/block-editor/src/components/inserter/block-patterns-explorer/patterns-list.js b/packages/block-editor/src/components/inserter/block-patterns-explorer/patterns-list.js
@@ -74,7 +74,7 @@ function PatternList( {
 	const [ destinationRootClientId, onInsertBlocks ] = useInsertionPoint( {
 		shouldFocusBlock: true,
 	} );
-	const { patterns: allPatterns, onClickPattern } = usePatternsState(
+	const [ patterns, , onClickPattern ] = usePatternsState(
 		onInsertBlocks,
 		destinationRootClientId
 	);
@@ -88,7 +88,7 @@ function PatternList( {
 	);
 
 	const filteredBlockPatterns = useMemo( () => {
-		const filteredPatterns = allPatterns.filter( ( pattern ) => {
+		const filteredPatterns = patterns.filter( ( pattern ) => {
 			if (
 				isPatternFiltered(
 					pattern,
@@ -127,7 +127,7 @@ function PatternList( {
 	}, [
 		searchValue,
 		patternSourceFilter,
-		allPatterns,
+		patterns,
 		selectedCategory,
 		registeredPatternCategories,
 		patternSyncFilter,

diff --git a/packages/block-editor/src/components/inserter/block-patterns-tab.js b/packages/block-editor/src/components/inserter/block-patterns-tab.js
@@ -74,20 +74,20 @@ export function isPatternFiltered( pattern, sourceFilter, syncFilter ) {
 }
 
 export function usePatternsCategories( rootClientId, sourceFilter = 'all' ) {
-	const { patterns: allPatterns, allCategories } = usePatternsState(
+	const [ patterns, allCategories ] = usePatternsState(
 		undefined,
 		rootClientId
 	);
 
 	const filteredPatterns = useMemo(
 		() =>
 			sourceFilter === 'all'
-				? allPatterns
-				: allPatterns.filter(
+				? patterns
+				: patterns.filter(
 						( pattern ) =>
 							! isPatternFiltered( pattern, sourceFilter )
 				  ),
-		[ sourceFilter, allPatterns ]
+		[ sourceFilter, patterns ]
 	);
 
 	const hasRegisteredCategory = useCallback(
@@ -192,7 +192,7 @@ export function BlockPatternsCategoryPanel( {
 	showTitlesAsTooltip,
 	patternFilter,
 } ) {
-	const { patterns: allPatterns, onClickPattern } = usePatternsState(
+	const [ allPatterns, , onClickPattern ] = usePatternsState(
 		onInsert,
 		rootClientId
 	);

diff --git a/packages/block-editor/src/components/inserter/hooks/use-patterns-state.js b/packages/block-editor/src/components/inserter/hooks/use-patterns-state.js
@@ -79,7 +79,7 @@ const usePatternsState = ( onInsert, rootClientId ) => {
 		[ createSuccessNotice, onInsert ]
 	);
 
-	return { patterns, allCategories, onClickPattern };
+	return [ patterns, allCategories, onClickPattern ];
 };
 
 export default usePatternsState;
diff --git a/packages/block-editor/src/components/inserter/quick-inserter.js b/packages/block-editor/src/components/inserter/quick-inserter.js
@@ -46,7 +46,7 @@ export default function QuickInserter( {
 		onInsertBlocks
 	);
 
-	const { patterns } = usePatternsState(
+	const [ patterns ] = usePatternsState(
 		onInsertBlocks,
 		destinationRootClientId
 	);

diff --git a/packages/block-editor/src/components/inserter/search-results.js b/packages/block-editor/src/components/inserter/search-results.js
@@ -81,7 +81,7 @@ function InserterSearchResults( {
 		blockTypeCollections,
 		onSelectBlockType,
 	] = useBlockTypesState( destinationRootClientId, onInsertBlocks );
-	const { patterns, onClickPattern } = usePatternsState(
+	const [ patterns, , onClickPattern ] = usePatternsState(
 		onInsertBlocks,
 		destinationRootClientId
 	);