From 18ec393c0a5f2a3c2335f3e05919b726631697e5 Mon Sep 17 00:00:00 2001 From: Konrad du Plessis Date: Sat, 16 May 2026 14:01:34 +0200 Subject: [PATCH] fix: reset Add-Adjustment pay-type filter on every modal open Prevents a pre-checked quick-adjust worker from opening hidden behind a stale 'Managers only'/'Daily only' filter. Display-only; no data impact. Co-Authored-By: Claude Opus 4.7 (1M context) --- core/templates/core/payroll_dashboard.html | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/core/templates/core/payroll_dashboard.html b/core/templates/core/payroll_dashboard.html index 9e0525e..57c5ec2 100644 --- a/core/templates/core/payroll_dashboard.html +++ b/core/templates/core/payroll_dashboard.html @@ -2086,6 +2086,18 @@ document.addEventListener('DOMContentLoaded', function() { var addAdjModal = document.getElementById('addAdjustmentModal'); if (addAdjModal) { addAdjModal.addEventListener('show.bs.modal', function() { + // Always start the picker from a clean, fully-visible state: + // reset the pay-type filter and un-hide every worker row. Must + // run before the quick-adjust / pay-salary early returns so a + // pre-checked worker can never open hidden behind a stale + // "Managers only" / "Daily only" filter. + if (addAdjPayTypeFilter) { + addAdjPayTypeFilter.value = ''; + addAdjWorkerCheckboxes.forEach(function(cb) { + var r = cb.closest('.form-check'); + if (r) r.style.display = ''; + }); + } if (_quickAdjustOpen) { _quickAdjustOpen = false; return; // Quick-adjust already set the values