Using if clause instead of lodaashGet

Signed-off-by: Tsaqif <[email protected]>
Expensify · Oct 16, 2023 · 2245a7e · 2245a7e
1 parent c58c53e
commit 2245a7e
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/components/PopoverProvider/index.js b/src/components/PopoverProvider/index.js
@@ -1,6 +1,5 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import lodashGet from 'lodash/get';
 
 const propTypes = {
     children: PropTypes.node.isRequired,
@@ -25,7 +24,9 @@ function PopoverContextProvider(props) {
         }
 
         activePopoverRef.current.close();
-        lodashGet(activePopoverRef, 'current.onCloseCallback', () => {})();
+        if (activePopoverRef && activePopoverRef.current && activePopoverRef.current.onCloseCallback) {
+            activePopoverRef.current.onCloseCallback();
+        }
         activePopoverRef.current = null;
         setIsOpen(false);
     }, []);
@@ -109,7 +110,9 @@ function PopoverContextProvider(props) {
                 closePopover(activePopoverRef.current.anchorRef);
             }
             activePopoverRef.current = popoverParams;
-            lodashGet(activePopoverRef, 'current.onOpenCallback', () => {})();
+            if (popoverParams && popoverParams.onOpenCallback) {
+                popoverParams.onOpenCallback();
+            }
             setIsOpen(true);
         },
         [closePopover],