expensify-common 2.0.25 → 2.0.26

package/dist/Cookie.js

@@ -67,7 +67,7 @@ function enabled() {
     const cookieName = `cookieTest_${Math.floor(Math.random() * 1000)}`;
     const cookieValue = 'enabled';
     set(cookieName, cookieValue, 1);
-    const result = Boolean(document.cookie.indexOf(cookieName) >= 0) || false;
+    const result = document.cookie.indexOf(cookieName) >= 0 || false;
     if (result) {
         remove(cookieName);
     }

package/dist/components/form/element/combobox.js

@@ -417,7 +417,7 @@ class Combobox extends react_1.default.Component {
         const dividerIndex = this.options.findIndex(findDivider);
         // Split into two arrays everything before and after the divider (if the divider does not exist then we'll return a single array)
         const splitOptions = dividerIndex ? [this.options.slice(0, dividerIndex + 1), this.options.slice(dividerIndex + 1)] : [this.options];
-        const formatOption = (option) => (Object.assign({ focused: false, isSelected: option.selected && ((0, lodash_1.isEqual)(option.value, currentValue) || Boolean(alreadySelected.find((item) => item.value === option.value))) }, option));
+        const formatOption = (option) => (Object.assign({ focused: false, isSelected: option.selected && ((0, lodash_1.isEqual)(option.value, currentValue) || !!alreadySelected.find((item) => item.value === option.value)) }, option));
         const sortByOption = (o) => {
             // Unselectable text-only entries (isFake: true) go to the bottom and selected entries go to the top only if alwaysShowSelectedOnTop was passed
             if (o.showLast) {
@@ -472,7 +472,7 @@ class Combobox extends react_1.default.Component {
         const deselectOption = (initialOption) => {
             const option = initialOption;
             const isSelected = (0, lodash_1.isEqual)(option.value, val);
-            option.isSelected = isSelected || Boolean(this.props.alreadySelectedOptions.find((optionItem) => optionItem.value === option.value));
+            option.isSelected = isSelected || !!this.props.alreadySelectedOptions.find((optionItem) => optionItem.value === option.value);
             return option;
         };
         const deselectOptions = (options) => options.map(deselectOption);
@@ -562,7 +562,7 @@ class Combobox extends react_1.default.Component {
         }
         const state = this.getStartState(noDefaultValue, this.options, newAlreadySelectedOptions);
         const handleDropdownStateChange = () => {
-            this.props.onDropdownStateChange(Boolean(state.isDropdownOpen));
+            this.props.onDropdownStateChange(!!state.isDropdownOpen);
         };
         this.setState(state, handleDropdownStateChange);
     }
@@ -739,8 +739,7 @@ class Combobox extends react_1.default.Component {
             }
         }
         matches = Array.from(matches);
-        const formatOption = (option) => (Object.assign({ focused: false, isSelected: (0, lodash_1.isEqual)(option.value ? option.value.toUpperCase : '', value.toUpperCase()) ||
-                Boolean(this.props.alreadySelectedOptions.find((optionItem) => optionItem.value === option.value)) }, option));
+        const formatOption = (option) => (Object.assign({ focused: false, isSelected: (0, lodash_1.isEqual)(option.value ? option.value.toUpperCase : '', value.toUpperCase()) || !!this.props.alreadySelectedOptions.find((optionItem) => optionItem.value === option.value) }, option));
         const options = matches.map(formatOption);
         // Focus the first option if there is one and show a message dependent on what options are present
         if (options.length) {

package/dist/str.js

@@ -333,7 +333,7 @@ const Str = {
      * @returns True if the string is a domain name
      */
     isValidDomainName(str) {
-        return Boolean(String(str).match(Constants.CONST.REG_EXP.DOMAIN));
+        return !!String(str).match(Constants.CONST.REG_EXP.DOMAIN);
     },
     /**
      * Checks that the string is a valid url
@@ -341,7 +341,7 @@ const Str = {
      * @returns True if the string is a valid hyperlink
      */
     isValidURL(str) {
-        return Boolean(String(str).match(Constants.CONST.REG_EXP.HYPERLINK));
+        return !!String(str).match(Constants.CONST.REG_EXP.HYPERLINK);
     },
     /**
      * Checks that the string is an email address.
@@ -352,7 +352,7 @@ const Str = {
      * @returns True if the string is an email
      */
     isValidEmail(str) {
-        return Boolean(String(str).match(Constants.CONST.REG_EXP.EMAIL));
+        return !!String(str).match(Constants.CONST.REG_EXP.EMAIL);
     },
     /**
      * Checks if the string is an valid email address formed during comment markdown formation.
@@ -362,7 +362,7 @@ const Str = {
      * @returns True if the string is an valid email created by comment markdown.
      */
     isValidEmailMarkdown(str) {
-        return Boolean(String(str).match(`^${Constants.CONST.REG_EXP.MARKDOWN_EMAIL}$`));
+        return !!String(str).match(`^${Constants.CONST.REG_EXP.MARKDOWN_EMAIL}$`);
     },
     /**
      * Remove trailing comma from a string.
@@ -752,7 +752,7 @@ const Str = {
         if (this.isString(value)) {
             return value.toLowerCase() === 'true';
         }
-        return Boolean(value);
+        return !!value;
     },
     /**
      * Checks if a string could be the masked version of another one.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expensify-common",
-  "version": "2.0.25",
+  "version": "2.0.26",
   "author": "Expensify, Inc.",
   "description": "Expensify libraries and components shared across different repos",
   "homepage": "https://expensify.com",
@@ -56,7 +56,7 @@
     "babel-jest": "^29.0.0",
     "babelify": "10.0.0",
     "eslint": "^8.57.0",
-    "eslint-config-expensify": "^2.0.48",
+    "eslint-config-expensify": "^2.0.50",
     "eslint-config-prettier": "^8.10.0",
     "eslint-plugin-jest": "^28.6.0",
     "eslint-plugin-prettier": "^5.1.3",