@wordpress/core-data 7.41.0 → 7.41.2-next.v.202603102151.0

package/src/utils/test/crdt-blocks.ts

@@ -31,6 +31,13 @@ jest.mock( '@wordpress/blocks', () => ( {
 			name: 'core/paragraph',
 			attributes: { content: { type: 'rich-text' } },
 		},
+		{
+			name: 'core/image',
+			attributes: {
+				blob: { type: 'string', role: 'local' },
+				url: { type: 'string' },
+			},
+		},
 	],
 } ) );
 
@@ -175,32 +182,30 @@ describe( 'crdt-blocks', () => {
 			expect( innerBlock.get( 'name' ) ).toBe( 'core/paragraph' );
 		} );
 
-		it( 'skips gallery blocks with unuploaded images (blob attributes)', () => {
-			const galleryWithBlobs: Block[] = [
+		it( 'strips local attributes when syncing blocks', () => {
+			const imageWithBlob: Block[] = [
 				{
-					name: 'core/gallery',
-					attributes: {},
-					innerBlocks: [
-						{
-							name: 'core/image',
-							attributes: {
-								url: 'http://example.com/image.jpg',
-								blob: 'blob:...',
-							},
-							innerBlocks: [],
-						},
-					],
+					name: 'core/image',
+					attributes: {
+						url: 'http://example.com/image.jpg',
+						blob: 'blob:...',
+					},
+					innerBlocks: [],
 				},
 			];
 
-			mergeCrdtBlocks( yblocks, galleryWithBlobs, null );
+			mergeCrdtBlocks( yblocks, imageWithBlob, null );
 
-			// Gallery block should not be synced because it has blob attributes
-			expect( yblocks.length ).toBe( 0 );
+			expect( yblocks.length ).toBe( 1 );
+			const block = yblocks.get( 0 );
+			expect( block.get( 'name' ) ).toBe( 'core/image' );
+			const attrs = block.get( 'attributes' ) as YBlockAttributes;
+			expect( attrs.get( 'url' ) ).toBe( 'http://example.com/image.jpg' );
+			expect( attrs.has( 'blob' ) ).toBe( false );
 		} );
 
-		it( 'syncs gallery blocks without blob attributes', () => {
-			const galleryWithoutBlobs: Block[] = [
+		it( 'strips local attributes from inner blocks', () => {
+			const galleryWithBlobs: Block[] = [
 				{
 					name: 'core/gallery',
 					attributes: {},
@@ -209,6 +214,7 @@ describe( 'crdt-blocks', () => {
 							name: 'core/image',
 							attributes: {
 								url: 'http://example.com/image.jpg',
+								blob: 'blob:...',
 							},
 							innerBlocks: [],
 						},
@@ -216,11 +222,17 @@ describe( 'crdt-blocks', () => {
 				},
 			];
 
-			mergeCrdtBlocks( yblocks, galleryWithoutBlobs, null );
+			mergeCrdtBlocks( yblocks, galleryWithBlobs, null );
 
 			expect( yblocks.length ).toBe( 1 );
-			const block = yblocks.get( 0 );
-			expect( block.get( 'name' ) ).toBe( 'core/gallery' );
+			const gallery = yblocks.get( 0 );
+			expect( gallery.get( 'name' ) ).toBe( 'core/gallery' );
+			const innerBlocks = gallery.get( 'innerBlocks' ) as YBlocks;
+			expect( innerBlocks.length ).toBe( 1 );
+			const image = innerBlocks.get( 0 );
+			const attrs = image.get( 'attributes' ) as YBlockAttributes;
+			expect( attrs.get( 'url' ) ).toBe( 'http://example.com/image.jpg' );
+			expect( attrs.has( 'blob' ) ).toBe( false );
 		} );
 
 		it( 'handles block reordering', () => {
@@ -522,7 +534,7 @@ describe( 'crdt-blocks', () => {
 			expect( content2.toString() ).toBe( 'Freeform content' );
 		} );
 
-		it( 'syncs nested blocks with blob attributes', () => {
+		it( 'strips local attributes from deeply nested blocks', () => {
 			const nestedGallery: Block[] = [
 				{
 					name: 'core/group',
@@ -554,7 +566,13 @@ describe( 'crdt-blocks', () => {
 
 			const innerBlocks = groupBlock.get( 'innerBlocks' ) as YBlocks;
 			expect( innerBlocks.length ).toBe( 1 );
-			expect( innerBlocks.get( 0 ).get( 'name' ) ).toBe( 'core/gallery' );
+			const gallery = innerBlocks.get( 0 );
+			const galleryInner = gallery.get( 'innerBlocks' ) as YBlocks;
+			expect( galleryInner.length ).toBe( 1 );
+			const image = galleryInner.get( 0 );
+			const attrs = image.get( 'attributes' ) as YBlockAttributes;
+			expect( attrs.get( 'url' ) ).toBe( 'http://example.com/image.jpg' );
+			expect( attrs.has( 'blob' ) ).toBe( false );
 		} );
 
 		it( 'handles complex block reordering', () => {

package/src/utils/user-permissions.js

@@ -7,18 +7,17 @@ export const ALLOWED_RESOURCE_ACTIONS = [
 
 export function getUserPermissionsFromAllowHeader( allowedMethods ) {
 	const permissions = {};
-	if ( ! allowedMethods ) {
-		return permissions;
-	}
-
 	const methods = {
 		create: 'POST',
 		read: 'GET',
 		update: 'PUT',
 		delete: 'DELETE',
 	};
+
 	for ( const [ actionName, methodName ] of Object.entries( methods ) ) {
-		permissions[ actionName ] = allowedMethods.includes( methodName );
+		permissions[ actionName ] = allowedMethods
+			? allowedMethods.includes( methodName )
+			: false;
 	}
 
 	return permissions;