code review changes

Author: nickgros

packages/antd/src/widgets/SelectWidget/index.tsx

@@ -71,11 +71,11 @@ export default function SelectWidget<
 
   const selectOptions: DefaultOptionType[] | undefined = useMemo(() => {
     if (Array.isArray(enumOptions)) {
+      const options = [...enumOptions];
       if (!multiple && schema.default === undefined) {
-        enumOptions.unshift({ value: '', label: placeholder || '' });
+        options.unshift({ value: '', label: placeholder || '' });
       }
-
-      return enumOptions.map(({ value: optionValue, label: optionLabel }, index) => ({
+      return options.map(({ value: optionValue, label: optionLabel }, index) => ({
         disabled: Array.isArray(enumDisabled) && enumDisabled.indexOf(optionValue) !== -1,
         key: String(index),
         value: String(index),

packages/chakra-ui/src/SelectWidget/SelectWidget.tsx

@@ -64,10 +64,11 @@ export default function SelectWidget<T = any, S extends StrictRJSFSchema = RJSFS
   const _valueLabelMap: any = {};
   const displayEnumOptions: OptionsOrGroups<any, any> = useMemo(() => {
     if (Array.isArray(enumOptions)) {
+      const options = [...enumOptions];
       if (!multiple && schema.default === undefined) {
-        enumOptions.unshift({ value: '', label: placeholder || '' });
+        options.unshift({ value: '', label: placeholder || '' });
       }
-      return enumOptions.map((option: EnumOptionsType<S>, index: number) => {
+      return options.map((option: EnumOptionsType<S>, index: number) => {
         const { value, label } = option;
         _valueLabelMap[index] = label || String(value);
         return {

packages/chakra-ui/test/__snapshots__/Form.test.tsx.snap

@@ -5230,7 +5230,7 @@ exports[`single fields select field single choice formData 1`] = `
               <div
                 className=" emotion-7"
               >
-                bar
+                foo
               </div>
               <div
                 className=" emotion-8"

packages/material-ui/src/SelectWidget/SelectWidget.tsx

@@ -86,7 +86,7 @@ export default function SelectWidget<
       }}
       aria-describedby={ariaDescribedByIds<T>(id)}
     >
-      {!multiple && schema.default === undefined && <MenuItem value=''>{placeholder || ''}</MenuItem>}
+      {!multiple && schema.default === undefined && <MenuItem value=''>{placeholder}</MenuItem>}
       {Array.isArray(enumOptions) &&
         enumOptions.map(({ value, label }, i: number) => {
           const disabled: boolean = Array.isArray(enumDisabled) && enumDisabled.indexOf(value) !== -1;

packages/mui/src/SelectWidget/SelectWidget.tsx

@@ -89,7 +89,7 @@ export default function SelectWidget<
       }}
       aria-describedby={ariaDescribedByIds<T>(id)}
     >
-      {!multiple && schema.default === undefined && <MenuItem value=''>{placeholder || ''}</MenuItem>}
+      {!multiple && schema.default === undefined && <MenuItem value=''>{placeholder}</MenuItem>}
       {Array.isArray(enumOptions) &&
         enumOptions.map(({ value, label }, i: number) => {
           const disabled: boolean = Array.isArray(enumDisabled) && enumDisabled.indexOf(value) !== -1;