From cd2fa7943bba5b0ae672bad678a86f1d6043b38e Mon Sep 17 00:00:00 2001
From: Mo Tarbin <mhed.t91@gmail.com>
Date: Mon, 23 Mar 2026 22:30:56 -0400
Subject: [PATCH] Refactor chore selection logic to pass 'chores' parameter to
 getSelectedChoresData in bulk actions

---
 src/views/Chores/MyChores.jsx             | 5 +++--
 src/views/Chores/hooks/useChoreActions.js | 8 ++++----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/views/Chores/MyChores.jsx b/src/views/Chores/MyChores.jsx
index cd9e859..d57c099 100644
--- a/src/views/Chores/MyChores.jsx
+++ b/src/views/Chores/MyChores.jsx
@@ -529,10 +529,11 @@ const MyChores = () => {
       onEnableMultiSelectAndSelectAll: () => {
         toggleMultiSelectMode()
         setTimeout(() => {
-          selectAllVisibleChores()
+          selectAllVisibleChores(null, choreSections, openChoreSections)
         }, 0)
       },
-      onSelectAll: () => selectAllVisibleChores(),
+      onSelectAll: () =>
+        selectAllVisibleChores(null, choreSections, openChoreSections),
       onClearSelection: clearSelection,
       onBulkComplete: handleBulkComplete,
       onBulkSkip: handleBulkSkip,
diff --git a/src/views/Chores/hooks/useChoreActions.js b/src/views/Chores/hooks/useChoreActions.js
index abf781f..9cf599d 100644
--- a/src/views/Chores/hooks/useChoreActions.js
+++ b/src/views/Chores/hooks/useChoreActions.js
@@ -472,7 +472,7 @@ export const useChoreActions = ({
   )
 
   const handleBulkComplete = useCallback(async () => {
-    const selectedData = getSelectedChoresData()
+    const selectedData = getSelectedChoresData(chores)
     if (selectedData.length === 0) return
 
     setConfirmModelConfig({
@@ -532,7 +532,7 @@ export const useChoreActions = ({
   }, [getSelectedChoresData, impersonatedUser, showSuccess, showError, refetchChores, clearSelection, setConfirmModelConfig])
 
   const handleBulkArchive = useCallback(async () => {
-    const selectedData = getSelectedChoresData()
+    const selectedData = getSelectedChoresData(chores)
     if (selectedData.length === 0) return
 
     setConfirmModelConfig({
@@ -594,7 +594,7 @@ export const useChoreActions = ({
   }, [getSelectedChoresData, archiveChore, setChores, setFilteredChores, showSuccess, showError, refetchChores, clearSelection, setConfirmModelConfig])
 
   const handleBulkDelete = useCallback(async () => {
-    const selectedData = getSelectedChoresData()
+    const selectedData = getSelectedChoresData(chores)
     if (selectedData.length === 0) return
 
     setConfirmModelConfig({
@@ -654,7 +654,7 @@ export const useChoreActions = ({
   }, [getSelectedChoresData, chores, filteredChores, setChores, setFilteredChores, showSuccess, showError, refetchChores, clearSelection, setConfirmModelConfig])
 
   const handleBulkSkip = useCallback(async () => {
-    const selectedData = getSelectedChoresData()
+    const selectedData = getSelectedChoresData(chores)
     if (selectedData.length === 0) return
 
     setConfirmModelConfig({