Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix AutocompleteInput when multiple is true and value is not an array #8602

Merged
merged 1 commit into from
Jan 27, 2023

Conversation

djhi
Copy link
Contributor

@djhi djhi commented Jan 27, 2023

No description provided.

@fzaninotto fzaninotto merged commit 1dc4e6b into master Jan 27, 2023
@fzaninotto fzaninotto deleted the make-autocompleteinput-resilient branch January 27, 2023 16:32
@fzaninotto fzaninotto added this to the 4.7.2 milestone Jan 27, 2023
@@ -742,7 +742,7 @@ const getSelectedItems = (
multiple
) => {
if (multiple) {
return (value || [])
return (Array.isArray(value || []) ? value : [value])
Copy link

@nerixim nerixim Jan 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fzaninotto @djhi
I'm not sure why I'm getting an empty string here, but I get an error here ((intermediate value)(intermediate value)(intermediate value).map is not a function)

const value = ""
Array.isArray(value || []) ? value : [value]
// ''

Nullish coalescing seems to fix this particular case, though I'm not sure of any caveats.

const value = ""
Array.isArray(value ?? []) ? value : [value]
// [ '' ]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants