The select all/clear all links to select items for editing or deleting on the serial collection page are broken and there is no reaction when clicking on them. To test: * Add a subscription * From the subscription detail page, click on serial collection link in left hand navigation * Create some additional issues * Clear all link for selecting issues to edit won't work, select all link neither
I found the same issues on another page – /cgi-bin/koha/circ/article-requests.pl It is possible that problems occur on other pages where wrappers have been applied. I will check and prepare a correction.
Created attachment 150250 [details] [review] 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.
Created attachment 150251 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to serials-collection.tt To test: * Add a subscription * From the subscription detail page, click on serial collection link in left hand navigation * Create some additional issues * "Clear all" link for selecting issues to edit won't work, "Select all" link neither * Apply the patch * * "Clear all" link for selecting issues to edit works now, "Select all" link works too.
Created attachment 150298 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to modborrowers.tt The submitted numbers were not processed by the search query. To test: 1. Go to /cgi-bin/koha/circ/modborrowers.pl 2. On the relevant tab, enter the card number or the borrower number in the text area, or upload a text file with a relevant list of numbers. 3. Click "Continue" and verify that a message appears stating "No patron card numbers or borrowernumbers given". 4. Apply the patch. 5. Restart the page and repeat steps 2 and 3. Make sure you receive the correct response -- a list of borrowers or a message indicating that the specified numbers were not found.
Hi Slava, is this ready for testing?
Created attachment 150377 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to export.tt The list of CSV profiles was not displayed if the CSV format was selected in the output format settings. To test: 1. Go to /cgi-bin/koha/tools/export.pl 2. At the bottom of the page in the Output format, select SCV format. *If SCV format is not active in the drop-down File format menu, create any profile to Usage export records on the link /cgi-bin/koha/tools/csv-profiles.pl 3. Note that the list of SCV profiles is not displayed. 4. Apply the patch. 5. Restart the page and repeat steps 2. 6. Verify that when you select the CSV format, the list of CSV profiles appears.
Hi Katrin! Here, I found another case, which I added above. That's all I found, for now, can be test it. If I find something later, I'll create a new ticket.
Created attachment 150457 [details] [review] 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>
Created attachment 150458 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to serials-collection.tt To test: * Add a subscription * From the subscription detail page, click on serial collection link in left hand navigation * Create some additional issues * "Clear all" link for selecting issues to edit won't work, "Select all" link neither * Apply the patch * * "Clear all" link for selecting issues to edit works now, "Select all" link works too. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Created attachment 150459 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to modborrowers.tt The submitted numbers were not processed by the search query. To test: 1. Go to /cgi-bin/koha/circ/modborrowers.pl 2. On the relevant tab, enter the card number or the borrower number in the text area, or upload a text file with a relevant list of numbers. 3. Click "Continue" and verify that a message appears stating "No patron card numbers or borrowernumbers given". 4. Apply the patch. 5. Restart the page and repeat steps 2 and 3. Make sure you receive the correct response -- a list of borrowers or a message indicating that the specified numbers were not found. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Created attachment 150460 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to export.tt The list of CSV profiles was not displayed if the CSV format was selected in the output format settings. To test: 1. Go to /cgi-bin/koha/tools/export.pl 2. At the bottom of the page in the Output format, select SCV format. *If SCV format is not active in the drop-down File format menu, create any profile to Usage export records on the link /cgi-bin/koha/tools/csv-profiles.pl 3. Note that the list of SCV profiles is not displayed. 4. Apply the patch. 5. Restart the page and repeat steps 2. 6. Verify that when you select the CSV format, the list of CSV profiles appears. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
I tested the 4.5 scenarios (serial collection, article requests (cases 1 and 2), patron batch modification and catalog export) and everything works with the patch. Thank you!
Created attachment 150549 [details] [review] 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>
Created attachment 150550 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to serials-collection.tt To test: * Add a subscription * From the subscription detail page, click on serial collection link in left hand navigation * Create some additional issues * "Clear all" link for selecting issues to edit won't work, "Select all" link neither * Apply the patch * * "Clear all" link for selecting issues to edit works now, "Select all" link works too. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Created attachment 150551 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to modborrowers.tt The submitted numbers were not processed by the search query. To test: 1. Go to /cgi-bin/koha/circ/modborrowers.pl 2. On the relevant tab, enter the card number or the borrower number in the text area, or upload a text file with a relevant list of numbers. 3. Click "Continue" and verify that a message appears stating "No patron card numbers or borrowernumbers given". 4. Apply the patch. 5. Restart the page and repeat steps 2 and 3. Make sure you receive the correct response -- a list of borrowers or a message indicating that the specified numbers were not found. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 150552 [details] [review] Bug 33590: Fix DOM id name for tabs after WRAPPER code added to export.tt The list of CSV profiles was not displayed if the CSV format was selected in the output format settings. To test: 1. Go to /cgi-bin/koha/tools/export.pl 2. At the bottom of the page in the Output format, select CSV format. *If CSV format is not active in the drop-down File format menu, create any profile to Usage export records on the link /cgi-bin/koha/tools/csv-profiles.pl 3. Note that the list of CSV profiles is not displayed. 4. Apply the patch. 5. Restart the page and repeat steps 2. 6. Verify that when you select the CSV format, the list of CSV profiles appears. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
*** Bug 33679 has been marked as a duplicate of this bug. ***
Pushed to master for 23.05. Nice work everyone, thanks!
Many hands makes light work, thankyou everyone! Pushed to 22.11.x for the next release
Not needed for 22.05.x, no backport