fix(theater): refresh Surgery Workbench Timed Services and Professional Fees tabs

SurgeryBillController's proEncounterComponents/timedEncounterComponents
lists were only ever initialized to an empty ArrayList and never queried
from the DB, so the Surgery Workbench summary tabs never showed data added
via the separate InwardTimedItemController/InwardProfessionalBillController
beans. Getters now lazily fetch from the DB, and the add/update/remove
actions in those beans invalidate the cache so the workbench re-fetches on
return.

Verified end-to-end with Playwright + DB checks against local Payara.

Fixes #20890

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: buddhika75

developer_docs/testing/playwright-e2e-workflow.md

@@ -1480,6 +1480,33 @@ field alone is not enough. Fix: search and select a Speciality (e.g. type
 statement reaches the server at all; with it filled, the insert fires
 immediately. Verified while testing issue #22665.
 
+## 56. `p:datePicker timeInput="true"` — typing into the input does not commit; use the PrimeFaces widget API for non-AJAX forms
+
+On `theater/inward_timed_service_consume_surgery.xhtml`'s Start/End Time
+fields (`p:datePicker showTime="true" timeInput="true"`, no `readonlyInput`
+set — `input.readOnly` is `false`), the documented "click → Ctrl+A →
+pressSequentially → Escape" pattern (§ "p:datePicker / p:calendar") left the
+input **empty** every time: `document.getElementById(...).value` read `""`
+both before and after `Escape`, with no visible error. Root cause wasn't
+narrowed further, but the fix that reliably works is to skip DOM typing
+entirely and drive the PrimeFaces widget directly — safe here because the
+submit button (`+ Add Service`) is `ajax="false"`, so (per §29) only the
+final submitted `_input` value matters:
+```js
+Object.keys(PrimeFaces.widgets).filter(k => /starttime|endtime/i.test(k))
+// -> ["widget_form_startTime", "widget_form_endTime"]
+PrimeFaces.widgets.widget_form_startTime.setDate(new Date(2026, 7, 5, 19, 0, 0));
+```
+`setDate()` both sets the widget's internal date **and** re-serializes the
+visible `_input` text using the field's configured pattern, so a DOM read
+right after confirms the committed value. Verified end-to-end for issue
+#20890: the typed-looking string round-tripped correctly through the
+non-AJAX submit and the saved `PATIENTITEM.FROMTIME`/`TOTIME` matched. Only
+use this shortcut for non-AJAX (full-postback) submits — for an AJAX
+`p:datePicker` where the *change* event itself must fire a listener, this
+bypasses that and the real key-event pattern would still be required (untested
+here).
+
 ## Quick checklist
 
 - [ ] Confirmed environment + URL with the developer; credentials kept out of the repo.

src/main/java/com/divudi/bean/inward/InwardProfessionalBillController.java

@@ -460,6 +460,7 @@ public void saveProfessionalFeeBill() {
         makeNullList();
         current = null;
         settlePreview = true;
+        surgeryBillController.refreshProEncounterComponents();
         JsfUtil.addSuccessMessage("Professional fee bill saved.");
     }
 

src/main/java/com/divudi/bean/inward/InwardTimedItemController.java

@@ -345,6 +345,7 @@ public void updateTimedService(BillFee bf) {
         bf.setFeeValue(value);
         bf.setFeeGrossValue(value);
         updateBillFee(bf);
+        surgeryBillController.refreshTimedEncounterComponents();
     }
 
     public void removeTimedEncFromList(EncounterComponent encounterComponent) {
@@ -366,6 +367,7 @@ public void removeTimedEncFromDbase(EncounterComponent encounterComponent) {
         updateBillItem(encounterComponent.getBillItem());
         updateBill(encounterComponent.getBillItem().getBill());
         getBillBean().updateBatchBill(getBatchBill());
+        surgeryBillController.refreshTimedEncounterComponents();
     }
 
     private void retiredEncounterComponent(EncounterComponent encounterComponent) {
@@ -569,6 +571,7 @@ public void saveSurgeryTimedService() {
         }
 
         getBillBean().updateBatchBill(getBatchBill());
+        surgeryBillController.refreshTimedEncounterComponents();
 
         JsfUtil.addSuccessMessage("Surgery Detail Successfull Updated");
 

src/main/java/com/divudi/bean/inward/SurgeryBillController.java

@@ -395,6 +395,7 @@ public void removeTimeService(PatientItem patientItem) {
             patientItem.setRetiredAt(new Date());
             patientItem.setRetired(true);
             getPatientItemFacade().edit(patientItem);
+            refreshTimedEncounterComponents();
         }
     }
 
@@ -1292,7 +1293,7 @@ public void addProfessionalFee() {
 
     public List<EncounterComponent> getProEncounterComponents() {
         if (proEncounterComponents == null) {
-            proEncounterComponents = new ArrayList<>();
+            fetchProEncounterComponents();
         }
         return proEncounterComponents;
     }
@@ -1301,6 +1302,29 @@ public void setProEncounterComponents(List<EncounterComponent> proEncounterCompo
         this.proEncounterComponents = proEncounterComponents;
     }
 
+    public void refreshProEncounterComponents() {
+        proEncounterComponents = null;
+    }
+
+    private void fetchProEncounterComponents() {
+        proEncounterComponents = new ArrayList<>();
+        if (surgeryBill == null || surgeryBill.getProcedure() == null
+                || surgeryBill.getProcedure().getId() == null) {
+            return;
+        }
+        String jpql = "SELECT ec FROM EncounterComponent ec"
+                + " WHERE ec.patientEncounter = :proc"
+                + " AND ec.billItem.bill.surgeryBillType = :sbt"
+                + " ORDER BY ec.orderNo";
+        HashMap<String, Object> params = new HashMap<>();
+        params.put("proc", surgeryBill.getProcedure());
+        params.put("sbt", SurgeryBillType.ProfessionalFee);
+        proEncounterComponents = getEncounterComponentFacade().findByJpql(jpql, params);
+        if (proEncounterComponents == null) {
+            proEncounterComponents = new ArrayList<>();
+        }
+    }
+
     public PatientEncounterFacade getPatientEncounterFacade() {
         return patientEncounterFacade;
     }
@@ -1359,7 +1383,7 @@ public void setBillNumberBean(BillNumberGenerator billNumberBean) {
 
     public List<EncounterComponent> getTimedEncounterComponents() {
         if (timedEncounterComponents == null) {
-            timedEncounterComponents = new ArrayList<>();
+            fetchTimedEncounterComponents();
         }
         return timedEncounterComponents;
     }
@@ -1368,6 +1392,22 @@ public void setTimedEncounterComponents(List<EncounterComponent> timedEncounterC
         this.timedEncounterComponents = timedEncounterComponents;
     }
 
+    public void refreshTimedEncounterComponents() {
+        timedEncounterComponents = null;
+    }
+
+    private void fetchTimedEncounterComponents() {
+        Bill timedBill = getSurgeryTimedServiceBill();
+        if (timedBill == null) {
+            timedEncounterComponents = new ArrayList<>();
+            return;
+        }
+        timedEncounterComponents = getBillBean().getEncounterComponents(timedBill);
+        if (timedEncounterComponents == null) {
+            timedEncounterComponents = new ArrayList<>();
+        }
+    }
+
     public PatientItemFacade getPatientItemFacade() {
         return patientItemFacade;
     }