From 5691052a977a672627baa6f54da57e976a43e845 Mon Sep 17 00:00:00 2001
From: Joshua Potter <jrpotter2112@gmail.com>
Date: Tue, 5 Dec 2023 15:55:01 -0700
Subject: [PATCH] Ensure language defaultValue is used.

---
 assets/js/react/components/SelectLanguage.tsx | 24 +++++++++----------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/assets/js/react/components/SelectLanguage.tsx b/assets/js/react/components/SelectLanguage.tsx
index 983d0b0..b528c81 100644
--- a/assets/js/react/components/SelectLanguage.tsx
+++ b/assets/js/react/components/SelectLanguage.tsx
@@ -1,6 +1,8 @@
 import * as React from "react"
 import { SelectProps } from "@mui/base/Select"
 
+import type { Language } from "../types/Language"
+
 import { Select, Option } from "./Select"
 import { useFetchLanguages } from "../utils/queries"
 
@@ -11,31 +13,29 @@ export const SelectLanguage = React.forwardRef(function SelectLanguage(
   ref: React.ForwardedRef<HTMLButtonElement>
 ) {
   const id = React.useId()
-  const [options, setOptions] = React.useState([
-    { value: "", label: "Loading..." },
-  ])
+  const [options, setOptions] = React.useState<Language[] | null>(null)
   const { defaultValue, ...other } = props
   const { isLoading, data } = useFetchLanguages()
 
   React.useEffect(() => {
-    if (!data) {
-      return
+    if (data) {
+      setOptions(data)
     }
-    setOptions(data.map((row) => ({ value: row.code, label: row.name })))
   }, [data])
 
   return (
     <Select
       ref={ref}
-      key={isLoading ? `${id}-loading` : `${id}-loaded`}
-      className={isLoading ? "text-slate-900/60" : ""}
-      defaultValue={defaultValue}
+      // Use key to force re-render and ensure the defaultValue takes effect.
+      key={!options && isLoading ? `${id}-loading` : `${id}-loaded`}
+      className={!options && isLoading ? "text-slate-900/60" : ""}
+      defaultValue={!options && isLoading ? [""] : defaultValue}
       {...other}
     >
-      {options.map((entry, index) => {
+      {options?.map((lang, index) => {
         return (
-          <Option key={index} value={entry.value}>
-            {entry.label}
+          <Option key={index} value={lang.code}>
+            {lang.name}
           </Option>
         )
       })}