refactor(*): nullish coalescing operator used throughout codebase (#119)

Co-authored-by: Matthew Brandt <matthew.brandt@bigladdersoftware.com>

Author: matthewbbrandt

src/chakra-ui/index.ts

@@ -121,12 +121,12 @@ export const DEFAULT_CONFIGURATION = {
 export const getTheme = (options?: Options, configuration?: Configuration): Theme => {
   const mergedOptions = {
     ...DEFAULT_OPTIONS,
-    ...(options ? options : {}),
+    ...(options ?? {}),
   };
 
   const mergedConfiguration = {
     ...DEFAULT_CONFIGURATION,
-    ...(configuration ? configuration : {}),
+    ...(configuration ?? {}),
   };
 
   return getZipTheme(mergedOptions, mergedConfiguration);

src/mantine/index.ts

@@ -150,12 +150,12 @@ export const DEFAULT_CONFIGURATION: ConfigurationSound = {
 export const getTheme = (options?: Options, configuration?: Configuration): Theme => {
   const mergedOptions = {
     ...DEFAULT_OPTIONS,
-    ...(options ? options : {}),
+    ...(options ?? {}),
   };
 
   const mergedConfiguration = {
     ...DEFAULT_CONFIGURATION,
-    ...(configuration ? configuration : {}),
+    ...(configuration ?? {}),
   };
 
   return getZipTheme(mergedOptions, mergedConfiguration);

src/material-ui/index.ts

@@ -149,12 +149,12 @@ export const DEFAULT_CONFIGURATION = {
 export const getTheme = (options?: Options, configuration?: Configuration): Theme => {
   const mergedOptions = {
     ...DEFAULT_OPTIONS,
-    ...(options ? options : {}),
+    ...(options ?? {}),
   };
 
   const mergedConfiguration = {
     ...DEFAULT_CONFIGURATION,
-    ...(configuration ? configuration : {}),
+    ...(configuration ?? {}),
   };
 
   return getZipTheme(mergedOptions, mergedConfiguration);

src/pagination/usePagination.ts

@@ -44,9 +44,10 @@ const usePagination = <T extends TableNode>(
   options?: PaginationOptions,
   context?: any,
 ): Pagination<T> => {
-  const controlledState: State = primary?.state
-    ? { ...DEFAULT_STATE, ...primary.state }
-    : { ...DEFAULT_STATE };
+  const controlledState: State = {
+    ...DEFAULT_STATE,
+    ...(primary?.state ?? {}),
+  };
 
   const onChange = primary?.onChange ? primary.onChange : () => {};
 
@@ -129,7 +130,7 @@ const usePagination = <T extends TableNode>(
 
   const mergedOptions = {
     ...DEFAULT_OPTIONS,
-    ...(options ? options : {}),
+    ...(options ?? {}),
   };
 
   const stateAndGetters = {

src/select/useRowSelect.tsx

@@ -110,9 +110,10 @@ const useRowSelect = <T extends TableNode>(
   options?: SelectOptions,
   context?: any,
 ): Select<T> => {
-  const controlledState: State = primary?.state
-    ? { ...DEFAULT_STATE, ...primary.state }
-    : { ...DEFAULT_STATE };
+  const controlledState: State = {
+    ...DEFAULT_STATE,
+    ...(primary?.state ?? {}),
+  };
 
   const onChange = primary?.onChange ? primary.onChange : () => {};
 
@@ -122,7 +123,7 @@ const useRowSelect = <T extends TableNode>(
 
   const mergedOptions = {
     ...DEFAULT_OPTIONS,
-    ...(options ? options : {}),
+    ...(options ?? {}),
   };
 
   return {

src/sort/useSort.tsx

@@ -89,9 +89,10 @@ const useSort = <T extends TableNode>(
   options: SortOptions,
   context?: any,
 ): Sort<T> => {
-  const controlledState: State = primary?.state
-    ? { ...DEFAULT_STATE, ...primary.state }
-    : { ...DEFAULT_STATE };
+  const controlledState: State = {
+    ...DEFAULT_STATE,
+    ...(primary.state ?? {}),
+  };
 
   const onChange = primary?.onChange ? primary.onChange : () => {};
 

src/table/useCustom/index.ts

@@ -35,9 +35,10 @@ const useCustom = <T extends TableNode>(
   primary: StateAndChange | Nullish,
   context?: any,
 ) => {
-  const controlledState: State = primary?.state
-    ? { ...DEFAULT_STATE, ...primary.state }
-    : { ...DEFAULT_STATE };
+  const controlledState: State = {
+    ...DEFAULT_STATE,
+    ...(primary?.state ?? {}),
+  };
 
   const onChange = primary?.onChange ? primary.onChange : () => {};
 

src/tree/useTree.tsx

@@ -105,9 +105,10 @@ const useTree = <T extends TableNode>(
   options?: TreeOptions<T>,
   context?: any,
 ): Tree<T> => {
-  const controlledState: State = primary?.state
-    ? { ...DEFAULT_STATE, ...primary.state }
-    : { ...DEFAULT_STATE };
+  const controlledState: State = {
+    ...DEFAULT_STATE,
+    ...(primary?.state ?? {}),
+  };
 
   const onChange = primary?.onChange ? primary.onChange : () => {};
 
@@ -117,10 +118,10 @@ const useTree = <T extends TableNode>(
 
   const mergedOptions = {
     ...DEFAULT_OPTIONS,
-    ...(options ? options : {}),
+    ...(options ?? {}),
     treeIcon: {
       ...DEFAULT_TREE_ICON,
-      ...(options ? options.treeIcon : {}),
+      ...(options?.treeIcon ?? {}),
     },
   };