fill Dependency List

Author: alex-Arc

apps/client/src/common/hooks/useEntryAction.ts

@@ -335,19 +335,6 @@ export const useEntryActions = () => {
     [getCurrentRundownData, updateEntryMutation],
   );
 
-  const matchGroupDuration = useCallback(async (eventId: EntryId, groupId: EntryId) => {
-    const rundown = queryClient.getQueryData<Rundown>(RUNDOWN);
-    if (!rundown) return;
-    const group = rundown.entries[groupId];
-    if (!group || !isOntimeGroup(group) || group.targetDuration === null) return;
-    const event = rundown.entries[eventId];
-    if (!event || !isOntimeEvent(event)) return;
-    const durationDiff = group.targetDuration - group.duration;
-    const newDuration = event.duration + durationDiff;
-    if (newDuration < 0) return;
-    updateTimer(eventId, 'duration', String(newDuration / MILLIS_PER_SECOND) + 's', false);
-  }, []);
-
   /**
    * Updates time of existing event
    * @param eventId {EntryId} - id of the event
@@ -446,6 +433,24 @@ export const useEntryActions = () => {
     [getCurrentRundownData, updateEntryMutation, queryClient],
   );
 
+  /**
+ * Updates time of existing event so it satisfies the group target duration
+ * @param eventId {EntryId} - id of the event
+ * @param groupId {EntryId} - id of the enclosing group
+ */
+  const matchGroupDuration = useCallback(async (eventId: EntryId, groupId: EntryId) => {
+    const rundown = queryClient.getQueryData<Rundown>(RUNDOWN);
+    if (!rundown) return;
+    const group = rundown.entries[groupId];
+    if (!group || !isOntimeGroup(group) || group.targetDuration === null) return;
+    const event = rundown.entries[eventId];
+    if (!event || !isOntimeEvent(event)) return;
+    const durationDiff = group.targetDuration - group.duration;
+    const newDuration = event.duration + durationDiff;
+    if (newDuration < 0) return;
+    updateTimer(eventId, 'duration', String(newDuration / MILLIS_PER_SECOND) + 's', false);
+  }, [queryClient, updateTimer]);
+
   /**
    * Calls mutation to edit multiple events
    * @private