fix metadata payload structure for update request

Author: priya-tik

src/configs/tableConfigs/usersTableConfig.ts

@@ -1,3 +1,4 @@
+import { User } from "../../slices/userSlice";
 import { TableConfig } from "./aclsTableConfig";
 
 /**

src/slices/userInfoSlice.ts

@@ -1,5 +1,5 @@
 import { PayloadAction, SerializedError, createSlice } from '@reduxjs/toolkit'
-import axios from 'axios';
+import axios, { AxiosError } from 'axios';
 import { addNotification } from './notificationSlice';
 import { createAppAsyncThunk } from '../createAsyncThunkWithTypes';
 
@@ -73,26 +73,31 @@ const initialState: UserInfoState = {
 	},
 };
 
-export const fetchUserInfo = createAppAsyncThunk('UserInfo/fetchUserInfo', async (_, { dispatch }) => {
-	// Just make the async request here, and return the response.
-	// This will automatically dispatch a `pending` action first,
-	// and then `fulfilled` or `rejected` actions based on the promise.
-	const res = await axios.get("/info/me.json")
-		.then((response) => {
-			return response.data;
-		})
-		.catch((response) => {
-			console.error(response);
-			dispatch(addNotification({type: "error", key: "USER_NOT_SAVED"}));
-		});
-
-	// Redirect to login if not in ROLE_ADMIN_UI
-	if (!(res.roles.includes('ROLE_ADMIN') || res.roles.includes('ROLE_ADMIN_UI'))) {
-		window.location.href = "/login.html";
+export const fetchUserInfo = createAppAsyncThunk(
+	'UserInfo/fetchUserInfo',
+	async (_, { dispatch }) => {
+	  try {
+		const response = await axios.get("/info/me.json");
+		const res = response.data;
+		if (!res || !(res.roles?.includes('ROLE_ADMIN') || res.roles?.includes('ROLE_ADMIN_UI'))) {
+		  window.location.href = "/login.html";
+		}
+		return res;
+	  } catch (err: unknown) {
+		const error = err as AxiosError;
+		// eslint-disable-next-line no-trailing-spaces
+		console.error(error);  
+		const status = error?.response?.status;
+		if (status === 401 || status === 403) {
+		  window.location.href = "/login.html";
+		// eslint-disable-next-line no-trailing-spaces
+		}  
+		dispatch(addNotification({ type: "error", key: "USER_NOT_SAVED" }));
+		throw error;
+	  }
 	}
-
-	return res;
-});
+  // eslint-disable-next-line no-trailing-spaces
+  );  
 
 export const fetchOcVersion = createAppAsyncThunk('UserInfo/fetchOcVersion', async () => {
 	const res = await axios.get("/sysinfo/bundles/version?prefix=opencast");

src/slices/userSlice.ts

@@ -69,6 +69,7 @@ export const fetchUsers = createAppAsyncThunk('users/fetchUsers', async (_, { ge
 	// This will automatically dispatch a `pending` action first,
 	// and then `fulfilled` or `rejected` actions based on the promise.
 	const res = await axios.get("/admin-ng/users/users.json", { params: params });
+	console.log("this is the response", res);
 	return res.data;
 });
 

src/utils/resourceUtils.ts

@@ -166,37 +166,45 @@ export const transformMetadataFields = (metadata: MetadataField[]) => {
 };
 
 // transform metadata catalog for update via post request
-export const transformMetadataForUpdate = (catalog: MetadataCatalog, values: { [key: string]: MetadataCatalog["fields"][0]["value"] }) => {
+export const transformMetadataForUpdate = (
+	catalog: MetadataCatalog,
+	values: { [key: string]: MetadataCatalog["fields"][0]["value"] }
+  ) => {
 	let fields: MetadataCatalog["fields"] = [];
-	let updatedFields: MetadataCatalog["fields"] = [];
+	let updatedFields: { id: string; value: unknown }[] = [];
 
 	catalog.fields.forEach((field) => {
-		if (field.value !== values[field.id]) {
-			let updatedField = {
-				...field,
-				value: values[field.id],
-			};
-			updatedFields.push(updatedField);
-			fields.push(updatedField);
-		} else {
-			fields.push({ ...field });
-		}
+	  const newValue = values[field.id];
+
+	  // update UI state with full field (optional)
+	  const fullField = { ...field, value: newValue };
+	  fields.push(fullField);
+
+	  // only include minimal clean data for backend if value changed
+	  if (field.value !== newValue) {
+		updatedFields.push({
+		  id: field.id,
+		  value: newValue,
+		});
+	  }
 	});
+
 	let data = new URLSearchParams();
 	data.append(
-		"metadata",
-		JSON.stringify([
-			{
-				flavor: catalog.flavor,
-				title: catalog.title,
-				fields: updatedFields,
-			},
-		])
+	  "metadata",
+	  JSON.stringify([
+		{
+		  flavor: catalog.flavor,
+		  title: catalog.title,
+		  fields: updatedFields,
+		},
+	  ])
 	);
+
 	const headers = getHttpHeaders();
 
 	return { fields, data, headers };
-};
+  };
 
 // Prepare metadata for post of new events or series
 export const prepareMetadataFieldsForPost = (