Merge pull request #334 from rclankhorst/master

fix: MultiSelectElement does not allow for options w/ { id: '0' }
dohomi · Dec 5, 2024 · c9fca0e · c9fca0e
2 parents a423841 + 74f0c5b
commit c9fca0e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/apps/storybook/stories/MultiSelectElement.stories.tsx b/apps/storybook/stories/MultiSelectElement.stories.tsx
@@ -108,10 +108,10 @@ export const WithCheckbox: Story = {
 }
 
 const objectVals = [
-  {id: 1, name: 'Alpha'},
-  {id: 2, name: 'Beta'},
-  {id: 3, name: 'Celsius'},
-  {id: 4, name: 'Delta'},
+  {id: 0, name: 'Alpha'},
+  {id: 1, name: 'Beta'},
+  {id: 2, name: 'Celsius'},
+  {id: 3, name: 'Delta'},
 ]
 
 export const WithObjectArray: Story = {

diff --git a/packages/rhf-mui/src/MultiSelectElement.tsx b/packages/rhf-mui/src/MultiSelectElement.tsx
@@ -261,7 +261,7 @@ const MultiSelectElement = forwardRef(function MultiSelectElement<
         inputRef={handleInputRef}
       >
         {options.map((item) => {
-          const val: string | number = item[itemValue || itemKey] || item
+          const val: string | number = item[itemValue || itemKey] ?? item
           const isChecked = Array.isArray(value)
             ? value.some((v) => v === val)
             : false