@webitel/ui-datalist 1.1.70 → 1.1.72

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@webitel/ui-datalist",
-	"version": "1.1.70",
+	"version": "1.1.72",
 	"description": "Toolkit for building data lists in webitel ui system",
 	"scripts": {
 		"build:types": "vue-tsc -b tsconfig.build.json",

package/src/modules/card/types/CardValidationFields.types.ts

@@ -1,5 +1,12 @@
-import type { RegleSchemaFieldStatus } from '@regle/schemas';
+import type { InferRegleSchemaStatusType } from '@regle/schemas';
 
+/**
+ * Validation statuses of a card entity, as handed to the form tabs.
+ *
+ * Mirrors what regle actually builds: an array field resolves to a collection
+ * status (`$each`), a nested object to a status with `$fields`. Typing every
+ * field as a plain field status hides both.
+ */
 export type CardValidationFields<T> = {
-	[K in keyof T]: RegleSchemaFieldStatus<T[K]>;
+	[K in keyof T]: InferRegleSchemaStatusType<T[K]>;
 };

package/src/modules/headers/__tests__/createTableHeadersStore.spec.ts

@@ -90,6 +90,34 @@ describe('tableHeadersStoreBody', () => {
 		]);
 	});
 
+	it('asks for a field once when several columns render from it', () => {
+		const store = tableHeadersStoreBody({
+			rawHeaders: [
+				{
+					field: 'grantee',
+					show: true,
+					sort: null,
+				},
+				{
+					field: 'granted',
+					show: true,
+					sort: null,
+				},
+				{
+					field: 'granted',
+					show: true,
+					sort: null,
+				},
+			] as DatalistTableHeader[],
+			id,
+		});
+
+		expect(store.fields.value).toEqual([
+			'grantee',
+			'granted',
+		]);
+	});
+
 	it('has no sort by default', () => {
 		expect(createStore().sort.value).toBeNull();
 	});

package/src/modules/headers/createTableHeadersStore.ts

@@ -45,8 +45,11 @@ export const tableHeadersStoreBody = ({
 		return headers.value.filter((header) => header.show);
 	});
 
+	/* several columns may render from one api field, ask for it once */
 	const fields = computed(() => {
-		return shownHeaders.value.map((header) => header.field);
+		return [
+			...new Set(shownHeaders.value.map((header) => header.field)),
+		];
 	});
 
 	const sort = computed(() => {

package/src/modules/permissions-page/scripts/PermissionsApiModule.ts

@@ -13,8 +13,25 @@ import type {
 export const PermissionsApiModule = (
 	rawApiModule: RawPermissionsApiModule,
 ): TableApiModule<PermissionEntity> => {
+	/**
+	 * The table store always asks for `id` (every registry needs it to open or
+	 * delete a row), but an access rule has no such attribute and the endpoint
+	 * answers `400 app.search.fields.invalid` — a rule is addressed by its
+	 * grantee. Rows are patched through the parent id, so nothing here needs it.
+	 */
+	const getList = ({
+		fields,
+		...params
+	}: {
+		fields?: string[];
+	} & Record<string, unknown>) =>
+		rawApiModule.getPermissionsList?.({
+			...params,
+			fields: fields?.filter((field) => field !== 'id'),
+		});
+
 	return {
-		getList: rawApiModule.getPermissionsList,
+		getList,
 		patch: rawApiModule.patchPermissions,
 	} as unknown as TableApiModule<PermissionEntity>;
 };

package/src/modules/permissions-page/stores/createPermissionsStore.ts

@@ -146,6 +146,9 @@ export const createPermissionsStore = (
 	},
 ) => {
 	const normalizedConfig: useTableStoreConfig<PermissionEntity> = {
+		/* a fixed 4-column tab: nothing worth persisting, and a `fields` list
+		   saved by an older version would keep being sent to the api */
+		disablePersistence: true,
 		...config,
 		apiModule: PermissionsApiModule(config.apiModule),
 		headers: config.headers || headers,

package/src/modules/table/createTableStore.store.ts

@@ -103,12 +103,16 @@ export const tableStoreBody = <Entity extends Identifiable>(
 		selected.value = value;
 	};
 
+	// Always request `id` (Vuex REQUIRED_FIELDS default) — needed for select/delete/open
 	const getLoadDataParams = () => ({
 		...filtersManager.value.getAllValues(),
 		page: page.value,
 		size: size.value,
 		sort: sort.value,
-		fields: fields.value,
+		fields: [
+			'id',
+			...fields.value.filter((field) => field !== 'id'),
+		],
 		parentId: parentId.value,
 	});