Merge pull request #170 from studiopress/try/move-blocks-to-their-own…

…-files

Try moving blocks to their own modular directories

wp-modules/editor/js/src/utils/registerPatternBlock.ts → wp-modules/editor/js/src/blocks/pattern-block/registerPatternBlock.ts

@@ -1,5 +1,5 @@
 import { __ } from '@wordpress/i18n';
-import PatternEdit from '../components/PatternEdit';
+import PatternEdit from './PatternEdit';
 
 export default function registerPatternBlock(
 	settings: Record< string, unknown >,

wp-modules/editor/js/src/components/PatternEdit/index.test.tsx → wp-modules/editor/js/src/blocks/pattern-block/test/PatternEdit.tsx

@@ -1,7 +1,7 @@
 import { render } from '@testing-library/react';
-import PatternEdit from './';
+import PatternEdit from '../PatternEdit';
 
-jest.mock( '../../globals', () => {
+jest.mock( '../../../globals', () => {
 	return {
 		patternManager: {
 			patterns: {
@@ -18,11 +18,11 @@ jest.mock( '../../globals', () => {
 	};
 } );
 
-jest.mock( '../../hooks/useSavedPostData', () => {
+jest.mock( '../../../hooks/useSavedPostData', () => {
 	return () => ( {} );
 } );
 
-jest.mock( '../../../../../app/js/src/components/PatternPreview', () => {
+jest.mock( '../../../../../../app/js/src/components/PatternPreview', () => {
 	return () => null;
 } );
 

wp-modules/editor/js/src/index.ts

@@ -7,7 +7,7 @@ import PatternManagerMetaControls from './components/PatternManagerMetaControls'
 import changeWords from './utils/changeWords';
 import preventTransform from './utils/preventTransform';
 import receiveActiveTheme from './utils/receiveActiveTheme';
-import registerPatternBlock from './utils/registerPatternBlock';
+import registerPatternBlock from './blocks/pattern-block/registerPatternBlock';
 
 registerPlugin( 'pattern-manager-postmeta-for-patterns', {
 	icon: null,

wp-modules/editor/js/src/utils/filterOutPatterns.ts

@@ -1,5 +1,5 @@
 import { parse } from '@wordpress/blocks';
-import hasPmPatternBlock from './hasPmPatternBlock';
+import hasBlock from './hasBlock';
 import type { Pattern, Patterns } from '../types';
 
 /**
@@ -14,7 +14,8 @@ export default function filterOutPatterns(
 		( accumulator, [ key, pattern ] ) => {
 			return {
 				...accumulator,
-				...( ! hasPmPatternBlock(
+				...( ! hasBlock(
+					'core/pattern',
 					parse( pattern.content ),
 					patternName
 				) && pattern.name !== patternName

wp-modules/editor/js/src/utils/hasPmPatternBlock.ts → wp-modules/editor/js/src/utils/hasBlock.ts

@@ -1,14 +1,15 @@
 import type { Block, Pattern } from '../types';
 
-export default function hasPmPatternBlock(
+export default function hasBlock(
+	blockName: String,
 	blocks: Block[],
 	patternName: Pattern[ 'name' ]
 ) {
 	return blocks.some( ( block ) => {
 		return (
-			( block.name === 'core/pattern' &&
+			( block.name === blockName &&
 				block.attributes?.slug === patternName ) ||
-			hasPmPatternBlock( block?.innerBlocks ?? [], patternName )
+			hasBlock( blockName, block?.innerBlocks ?? [], patternName )
 		);
 	} );
 }

wp-modules/editor/js/src/utils/test/hasPmPatternBlock.ts → wp-modules/editor/js/src/utils/test/hasBlock.ts

@@ -1,6 +1,6 @@
-import hasPmPatternBlock from '../hasPmPatternBlock';
+import hasBlock from '../hasBlock';
 
-describe( 'hasPmPatternBlock', () => {
+describe( 'hasBlock', () => {
 	it.each( [
 		[ [ { name: '', attributes: {} } ], '', false ],
 		[ [ { name: '', attributes: {} } ], 'example-slug', false ],
@@ -75,7 +75,7 @@ describe( 'hasPmPatternBlock', () => {
 	] )(
 		'should get whether there is a Pattern Block',
 		( blocks, patternSlug, expected ) => {
-			expect( hasPmPatternBlock( blocks, patternSlug ) ).toEqual(
+			expect( hasBlock( 'core/pattern', blocks, patternSlug ) ).toEqual(
 				expected
 			);
 		}