https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43293

--- Comment #2 from Owen Leonard <[email protected]> ---
Created attachment 203635
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203635&action=edit
Bug 43293: Add IDs to the Pay/Write off confirm buttons on paycollect.tt

The Confirm submit button on the combined pay/writeoff form doesn't
have an id, making it hard for local customisations, hardware
integrations (e.g. receipt printer triggers), and automated tests to
reliably target the pay action versus the writeoff action - both
render as the same markup with only the value/hidden op field
differing.

Add id="paysubmit" and id="writeoffsubmit" to the two branches of
the existing type == 'WRITEOFF' conditional, matching the existing
op=cud-pay/op=cud-writeoff split right above it.

Test plan:
1) Go to a patron's Pay fines page and select multiple charges
2) Choose Pay - confirm the Confirm button has id="paysubmit"
3) Choose Write off - confirm the Confirm button has id="writeoffsubmit"
4) Confirm both flows still submit and behave exactly as before

Signed-off-by: Owen Leonard <[email protected]>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to