Merge pull request #2105 from appwrite/fix-language-persistency

Fix: only save and retrieve valid platforms

Author: ItzNotABug

src/lib/utils/references.ts

@@ -65,6 +65,7 @@ export const Platform = {
 
 type PlatformType = typeof Platform;
 export type Platform = (typeof Platform)[keyof typeof Platform];
+export const VALID_PLATFORMS = new Set(Object.values(Platform));
 
 export const Framework = {
     NextJs: 'Next.js',
@@ -154,16 +155,27 @@ export const preferredVersion = writable<Version | null>(
     globalThis?.localStorage?.getItem('preferredVersion') as Version
 );
 
-export const preferredPlatform = writable<Platform>(
-    (globalThis?.localStorage?.getItem('preferredPlatform') ?? 'client-web') as Platform
-);
+function getInitialPlatform(): Platform {
+    const stored = globalThis?.localStorage?.getItem('preferredPlatform') ?? Platform.ClientWeb;
+    // return if this platform is valid
+    if (VALID_PLATFORMS.has(stored as Platform)) {
+        return stored as Platform;
+    } else {
+        return Platform.ClientWeb;
+    }
+}
+
+export const preferredPlatform = writable<Platform>(getInitialPlatform());
 
 if (browser) {
     preferredVersion.subscribe((value) => {
         if (value) globalThis?.localStorage?.setItem('preferredVersion', value);
     });
 
     preferredPlatform.subscribe((value) => {
-        if (value) globalThis?.localStorage?.setItem('preferredPlatform', value);
+        // only save the ones for which we have api references.
+        if (value && VALID_PLATFORMS.has(value)) {
+            globalThis?.localStorage?.setItem('preferredPlatform', value);
+        }
     });
 }