First seen in Bug #32914. If you enter Koha's Batch record modification tool, select Authority records, select a template and then hit continue without actually uploading a file or entering record numbers, the error message "No records were modified." will be displayed. All correct so far. If you then don't click on "New batch record modification" and instead use the back button in Firefox to go back one page, you will land in batch record modification with the option "Authority records" selected but with the tab "Select a list of records" (meant for lists of Biblios) now again displayed and usable. Since biblio records and authorities records could have similar biblionumbers, this could lead to a situation where you're accidentally modifying the wrong records. To recreate: - Make sure you have at least one template for MARC Modification - Also make sure you have at least one list of biblio records - navigate to Cataloging -> Batch record modification - Check the "Authorities" radio button - Select your template and continue - No records were modified -> use your browser's Back button to go back one page - The "Authorities" radio button should still be selected, but you should now see the tab "Select a list of records".
I am not sure I understand. "Select a list of records" exists for both biblios and authorities.
Oops, sorry - forgot to mention that you need to have applied the patch for bug 32914 first. https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32914 New steps to recreate: - Apply the patch for bug 32914 - Make sure you have at least one template for MARC Modification - Also make sure you have at least one list of biblio records - navigate to Cataloging -> Batch record modification - Check the "Authorities" radio button - Select your template and continue - No records were modified -> use your browser's Back button to go back one page - The "Authorities" radio button should still be selected, but you should now see the tab "Select a list of records".
(In reply to Philip Orr from comment #2) > Oops, sorry - forgot to mention that you need to have applied the patch for > bug 32914 first. > https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32914 > > New steps to recreate: > - Apply the patch for bug 32914 > - Make sure you have at least one template for MARC Modification > - Also make sure you have at least one list of biblio records > - navigate to Cataloging -> Batch record modification > - Check the "Authorities" radio button > - Select your template and continue > - No records were modified -> use your browser's Back button to go back one > page > - The "Authorities" radio button should still be selected, > but you should now see the tab "Select a list of records". Ho right, the list tab, I meant "Enter a list of record numbers" in my previous comment. I didn't notice the list one as I didn't have one created. I do recreate the problem on master.
Created attachment 151072 [details] [review] Bug 33624: Hide the "list" tab if authority is selected The "Select a list of records" should only be available for biblio records. But if the page is accessed using the back button of the browser it will be displayed for authorities as well. Test plan: - Make sure you have at least one template for MARC Modification - Also make sure you have at least one list of biblio records - navigate to Cataloging -> Batch record modification - Check the "Authorities" radio button - Select your template and continue - No records were modified -> use your browser's Back button to go back one page - The "Authorities" radio button should still be selected => Without this patch you see the tab "Select a list of records" => With this patch applied it's hidden when the page is loaded
Bug is in master, without bug 32914. But it's PQA so I built the fix on top of it.
Created attachment 151327 [details] [review] Bug 33624: Hide the "list" tab if authority is selected The "Select a list of records" should only be available for biblio records. But if the page is accessed using the back button of the browser it will be displayed for authorities as well. Test plan: - Make sure you have at least one template for MARC Modification - Also make sure you have at least one list of biblio records - navigate to Cataloging -> Batch record modification - Check the "Authorities" radio button - Select your template and continue - No records were modified -> use your browser's Back button to go back one page - The "Authorities" radio button should still be selected => Without this patch you see the tab "Select a list of records" => With this patch applied it's hidden when the page is loaded Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
I dont understand what this patch actually is supposed to solve. If the list is not displayed but still contains the numbers entered, why hide it ?
Created attachment 151335 [details] [review] Bug 33624: Hide the "list" tab if authority is selected The "Select a list of records" should only be available for biblio records. But if the page is accessed using the back button of the browser it will be displayed for authorities as well. Test plan: - Make sure you have at least one template for MARC Modification - Also make sure you have at least one list of biblio records - navigate to Cataloging -> Batch record modification - Check the "Authorities" radio button - Select your template and continue - No records were modified -> use your browser's Back button to go back one page - The "Authorities" radio button should still be selected => Without this patch you see the tab "Select a list of records" => With this patch applied it's hidden when the page is loaded Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to stable for 22.11.x