@hero-design/rn 8.100.0 → 8.100.1

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
-(node:3207) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
+(node:3221) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
 (Use `node --trace-warnings ...` to show where the warning was created)
 
 src/index.ts → lib/index.js, es/index.js...
@@ -15,9 +15,9 @@ node_modules/d3-selection/src/selection/index.js -> node_modules/d3-selection/sr
                                      ~~~~~~~~~~~~~~~~~~~
 
 (!) [plugin node-resolve] preferring built-in module 'events' over local alternative at '/home/runner/work/hero-design/hero-design/node_modules/events/events.js', pass 'preferBuiltins: false' to disable this behavior or 'preferBuiltins: true' to disable this warning.or passing a function to 'preferBuiltins' to provide more fine-grained control over which built-in modules to prefer.
-created lib/index.js, es/index.js in 1m 17.5s
+created lib/index.js, es/index.js in 1m 9.2s
 
 /home/runner/work/hero-design/hero-design/packages/rn/src/locales/en_AU.ts, /home/runner/work/hero-design/hero-design/packages/rn/src/locales/en_CA.ts, /home/runner/work/hero-design/hero-design/packages/rn/src/locales/index.ts, /home/runner/work/hero-design/hero-design/packages/rn/src/locales/types.ts → ., ....
 (!) Generated empty chunks
 "locales/types" and "locales/types"
-created ., . in 22.3s
+created ., . in 20.1s

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @hero-design/rn
 
+## 8.100.1
+
+### Patch Changes
+
+- [#3914](https://github.com/Thinkei/hero-design/pull/3914) [`e89cfb49ca75f9b2104e72b630bfb20bfa896e6f`](https://github.com/Thinkei/hero-design/commit/e89cfb49ca75f9b2104e72b630bfb20bfa896e6f) Thanks [@vinhphan-eh](https://github.com/vinhphan-eh)! - [Select.Multi] Fix crash issue when value is falsy
+
 ## 8.100.0
 
 ### Minor Changes

package/es/index.js

@@ -24884,7 +24884,10 @@ var Option$2 = function Option(_ref) {
 };
 
 var isOptionSelected = function isOptionSelected(value, option) {
-  if (typeof option.value === 'string') {
+  if (!value || !option || option.value === null || option.value === undefined) {
+    return false;
+  }
+  if (_typeof$1(option.value) !== 'object') {
     return value.includes(option.value);
   }
   return value.some(function (v) {

package/lib/index.js

@@ -24913,7 +24913,10 @@ var Option$2 = function Option(_ref) {
 };
 
 var isOptionSelected = function isOptionSelected(value, option) {
-  if (typeof option.value === 'string') {
+  if (!value || !option || option.value === null || option.value === undefined) {
+    return false;
+  }
+  if (_typeof$1(option.value) !== 'object') {
     return value.includes(option.value);
   }
   return value.some(function (v) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hero-design/rn",
-  "version": "8.100.0",
+  "version": "8.100.1",
   "license": "MIT",
   "main": "lib/index.js",
   "module": "es/index.js",

package/src/components/Select/MultiSelect/__tests__/utils.spec.ts

@@ -28,4 +28,25 @@ describe('isOptionSelected', () => {
 
     expect(isOptionSelected(selectedValues, option)).toBe(false);
   });
+
+  it('should return false when value is falsy', () => {
+    const selectedValues = undefined;
+    const option = { value: 'option1', text: 'Option 1' };
+
+    expect(isOptionSelected(selectedValues, option)).toBe(false);
+  });
+
+  it('should return false when option is falsy', () => {
+    const selectedValues = ['option1', 'option2'];
+    const option = undefined;
+
+    expect(isOptionSelected(selectedValues, option)).toBe(false);
+  });
+
+  it('should work with boolean value', () => {
+    const selectedValues = [true, false];
+    const option = { value: true, text: 'Option 1' };
+
+    expect(isOptionSelected(selectedValues, option)).toBe(true);
+  });
 });

package/src/components/Select/MultiSelect/utils.ts

@@ -2,11 +2,21 @@ import { deepCompareValue } from '../../../utils/helpers';
 import { OptionType } from '../types';
 
 export const isOptionSelected = <V, T extends OptionType<V>>(
-  value: V[],
-  option: T
+  value?: V[],
+  option?: T
 ): boolean => {
-  if (typeof option.value === 'string') {
+  if (
+    !value ||
+    !option ||
+    option.value === null ||
+    option.value === undefined
+  ) {
+    return false;
+  }
+
+  if (typeof option.value !== 'object') {
     return value.includes(option.value);
   }
+
   return value.some((v) => deepCompareValue(v, option.value));
 };