]> git.koha-community.org Git - koha.git/commit
Bug 34085: Remove the use of event attributes from basket groups template
authorOwen Leonard <oleonard@myacpl.org>
Wed, 21 Jun 2023 17:24:20 +0000 (17:24 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 8 Sep 2023 14:54:57 +0000 (11:54 -0300)
commit059e598f388bd99d998e69e30515b419518e31cb
tree2bd9358a5d46add4f4ddf3915decae77620f7824
parentc23f36b81218032ed2aa3f2f476fd880e2d1b704
Bug 34085: Remove the use of event attributes from basket groups template

This patch removes the use of event attributes (onclick, onsubmit) from
the basket groups template. These events are defined now along with the
other in-page JS.

This change allows us to move the scripts to the bottom of the page.

Unrelated changes: There was no confirmation tied to the process of
deleting a basket group. I've added one. And there were several "MSG_"
variables defined which were unused and have been removed.

To test you should have at least one vendor with at least one basket
group.

- Apply the patch and go to Acquisitions.
- Locate a vendor with an associated basket group.
- Edit a basketgroup
  - Confirm that editing works correctly (leaving "Close basket group"
    unchecked")
  - Confirm that editing works correctly with "Close basket group"
    checked.
- In the list of closed basket groups, confirm that all the controls
  work correctly.
- Create an empty basket group and confirm that clicking the associated
  "Delete" button in the list of basket groups triggers a confirmation,
  "Are you sure you want to delete this basket group?" Test that both
  confirming and cancelling work correctly.

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt