Merge pull request #36 from DumbWareio/dev

# Fix: Transaction Type Reference Error in Edit Mode

Author: abiteman

public/script.js

@@ -1,6 +1,13 @@
 import { ToastManager } from "./managers/toast";
 const toastManager = new ToastManager(document.getElementById('toast-container'));
 
+// Global variables
+let currentTransactionType = 'income'; // Default transaction type
+let currentFilter = null; // null = show all, 'income' = only income, 'expense' = only expenses
+let editingTransactionId = null;
+let currentSortField = 'date';
+let currentSortDirection = 'desc';
+
 // Theme toggle functionality
 function getBaseUrl() {
     // First try to get it from the server-provided meta tag
@@ -314,16 +321,6 @@ async function handleFetchResponse(response) {
     return response;
 }
 
-// Add currentFilter variable at the top with other shared variables
-let currentFilter = null; // null = show all, 'income' = only income, 'expense' = only expenses
-
-// Add at the top with other variables
-let editingTransactionId = null;
-
-// Add at the top with other shared variables
-let currentSortField = 'date';
-let currentSortDirection = 'desc';
-
 // Update loadTransactions function
 async function loadTransactions() {
     try {
@@ -470,6 +467,9 @@ function editTransaction(id, transaction, isRecurringInstance) {
     document.getElementById('description').value = transaction.description;
     document.getElementById('transactionDate').value = transaction.date;
 
+    // Update the currentTransactionType to match the transaction being edited
+    currentTransactionType = transaction.type;
+    
     // Set transaction type
     toggleBtns.forEach(btn => {
         btn.classList.toggle('active', btn.dataset.type === transaction.type);
@@ -651,8 +651,6 @@ function initModalHandling() {
     const toggleBtns = document.querySelectorAll('.toggle-btn');
     const amountInput = document.getElementById('amount');
 
-    let currentTransactionType = 'income';
-
     // Initialize category handling
     initCategoryHandling();