From 3e68295f9562339f113d90fe134d9d122ba644ef Mon Sep 17 00:00:00 2001
From: richardmilles <76790700+richardmilles@users.noreply.github.com>
Date: Mon, 5 Oct 2026 08:40:29 +0200
Subject: [PATCH] fix(expenses): refresh table when date range or filters
 change (#4633)

Restore the daterangepicker apply and filter change handlers so
/expenses/search receives the selected start_date and end_date
instead of today's date for both. Also removes leftover comments
from the expenses manage view.

Fixes #4596

---------

Co-authored-by: jekkos <jeroen.peelaerts@gmail.com>
---
 app/Views/expenses/manage.php | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/Views/expenses/manage.php b/app/Views/expenses/manage.php
index 945429d7d..ecdba7450 100644
--- a/app/Views/expenses/manage.php
+++ b/app/Views/expenses/manage.php
@@ -14,12 +14,18 @@
 
 <script type="text/javascript">
     $(document).ready(function() {
-        // Load the preset datarange picker
+        $('#filters').on('hidden.bs.select', function(e) {
+            table_support.refresh();
+        });
+
         <?= view('partial/daterangepicker') ?>
 
+        $("#daterangepicker").on('apply.daterangepicker', function(ev, picker) {
+            table_support.refresh();
+        });
+
         <?= view('partial/bootstrap_tables_locale') ?>
 
-        // Override dates from server if provided
         <?php if (isset($start_date) && $start_date): ?>
         start_date = "<?= esc($start_date) ?>";
         <?php endif; ?>