From 5b593e7a2c4fc52c53623526921904b63a158cee Mon Sep 17 00:00:00 2001
From: "Bobby Abellana (aider)" <bobby@jumpstarttech.com>
Date: Fri, 21 Feb 2025 09:20:31 -0800
Subject: [PATCH] fix: Rename event handlers to avoid recursive calls

---
 EventHandlers.ps1 | 12 ++++++------
 MainForm.ps1      |  6 +++---
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/EventHandlers.ps1 b/EventHandlers.ps1
index b4952e9..4bc054b 100644
--- a/EventHandlers.ps1
+++ b/EventHandlers.ps1
@@ -1,15 +1,15 @@
 # Event handlers for button clicks
-function List-XlsFiles {
+function List-XlsFiles_Click { # Rename to avoid conflict
     $script:txtStatus.Text = "Listing XLS files...`n"
-    List-XlsFiles
+    List-XlsFiles # Call the function in SharePointFunctions.ps1
 }
 
-function Convert-Files {
+function Convert-Files_Click { # Rename to avoid conflict
     $script:txtStatus.Text += "Starting conversion...`n"
-    Convert-Files
+    Convert-Files # Call the function in SharePointFunctions.ps1
 }
 
-function Move-Files {
+function Move-Files_Click { # Rename to avoid conflict
     $script:txtStatus.Text += "Moving files...`n"
-    Move-Files
+    Move-Files # Call the function in SharePointFunctions.ps1
 }
diff --git a/MainForm.ps1 b/MainForm.ps1
index d8e275e..db11a3b 100644
--- a/MainForm.ps1
+++ b/MainForm.ps1
@@ -111,9 +111,9 @@ else {
 }
 
 # Attach event handlers
-$btnList.Add_Click({ List-XlsFiles })
-$btnConvert.Add_Click({ Convert-Files })
-$btnMove.Add_Click({ Move-Files })
+$btnList.Add_Click({ List-XlsFiles_Click })
+$btnConvert.Add_Click({ Convert-Files_Click })
+$btnMove.Add_Click({ Move-Files_Click })
 $btnBrowse.Add_Click({
     try {
         if (-not (Connect-SharePoint)) {