Summary: | Using Browser "Back" button in Batch Record Modification causes biblio options to be displayed | ||
---|---|---|---|
Product: | Koha | Reporter: | Philip Orr <philip.orr> |
Component: | Cataloging | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | jonathan.druart, m.de.rooy, phil |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
23.05.00,22.11.07
|
|
Circulation function: | |||
Bug Depends on: | 32914 | ||
Bug Blocks: | |||
Attachments: |
Bug 33624: Hide the "list" tab if authority is selected
Bug 33624: Hide the "list" tab if authority is selected Bug 33624: Hide the "list" tab if authority is selected |
Description
Philip Orr
2023-04-27 07:37:35 UTC
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 |