@wordpress/core-data 7.41.2-next.v.202603161435.0 → 7.43.0

package/src/queried-data/selectors.js

@@ -32,15 +32,23 @@ const queriedItemsCacheByState = new WeakMap();
  * @return {?Array} Query items.
  */
 function getQueriedItemsUncached( state, query ) {
-	const { stableKey, page, perPage, include, fields, context } =
-		getQueryParts( query );
+	const {
+		stableKey,
+		page,
+		perPage,
+		offset: queryOffset,
+		include,
+		fields,
+		context,
+	} = getQueryParts( query );
 
 	const itemIds = state.queries?.[ context ]?.[ stableKey ]?.itemIds;
 	if ( ! itemIds ) {
 		return null;
 	}
 
-	const startOffset = perPage === -1 ? 0 : ( page - 1 ) * perPage;
+	const startOffset =
+		perPage === -1 ? 0 : queryOffset ?? ( page - 1 ) * perPage;
 	const endOffset =
 		perPage === -1
 			? itemIds.length

package/src/queried-data/test/get-query-parts.js

@@ -11,6 +11,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 2,
 			perPage: 2,
+			offset: null,
 			stableKey: '',
 			fields: null,
 			include: null,
@@ -28,6 +29,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 1,
 			perPage: 10,
+			offset: null,
 			stableKey: 'include=1',
 			fields: null,
 			include: [ 1 ],
@@ -43,6 +45,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 1,
 			perPage: 10,
+			offset: null,
 			stableKey: '%3F=%26&b=2',
 			fields: null,
 			include: null,
@@ -56,6 +59,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 1,
 			perPage: 10,
+			offset: null,
 			stableKey: 'a%5B0%5D=1&a%5B1%5D=2',
 			fields: null,
 			include: null,
@@ -71,6 +75,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 1,
 			perPage: 10,
+			offset: null,
 			stableKey: 'b=2',
 			fields: null,
 			include: null,
@@ -84,6 +89,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 1,
 			perPage: -1,
+			offset: null,
 			stableKey: 'b=2',
 			fields: null,
 			include: null,
@@ -97,6 +103,7 @@ describe( 'getQueryParts', () => {
 			context: 'default',
 			page: 1,
 			perPage: 10,
+			offset: null,
 			stableKey: '_fields=id%2Ctitle',
 			fields: [ 'id', 'title' ],
 			include: null,
@@ -109,10 +116,37 @@ describe( 'getQueryParts', () => {
 		expect( parts ).toEqual( {
 			page: 1,
 			perPage: 10,
+			offset: null,
 			stableKey: '',
 			include: null,
 			fields: null,
 			context: 'view',
 		} );
 	} );
+
+	it( 'extracts offset and excludes it from stableKey', () => {
+		const parts = getQueryParts( {
+			per_page: 50,
+			offset: 100,
+		} );
+
+		expect( parts ).toEqual( {
+			context: 'default',
+			page: 1,
+			perPage: 50,
+			offset: 100,
+			stableKey: '',
+			fields: null,
+			include: null,
+		} );
+	} );
+
+	it( 'ignores non-numeric offset values', () => {
+		const parts = getQueryParts( {
+			per_page: 10,
+			offset: 'abc',
+		} );
+
+		expect( parts.offset ).toBeNull();
+	} );
 } );

package/src/queried-data/test/reducer.js

@@ -11,7 +11,10 @@ import { removeItems } from '../actions';
 
 describe( 'getMergedItemIds', () => {
 	it( 'should receive a page', () => {
-		const result = getMergedItemIds( [], [ 4, 5, 6 ], 2, 3 );
+		const result = getMergedItemIds( [], [ 4, 5, 6 ], {
+			page: 2,
+			perPage: 3,
+		} );
 
 		expect( result ).toEqual( [
 			undefined,
@@ -32,46 +35,68 @@ describe( 'getMergedItemIds', () => {
 			5,
 			6,
 		] );
-		const result = getMergedItemIds( original, [ 1, 2, 3 ], 1, 3 );
+		const result = getMergedItemIds( original, [ 1, 2, 3 ], {
+			page: 1,
+			perPage: 3,
+		} );
 
 		expect( result ).toEqual( [ 1, 2, 3, 4, 5, 6 ] );
 	} );
 
 	it( 'should replace with new page', () => {
 		const original = deepFreeze( [ 1, 2, 3, 4, 5, 6 ] );
-		const result = getMergedItemIds( original, [ 'replaced', 5, 6 ], 2, 3 );
+		const result = getMergedItemIds( original, [ 'replaced', 5, 6 ], {
+			page: 2,
+			perPage: 3,
+		} );
 
 		expect( result ).toEqual( [ 1, 2, 3, 'replaced', 5, 6 ] );
 	} );
 
 	it( 'should append a new partial page', () => {
 		const original = deepFreeze( [ 1, 2, 3, 4, 5, 6 ] );
-		const result = getMergedItemIds( original, [ 7 ], 3, 3 );
+		const result = getMergedItemIds( original, [ 7 ], {
+			page: 3,
+			perPage: 3,
+		} );
 
 		expect( result ).toEqual( [ 1, 2, 3, 4, 5, 6, 7 ] );
 	} );
 
 	it( 'should return a copy of nextItemIds if it represents all ids (single id removed) (page=1 and perPage=-1)', () => {
 		const original = deepFreeze( [ 1, 2, 3 ] );
-		const result = getMergedItemIds( original, [ 1, 3 ], 1, -1 );
+		const result = getMergedItemIds( original, [ 1, 3 ], {
+			page: 1,
+			perPage: -1,
+		} );
 
 		expect( result ).toEqual( [ 1, 3 ] );
 	} );
 
 	it( 'should return a copy of nextItemIds if it represents all ids (single id removed and another one added) (page=1 and perPage=-1)', () => {
 		const original = deepFreeze( [ 1, 2, 3 ] );
-		const result = getMergedItemIds( original, [ 1, 3, 4 ], 1, -1 );
+		const result = getMergedItemIds( original, [ 1, 3, 4 ], {
+			page: 1,
+			perPage: -1,
+		} );
 
 		expect( result ).toEqual( [ 1, 3, 4 ] );
 	} );
+
 	it( 'should update a page properly if less items are provided than previously stored', () => {
 		let original = deepFreeze( [ 1, 2, 3 ] );
-		let result = getMergedItemIds( original, [ 1, 2 ], 1, 3 );
+		let result = getMergedItemIds( original, [ 1, 2 ], {
+			page: 1,
+			perPage: 3,
+		} );
 
 		expect( result ).toEqual( [ 1, 2 ] );
 
 		original = deepFreeze( [ 1, 2, 3, 4, 5, 6 ] );
-		result = getMergedItemIds( original, [ 9 ], 2, 2 );
+		result = getMergedItemIds( original, [ 9 ], {
+			page: 2,
+			perPage: 2,
+		} );
 
 		expect( result ).toEqual( [ 1, 2, 9, undefined, 5, 6 ] );
 	} );
@@ -197,6 +222,51 @@ describe( 'reducer', () => {
 		} );
 	} );
 
+	it( 'receives a sparse subset of items at given offsets', () => {
+		const original = deepFreeze( {
+			items: { default: {} },
+			queries: {},
+			itemIsComplete: { default: {} },
+		} );
+		const state = [
+			{
+				type: 'RECEIVE_ITEMS',
+				query: { offset: 1, per_page: 2 },
+				items: [
+					{ id: 2, name: 'def' },
+					{ id: 3, name: 'ghi' },
+				],
+			},
+			{
+				type: 'RECEIVE_ITEMS',
+				query: { offset: 4, per_page: 2 },
+				items: [
+					{ id: 5, name: 'mno' },
+					{ id: 6, name: 'pqr' },
+				],
+			},
+		].reduce( reducer, original );
+
+		expect( state ).toEqual( {
+			items: {
+				default: {
+					2: { id: 2, name: 'def' },
+					3: { id: 3, name: 'ghi' },
+					5: { id: 5, name: 'mno' },
+					6: { id: 6, name: 'pqr' },
+				},
+			},
+			itemIsComplete: {
+				default: { 2: true, 3: true, 5: true, 6: true },
+			},
+			queries: {
+				default: {
+					'': { itemIds: [ undefined, 2, 3, undefined, 5, 6 ] },
+				},
+			},
+		} );
+	} );
+
 	it( 'deletes an item', () => {
 		const kind = 'root';
 		const name = 'menu';

package/src/queried-data/test/selectors.js

@@ -2,6 +2,7 @@
  * Internal dependencies
  */
 import { getQueriedItems } from '../selectors';
+import { getMergedItemIds } from '../reducer';
 
 describe( 'getQueriedItems', () => {
 	it( 'should return null if requesting but no item IDs', () => {
@@ -264,4 +265,174 @@ describe( 'getQueriedItems', () => {
 		const result = getQueriedItems( state, { per_page: 3 } );
 		expect( result ).toBe( null );
 	} );
+
+	it( 'should return items for offset-based query on the last partial page', () => {
+		// Infinite scroll scenario: 103 total items, perPage=50, and the
+		// last batch starts at offset=100. The API returns 3 items (items
+		// 101-103). X-WP-Total is 103 (the global count). The selector
+		// should recognise this as a complete response since
+		// 103 - 100 = 3 expected items.
+		const state = {
+			items: {
+				default: {
+					101: { id: 101 },
+					102: { id: 102 },
+					103: { id: 103 },
+				},
+			},
+			itemIsComplete: {
+				default: { 101: true, 102: true, 103: true },
+			},
+			queries: {
+				default: {
+					'': {
+						itemIds: getMergedItemIds( [], [ 101, 102, 103 ], {
+							offset: 100,
+							perPage: 50,
+						} ),
+						meta: { totalItems: 103 },
+					},
+				},
+			},
+		};
+
+		const result = getQueriedItems( state, {
+			per_page: 50,
+			offset: 100,
+		} );
+		expect( result ).toEqual( [ { id: 101 }, { id: 102 }, { id: 103 } ] );
+	} );
+
+	it( 'should return null for offset-based query when items are still missing', () => {
+		// Offset=50, perPage=50, totalItems=200: the API should return
+		// 50 items for this batch but only 2 are stored so far.
+		const state = {
+			items: {
+				default: {
+					51: { id: 51 },
+					52: { id: 52 },
+				},
+			},
+			itemIsComplete: {
+				default: { 51: true, 52: true },
+			},
+			queries: {
+				default: {
+					'': {
+						itemIds: getMergedItemIds( [], [ 51, 52 ], {
+							offset: 50,
+							perPage: 50,
+						} ),
+						meta: { totalItems: 200 },
+					},
+				},
+			},
+		};
+
+		const result = getQueriedItems( state, {
+			per_page: 50,
+			offset: 50,
+		} );
+		expect( result ).toBe( null );
+	} );
+
+	it( 'should return null for offset query when items are still missing', () => {
+		// Query Block scenario: offset=3 with per_page=10. The effective
+		// total is totalItems - offset = 47. Only 5 items are stored, so
+		// the data is still incomplete.
+		const state = {
+			items: {
+				default: {
+					4: { id: 4 },
+					5: { id: 5 },
+					6: { id: 6 },
+					7: { id: 7 },
+					8: { id: 8 },
+				},
+			},
+			itemIsComplete: {
+				default: { 4: true, 5: true, 6: true, 7: true, 8: true },
+			},
+			queries: {
+				default: {
+					'': {
+						itemIds: getMergedItemIds( [], [ 4, 5, 6, 7, 8 ], {
+							offset: 3,
+							perPage: 10,
+						} ),
+						meta: { totalItems: 50 },
+					},
+				},
+			},
+		};
+
+		const result = getQueriedItems( state, {
+			per_page: 10,
+			offset: 3,
+		} );
+		expect( result ).toBe( null );
+	} );
+
+	it( 'should treat offset=0 the same as no offset', () => {
+		// The Query Block defaults to offset=0. Since
+		// effectiveTotal = totalItems - 0 = totalItems, this should
+		// behave identically to a query without offset.
+		const state = {
+			items: {
+				default: {
+					1: { id: 1 },
+					2: { id: 2 },
+				},
+			},
+			itemIsComplete: {
+				default: { 1: true, 2: true },
+			},
+			queries: {
+				default: {
+					'': {
+						itemIds: [ 1, 2 ],
+						meta: { totalItems: 5 },
+					},
+				},
+			},
+		};
+
+		// 2 items stored, but 5 total exist — should return null.
+		const result = getQueriedItems( state, {
+			per_page: 3,
+			offset: 0,
+		} );
+		expect( result ).toBe( null );
+	} );
+
+	it( 'should return empty array when offset equals totalItems', () => {
+		// Edge case: offset lands exactly at the end (e.g. 84 items,
+		// per_page=7, offset=84). The API returns 0 items and that is
+		// a complete response — effectiveTotal is 0.
+		const state = {
+			items: {
+				default: {},
+			},
+			itemIsComplete: {
+				default: {},
+			},
+			queries: {
+				default: {
+					'': {
+						itemIds: getMergedItemIds( [], [], {
+							offset: 84,
+							perPage: 7,
+						} ),
+						meta: { totalItems: 84 },
+					},
+				},
+			},
+		};
+
+		const result = getQueriedItems( state, {
+			per_page: 7,
+			offset: 84,
+		} );
+		expect( result ).toEqual( [] );
+	} );
 } );

package/src/reducer.js

@@ -16,6 +16,7 @@ import { createUndoManager } from '@wordpress/undo-manager';
 import { ifMatchingAction, replaceAction } from './utils';
 import { reducer as queriedDataReducer } from './queried-data';
 import { rootEntitiesConfig, DEFAULT_ENTITY_KEY } from './entities';
+import { ConnectionErrorCode } from './sync';
 
 /** @typedef {import('./types').AnyFunction} AnyFunction */
 
@@ -706,6 +707,35 @@ export function collaborationSupported( state = true, action ) {
 	switch ( action.type ) {
 		case 'SET_COLLABORATION_SUPPORTED':
 			return action.supported;
+
+		case 'SET_SYNC_CONNECTION_STATUS':
+			if (
+				ConnectionErrorCode.DOCUMENT_SIZE_LIMIT_EXCEEDED ===
+				action.status?.error?.code
+			) {
+				return false;
+			}
+
+			return state;
+	}
+	return state;
+}
+
+/**
+ * Reducer managing view configs, keyed by `kind/name`.
+ *
+ * @param {Object} state  Current state.
+ * @param {Object} action Dispatched action.
+ *
+ * @return {Object} Updated state.
+ */
+export function viewConfigs( state = {}, action ) {
+	switch ( action.type ) {
+		case 'RECEIVE_VIEW_CONFIG':
+			return {
+				...state,
+				[ `${ action.kind }/${ action.name }` ]: action.config,
+			};
 	}
 	return state;
 }
@@ -734,4 +764,5 @@ export default combineReducers( {
 	editorAssets,
 	syncConnectionStatuses,
 	collaborationSupported,
+	viewConfigs,
 } );

package/src/resolvers.js

@@ -991,11 +991,12 @@ export const getDefaultTemplateId =
 		const id = window?.__experimentalTemplateActivate
 			? template?.wp_id || template?.id
 			: template?.id;
-		// Endpoint may return an empty object if no template is found.
-		if ( id ) {
-			template.id = id;
-			registry.batch( () => {
-				dispatch.receiveDefaultTemplateId( query, id );
+
+		registry.batch( () => {
+			dispatch.receiveDefaultTemplateId( query, id || '' );
+			// Endpoint may return an empty object if no template is found.
+			if ( id ) {
+				template.id = id;
 				dispatch.receiveEntityRecords(
 					'postType',
 					template.type,
@@ -1007,8 +1008,8 @@ export const getDefaultTemplateId =
 					template.type,
 					id,
 				] );
-			} );
-		}
+			}
+		} );
 	};
 
 getDefaultTemplateId.shouldInvalidate = ( action ) => {
@@ -1320,3 +1321,18 @@ export const getEditorAssets =
 		} );
 		dispatch.receiveEditorAssets( assets );
 	};
+
+/**
+ * Requests view config for a given entity type from the REST API.
+ *
+ * @param {string} kind Entity kind.
+ * @param {string} name Entity name.
+ */
+export const getViewConfig =
+	( kind, name ) =>
+	async ( { dispatch } ) => {
+		const config = await apiFetch( {
+			path: addQueryArgs( '/wp/v2/view-config', { kind, name } ),
+		} );
+		dispatch.receiveViewConfig( kind, name, config );
+	};

package/src/selectors.ts

@@ -20,7 +20,6 @@ import { DEFAULT_ENTITY_KEY } from './entities';
 import { getUndoManager } from './private-selectors';
 import {
 	getNormalizedCommaSeparable,
-	isRawAttribute,
 	setNestedValue,
 	isNumericID,
 	getUserPermissionCacheKey,
@@ -55,6 +54,7 @@ export interface State {
 	editorAssets: Record< string, any > | null;
 	syncConnectionStatuses?: Record< string, ConnectionStatus >;
 	collaborationSupported: boolean;
+	viewConfigs: Record< string, Record< string, any > >;
 }
 
 type EntityRecordKey = string | number;
@@ -523,25 +523,26 @@ export const getRawEntityRecord = createSelector(
 			name,
 			key
 		);
-		return (
-			record &&
-			Object.keys( record ).reduce( ( accumulator, _key ) => {
-				if (
-					isRawAttribute( getEntityConfig( state, kind, name ), _key )
-				) {
-					// Because edits are the "raw" attribute values,
-					// we return those from record selectors to make rendering,
-					// comparisons, and joins with edits easier.
-					accumulator[ _key ] =
-						record[ _key ]?.raw !== undefined
-							? record[ _key ]?.raw
-							: record[ _key ];
-				} else {
-					accumulator[ _key ] = record[ _key ];
+		const config = getEntityConfig( state, kind, name );
+		if ( ! record || ! config?.rawAttributes?.length ) {
+			return record;
+		}
+
+		// Because edits are the "raw" attribute values,
+		// we return those from record selectors to make rendering,
+		// comparisons, and joins with edits easier.
+		return Object.fromEntries(
+			Object.keys( record ).map( ( _key ) => {
+				if ( config.rawAttributes.includes( _key ) ) {
+					const rawValue = record[ _key ]?.raw;
+					return [
+						_key,
+						rawValue !== undefined ? rawValue : record[ _key ],
+					];
 				}
-				return accumulator;
-			}, {} as any )
-		);
+				return [ _key, record[ _key ] ];
+			} )
+		) as EntityRecord;
 	},
 	(
 		state: State,

package/src/sync.ts

@@ -12,6 +12,7 @@ import {
 import { unlock } from './lock-unlock';
 
 const {
+	ConnectionErrorCode,
 	createSyncManager,
 	Delta,
 	CRDT_DOC_META_PERSISTENCE_KEY,
@@ -22,6 +23,7 @@ const {
 } = unlock( syncPrivateApis );
 
 export {
+	ConnectionErrorCode,
 	Delta,
 	CRDT_DOC_META_PERSISTENCE_KEY,
 	CRDT_RECORD_MAP_KEY,