From 498716727b0c23e185216dbb03b522347d1191eb Mon Sep 17 00:00:00 2001
From: Taras Perun <terry.perun@gmail.com>
Date: Wed, 10 Jan 2024 23:23:04 +0100
Subject: [PATCH] undo runHoverAfterInteraction

---
 src/components/Hoverable/ActiveHoverable.tsx  | 31 ++++--------------
 .../CheckForPreviousReportActionIDClean.ts    | 32 -------------------
 2 files changed, 7 insertions(+), 56 deletions(-)
 delete mode 100644 src/libs/migrations/CheckForPreviousReportActionIDClean.ts

diff --git a/src/components/Hoverable/ActiveHoverable.tsx b/src/components/Hoverable/ActiveHoverable.tsx
index 6037092a562d..028fdd30cf35 100644
--- a/src/components/Hoverable/ActiveHoverable.tsx
+++ b/src/components/Hoverable/ActiveHoverable.tsx
@@ -1,32 +1,15 @@
 import type {Ref} from 'react';
 import {cloneElement, forwardRef, useCallback, useEffect, useMemo, useRef, useState} from 'react';
-import {DeviceEventEmitter, InteractionManager} from 'react-native';
+import {DeviceEventEmitter} from 'react-native';
 import mergeRefs from '@libs/mergeRefs';
 import {getReturnValue} from '@libs/ValueUtils';
 import CONST from '@src/CONST';
 import type HoverableProps from './types';
 
 type ActiveHoverableProps = Omit<HoverableProps, 'disabled'>;
-type UseHoveredReturnType = [boolean, (newValue: boolean) => void];
-// This is a workaround specifically for the web part of comment linking. Without this adjustment, you might observe sliding effects due to conflicts between  MVCPFlatList implementation and this file. Check it once https://github.com/necolas/react-native-web/pull/2588 is merged
-function useHovered(initialValue: boolean, runHoverAfterInteraction: boolean): UseHoveredReturnType {
-    const [state, setState] = useState(initialValue);
-
-    const interceptedSetState = useCallback((newValue: boolean) => {
-        if (runHoverAfterInteraction) {
-            InteractionManager.runAfterInteractions(() => {
-                setState(newValue);
-            });
-        } else {
-            setState(newValue);
-        }
-        // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, []);
-    return [state, interceptedSetState];
-}
 
-function ActiveHoverable({onHoverIn, onHoverOut, shouldHandleScroll, children, runHoverAfterInteraction = false}: ActiveHoverableProps, outerRef: Ref<HTMLElement>) {
-    const [isHovered, setIsHovered] = useHovered(false, runHoverAfterInteraction);
+function ActiveHoverable({onHoverIn, onHoverOut, shouldHandleScroll, children}: ActiveHoverableProps, outerRef: Ref<HTMLElement>) {
+    const [isHovered, setIsHovered] = useState(false);
 
     const elementRef = useRef<HTMLElement | null>(null);
     const isScrollingRef = useRef(false);
@@ -40,7 +23,7 @@ function ActiveHoverable({onHoverIn, onHoverOut, shouldHandleScroll, children, r
             }
             setIsHovered(hovered);
         },
-        [setIsHovered, shouldHandleScroll],
+        [shouldHandleScroll],
     );
 
     useEffect(() => {
@@ -64,7 +47,7 @@ function ActiveHoverable({onHoverIn, onHoverOut, shouldHandleScroll, children, r
         });
 
         return () => scrollingListener.remove();
-    }, [setIsHovered, shouldHandleScroll]);
+    }, [shouldHandleScroll]);
 
     useEffect(() => {
         // Do not mount a listener if the component is not hovered
@@ -89,7 +72,7 @@ function ActiveHoverable({onHoverIn, onHoverOut, shouldHandleScroll, children, r
         document.addEventListener('mouseover', unsetHoveredIfOutside);
 
         return () => document.removeEventListener('mouseover', unsetHoveredIfOutside);
-    }, [setIsHovered, isHovered, elementRef]);
+    }, [isHovered, elementRef]);
 
     useEffect(() => {
         const unsetHoveredWhenDocumentIsHidden = () => document.visibilityState === 'hidden' && setIsHovered(false);
@@ -130,7 +113,7 @@ function ActiveHoverable({onHoverIn, onHoverOut, shouldHandleScroll, children, r
 
             child.props.onBlur?.(event);
         },
-        [setIsHovered, child.props],
+        [child.props],
     );
 
     return cloneElement(child, {
diff --git a/src/libs/migrations/CheckForPreviousReportActionIDClean.ts b/src/libs/migrations/CheckForPreviousReportActionIDClean.ts
deleted file mode 100644
index 4362ae79114b..000000000000
--- a/src/libs/migrations/CheckForPreviousReportActionIDClean.ts
+++ /dev/null
@@ -1,32 +0,0 @@
-import Onyx, {OnyxCollection} from 'react-native-onyx';
-import ONYXKEYS from '@src/ONYXKEYS';
-import * as OnyxTypes from '@src/types/onyx';
-
-function getReportActionsFromOnyx(): Promise<OnyxCollection<OnyxTypes.ReportActions>> {
-    return new Promise((resolve) => {
-        const connectionID = Onyx.connect({
-            key: ONYXKEYS.COLLECTION.REPORT_ACTIONS,
-            waitForCollectionCallback: true,
-            callback: (allReportActions) => {
-                Onyx.disconnect(connectionID);
-                return resolve(allReportActions);
-            },
-        });
-    });
-}
-
-/**
- * This migration checks for the 'previousReportActionID' key in the first valid reportAction of a report in Onyx.
- * If the key is not found then all reportActions for all reports are removed from Onyx.
- */
-export default function (): Promise<void> {
-    return getReportActionsFromOnyx().then((allReportActions) => {
-        const onyxData: OnyxCollection<OnyxTypes.ReportActions> = {};
-
-        Object.keys(allReportActions ?? {}).forEach((onyxKey) => {
-            onyxData[onyxKey] = {};
-        });
-
-        return Onyx.multiSet(onyxData as Record<`${typeof ONYXKEYS.COLLECTION.REPORT_ACTIONS}`, Record<string, never>>);
-    });
-}