refactor: 💡 rename comparison function

streamich · Aug 5, 2019 · 8462a59 · 8462a59
1 parent 6875e13
commit 8462a59
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/useUpsert.ts b/src/useUpsert.ts
@@ -5,17 +5,17 @@ export interface Actions<T> extends ListActions<T> {
 }
 
 const useUpsert = <T>(
-  itemsAreTheSame: (upsertedItem: T, existingItem: T) => boolean,
+  comparisonFunction: (upsertedItem: T, existingItem: T) => boolean,
   initialList: T[] = []
 ): [T[], Actions<T>] => {
   const [items, actions] = useList(initialList);
 
   const upsert = (upsertedItem: T) => {
-    const itemAlreadyExists = items.find(item => itemsAreTheSame(upsertedItem, item));
+    const itemAlreadyExists = items.find(item => comparisonFunction(upsertedItem, item));
     if (itemAlreadyExists) {
       return actions.set(
         items.map(existingItem => {
-          if (itemsAreTheSame(upsertedItem, existingItem)) {
+          if (comparisonFunction(upsertedItem, existingItem)) {
             return upsertedItem;
           }
           return existingItem;