Bug 33590: Fix DOM id name for tabs after WRAPPER code added to article-requests.tt
authorSlava Shishkin <slavashishkin@gmail.com>
Wed, 26 Apr 2023 16:59:14 +0000 (19:59 +0300)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 5 May 2023 12:13:16 +0000 (09:13 -0300)
commit6ebe303e14e0c5ca542c6d3da5df1369ab3b6c18
tree47985af65dad06e7752b4c03df8c2243ef2259d6
parent0a0c6554d72ea5acec1a6c4fe63b77e9383e2e56
Bug 33590: Fix DOM id name for tabs after WRAPPER code added to article-requests.tt

This commit fixed two cases:
1. Fixing "Select all" and "Clear all" links on the page.
2. Fixing the interaction with selected entries when switching between the "New" and "Pending" tabs.

To test:
(Create article requests if they don't exist)

Case 1:
1. Go to /cgi-bin/koha/circ/article-requests.pl and don't switch between tabs.
2. Click on "Select all" and verify that it's not working -- the checkboxes are not selected.
3. Activate any checkbox and click "Clear all" -- the checkbox is not cleared.
4. Apply the patch.
5. Restart the page and, without switching between tabs, repeat steps 2 and 3. Verify that it works now.

Case 2:
1. Go to /cgi-bin/koha/circ/article-requests.pl.
2. Switch between tabs and leave "New" active.
3. Select any entry and delete or move it to another tab.
4. Verify that the entry remains in the table until the page reloads.
5. Apply the patch.
6. Restart the page and repeat steps 2 and 3. Make sure that everything works now.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/circ/article-requests.tt