refac

backend/open_webui/routers/users.py

@@ -508,7 +508,7 @@ async def update_user_settings_by_session_user(
             detail=ERROR_MESSAGES.ACCESS_PROHIBITED,
         )
 
-    updated_user_settings = form_data.model_dump()
+    updated_user_settings = form_data.model_dump(exclude_unset=True)
     ui_settings = updated_user_settings.get('ui')
     if (
         user.role != 'admin'

src/routes/(app)/+layout.svelte

@@ -90,19 +90,7 @@
 	};
 
 	const setUserSettings = async (cb?: () => Promise<void>) => {
-		let userSettings = await getUserSettings(localStorage.token).catch((error) => {
+		const userSettings = await getUserSettings(localStorage.token);
-			console.error(error);
-			return null;
-		});
-
-		if (!userSettings) {
-			try {
-				userSettings = JSON.parse(localStorage.getItem('settings') ?? '{}');
-			} catch (e: unknown) {
-				console.error('Failed to parse settings from localStorage', e);
-				userSettings = {};
-			}
-		}
 
 		if (userSettings?.ui) {
 			settings.set(userSettings.ui);
@@ -254,14 +242,20 @@
 		}
 
 		clearChatInputStorage();
-		await Promise.all([
+		try {
-			checkLocalDBChats(),
+			await Promise.all([
-			setBanners().catch((e) => console.error('Failed to load banners:', e)),
+				checkLocalDBChats(),
-			setTools().catch((e) => console.error('Failed to load tools:', e)),
+				setBanners().catch((e) => console.error('Failed to load banners:', e)),
-			setUserSettings(async () => {
+				setTools().catch((e) => console.error('Failed to load tools:', e)),
-				await setModels().catch((e) => console.error('Failed to load models:', e));
+				setUserSettings(async () => {
-			}).catch((e) => console.error('Failed to load user settings:', e))
+					await setModels().catch((e) => console.error('Failed to load models:', e));
-		]);
+				})
+			]);
+		} catch (e) {
+			console.error('Failed to load user settings:', e);
+			toast.error($i18n.t('Failed to load Interface settings'));
+			return;
+		}
 
 		selectedTerminalId.set(localStorage.selectedTerminalId ?? null);