https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 --- Comment #1 from Didier Gautheron <didier.gautheron@biblibre.com> --- Created attachment 149582 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149582&action=edit Bug 33513: (follow-up) Remove whitespace in batch report lists Bug 32368 added whitespaces, remove them. To test: 1 - Create a report like: SELECT biblionumber FROM biblio 2 - Run it 3 - Try the various options from the 'Batch operations' button 4 - The operations don't find the biblios 5 - Create a report like: SELECT cardnumber FROM borrowers 6 - Run it 7 - Test the batch operations 8 - They fail 9 - Apply patch 10 - Run each report again and test batch operations 11 - They work! -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #2 from David Nind <david@davidnind.com> --- For me (testing using koha-testing-docker): 1. For the first report (step 1), I had no issues before the patch was applied. I was able to successfully use the batch record modification, batch record deletion, and add to list options. Things worked the same after the patch was applied. 2. For the second report (step 5), I did have issues that the patch fixed. Before the patch was applied, there was a warning when the batch patron modification was used: "Warning, the following card numbers were not found:". The odd thing about this is that card number 1 was able to be modified, and was the only one there wasn't a warning about. After the patch was applied, all were able to be modified. On the basis that it does fix things, just not exactly as set out in the test plan, I will sign off. Maybe you could comment on why I got different results than the test plan, which may help the QA Team? Testing notes (using KTD): 1. To find the batch operations after running a report, look under 'Total number of results' - there should be buttons saying: 'Batch operations with 20 visible records', 'Rows per page: 20', and 'Hide data menus'. Clicking on the 'Batch operations...' gives you some options, and clicking on 'Rows per page: ...' lets you change the number of results displayed. (It took me a while to find this, even though it was staring me in the face! 8-)) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #149582|0 |1 is obsolete| | --- Comment #3 from David Nind <david@davidnind.com> --- Created attachment 149628 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149628&action=edit Bug 33513: (follow-up) Remove whitespace in batch report lists Bug 32368 added whitespaces, remove them. To test: 1 - Create a report like: SELECT biblionumber FROM biblio 2 - Run it 3 - Try the various options from the 'Batch operations' button 4 - The operations don't find the biblios 5 - Create a report like: SELECT cardnumber FROM borrowers 6 - Run it 7 - Test the batch operations 8 - They fail 9 - Apply patch 10 - Run each report again and test batch operations 11 - They work! Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.05.00 released in| | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable Version(s)|23.05.00 |23.05.00,22.11.06 released in| | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33513 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |RESOLVED CC| |lucas@bywatersolutions.com Resolution|--- |FIXED --- Comment #7 from Lucas Gass <lucas@bywatersolutions.com> --- Missing dependencies for 22.05.x, no backport. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.