Summary: | Don't alert when replacing an authority record via Z39.50 | ||
---|---|---|---|
Product: | Koha | Reporter: | Phil Ringnalda <phil> |
Component: | MARC Authority data support | Assignee: | Phil Ringnalda <phil> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | 1joynelson, lucas |
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: |
20.05.00, 19.11.06
|
|
Circulation function: | |||
Attachments: |
Bug 25235: Rename the button instead of alerting when replacing an existing record
Bug 25235: Rename the button instead of alerting when replacing an existing record Bug 25235: (follow-up) Shorten label and apply to addbiblio too Bug 25235: Rename the button instead of alerting when replacing an existing record |
Description
Phil Ringnalda
2020-04-22 04:30:40 UTC
Created attachment 103439 [details] [review] Bug 25235: Rename the button instead of alerting when replacing an existing record Test plan: Pre-patch 1 - Go to the main Authorities page, search for any authority, click the Actions menubutton and choose Edit 2 - Note the button saying Z39.50 search 3 - Note the modal alert forcing you to click it 4 - Cancel and cancel again, and in the New authority menubutton choose Default 5 - Click the button saying Z39.50 search again, note that it warns you about replacing your totally blank record Apply this patch 6 - Edit an existing authority, note the button says "Replace record via Z39.50/SRU search" 2 - Click the button, verify it still opens the search window with the main entry of the record filled in without an alert 3 - Create a new authority, note the button says "Z39.50/SRU search" 4 - Click the button, verify it still opens the search window but without an alert Created attachment 103472 [details] [review] Bug 25235: Rename the button instead of alerting when replacing an existing record Test plan: Pre-patch 1 - Go to the main Authorities page, search for any authority, click the Actions menubutton and choose Edit 2 - Note the button saying Z39.50 search 3 - Note the modal alert forcing you to click it 4 - Cancel and cancel again, and in the New authority menubutton choose Default 5 - Click the button saying Z39.50 search again, note that it warns you about replacing your totally blank record Apply this patch 6 - Edit an existing authority, note the button says "Replace record via Z39.50/SRU search" 2 - Click the button, verify it still opens the search window with the main entry of the record filled in without an alert 3 - Create a new authority, note the button says "Z39.50/SRU search" 4 - Click the button, verify it still opens the search window but without an alert Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 103473 [details] [review] Bug 25235: (follow-up) Shorten label and apply to addbiblio too This follow-up changes the new label from "Replace record via Z39.50/SRU search" to the more button-friendly "Replace via Z39.50/SRU" This patch also applies the same change to the basic MARC editor interface. To test, apply the patch and follow the previous test plan to confirm the change on the authorities page. Go to Cataloging and edit an existing record in the basic MARC editor. You should see the "Replace via Z39.50/SRU" button, and clicking it should trigger the Z39.50 search window without asking for confirmation. Create a new record in the basic MARC editor. The button should be labeled "Z39.50/SRU" search, and it should trigger the popup window without asking for confirmation. Created attachment 103476 [details] [review] Bug 25235: Rename the button instead of alerting when replacing an existing record Test plan: Pre-patch 1 - Go to the main Authorities page, search for any authority, click the Actions menubutton and choose Edit 2 - Note the button saying Z39.50 search 3 - Note the modal alert forcing you to click it 4 - Cancel and cancel again, and in the New authority menubutton choose Default 5 - Click the button saying Z39.50 search again, note that it warns you about replacing your totally blank record Apply this patch 6 - Edit an existing authority, note the button says "Replace record via Z39.50/SRU search" 2 - Click the button, verify it still opens the search window with the main entry of the record filled in without an alert 3 - Create a new authority, note the button says "Z39.50/SRU search" 4 - Click the button, verify it still opens the search window but without an alert Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Nice work everyone! Pushed to master for 20.05 backported to 19.11.x for 19.11.06 enhancement will not be backported to 19.05.x |